Promote to uint32_t before left shift
authorYaowu Xu <yaowu@google.com>
Tue, 17 May 2016 18:39:57 +0000 (11:39 -0700)
committerYaowu Xu <yaowu@google.com>
Tue, 17 May 2016 18:39:57 +0000 (11:39 -0700)
This commit change to promote uint8_t explicitly to uint32_t before
left shift operation.

BUG=https://bugs.chromium.org/p/chromium/issues/detail?id=612021

Change-Id: Id7059154efb5bdfa45889dabe72aaafd46d79f23

vp9/decoder/vp9_decoder.c

index ea018d0..935c04f 100644 (file)
@@ -508,7 +508,7 @@ vpx_codec_err_t vp9_parse_superframe_index(const uint8_t *data,
         uint32_t this_sz = 0;
 
         for (j = 0; j < mag; ++j)
-          this_sz |= (*x++) << (j * 8);
+          this_sz |= ((uint32_t)(*x++)) << (j * 8);
         sizes[i] = this_sz;
       }
       *count = frames;