Change order of operation to avoid ubsan warnings
authorYaowu Xu <yaowu@google.com>
Wed, 14 Dec 2016 17:37:14 +0000 (09:37 -0800)
committerYaowu Xu <yaowu@google.com>
Wed, 14 Dec 2016 17:37:14 +0000 (09:37 -0800)
This commit change an order of operation to avoid left shifts of
negative numbers.

Change-Id: I607c7eb91658c7a5ef397fc1504721d1b10e3dd6

vp8/decoder/threading.c

index 5b6200f..9f77519 100644 (file)
@@ -318,7 +318,7 @@ static void mt_decode_mb_rows(VP8D_COMP *pbi, MACROBLOCKD *xd,
 
     xd->left_available = 0;
 
-    xd->mb_to_top_edge = -((mb_row * 16)) << 3;
+    xd->mb_to_top_edge = -((mb_row * 16) << 3);
     xd->mb_to_bottom_edge = ((pc->mb_rows - 1 - mb_row) * 16) << 3;
 
     if (pbi->common.filter_level) {