[Libva] [PATCH V1] libva-intel-driver: fix i965 encoder wrong bits shift operation

[Xiang, Haihao]

Thanks for the patch, applied.

> From: Kuang-che Wu <kcwu at chromium.org>
> 
> shift uint32_t by 32 bits is undefined behavior.
> 
> For this particular case: when invoke avc_bitstream_put_ui() with 32
> bits value at byte position of multiple of 4, existing 32 bits garbage
> data in the buffer may be retained instead of cleared. The result is,
> the position of NALU start code (0x00000001) looks like overwritten by
> garbage value.
> 
> Patch has been tested and used upstream:
> https://chromium-review.googlesource.com/#/c/410541/
> 
> Signed-off-by: Kuang-che Wu <kcwu at chromium.org>
> Signed-off-by: Sean V Kelley <seanvk at posteo.de>
> ---
>  src/i965_encoder_utils.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/src/i965_encoder_utils.c b/src/i965_encoder_utils.c
> index ac58cd1..8db1b87 100644
> --- a/src/i965_encoder_utils.c
> +++ b/src/i965_encoder_utils.c
> @@ -134,7 +134,11 @@ avc_bitstream_put_ui(avc_bitstream *bs, unsigned int val,
> int size_in_bits)
>          bs->buffer[pos] = (bs->buffer[pos] << size_in_bits | val);
>      } else {
>          size_in_bits -= bit_left;
> -        bs->buffer[pos] = (bs->buffer[pos] << bit_left) | (val >>
> size_in_bits);
> +        if (bit_left == 32) {
> +            bs->buffer[pos] = val;
> +        } else {
> +            bs->buffer[pos] = (bs->buffer[pos] << bit_left) | (val >>
> size_in_bits);
> +        }
>          bs->buffer[pos] = swap32(bs->buffer[pos]);
>  
>          if (pos + 1 == bs->max_size_in_dword) {