]> granicus.if.org Git - libvpx/commitdiff
vp9-decoder: use long int for buffer offset.
authorJerome Jiang <jianj@google.com>
Mon, 11 Mar 2019 22:13:19 +0000 (15:13 -0700)
committerJerome Jiang <jianj@google.com>
Mon, 11 Mar 2019 22:24:37 +0000 (15:24 -0700)
integer overflow when frame size too big.

BUG=webm:1603

Change-Id: Ifbb81b5fb6a2043d09d403e7c50ab8d7bf125dca

vp9/common/vp9_reconinter.c
vp9/common/vp9_reconinter.h

index a108a65153bf8def816731332d8c9e1b6d5f709c..0834c722f6ab34366d306f3064ca620e8a5fac71 100644 (file)
@@ -178,7 +178,7 @@ static void build_inter_predictors(MACROBLOCKD *xd, int plane, int block,
       xs = sf->x_step_q4;
       ys = sf->y_step_q4;
     } else {
-      pre = pre_buf->buf + (y * pre_buf->stride + x);
+      pre = pre_buf->buf + ((int64_t)y * pre_buf->stride + x);
       scaled_mv.row = mv_q4.row;
       scaled_mv.col = mv_q4.col;
       xs = ys = 16;
index 992e30c344b06d7b5bad97ab93f05d1c0c7099c5..12b545831a897bcb8980feac4a5f576974d044e9 100644 (file)
@@ -74,11 +74,12 @@ void vp9_highbd_build_inter_predictor(
     int bd);
 #endif
 
-static INLINE int scaled_buffer_offset(int x_offset, int y_offset, int stride,
-                                       const struct scale_factors *sf) {
+static INLINE int64_t scaled_buffer_offset(int x_offset, int y_offset,
+                                           int stride,
+                                           const struct scale_factors *sf) {
   const int x = sf ? sf->scale_value_x(x_offset, sf) : x_offset;
   const int y = sf ? sf->scale_value_y(y_offset, sf) : y_offset;
-  return y * stride + x;
+  return (int64_t)y * stride + x;
 }
 
 static INLINE void setup_pred_plane(struct buf_2d *dst, uint8_t *src,