commit 836b69b9a0a96cc7bb1e9eaec441150b0d64bba6
author chiyotsai <chiyotsai@google.com> Tue Apr 09 13:41:24 2019 -0700
committer Chi Yo Tsai <chiyotsai@google.com> Wed Apr 10 20:44:18 2019 +0000
tree 47595ead0af241b7367037dcf93886552d088b1e
parent 6fdf93469276de7cce28a2f98c684066596c532d

Fix a potential segfault when NSTEP is used on scaled reference

This commit makes sure that search_sites_cfg is always update once per
frame to avoid segfault due to using incorrect y_stride on scaled
refrences.

Change-Id: If11a7ea41be7880c3e179fe416ab13f641e8f7d7

av1/encoder/mcomp.c

diff --git a/av1/encoder/mcomp.c b/av1/encoder/mcomp.c
index a3f7d25..1a64d1a 100644
--- a/av1/encoder/mcomp.c
+++ b/av1/encoder/mcomp.c
@@ -123,6 +123,7 @@
 
   cfg->ss[0].mv.col = cfg->ss[0].mv.row = 0;
   cfg->ss[0].offset = 0;
+  cfg->stride = stride;
 
   for (len = MAX_FIRST_STEP; len > 0; len /= 2) {
     // Generate offsets for 4 search sites per step.
@@ -144,6 +145,7 @@
 
   cfg->ss[0].mv.col = cfg->ss[0].mv.row = 0;
   cfg->ss[0].offset = 0;
+  cfg->stride = stride;
 
   for (len = MAX_FIRST_STEP; len > 0; len /= 2) {
     // Generate offsets for 8 search sites per step.