commit ff731dae9770cbe797b48c2756efbe48030d59ce
author Satish Kumar Suman <satish.suman@ittiam.com> Mon Aug 05 18:38:04 2019 +0530
committer Yunqing Wang <yunqingwang@google.com> Tue Aug 06 01:15:18 2019 +0000
tree 5e44794170651785ae79e313c41e57544f3c0a87
parent 46ccbf471caddce2bec315210ff082b10a02f856

Fix bug in av1_compute_global_motion()

Sometimes num_inliers is not initialized due to early
exit present in ransac() function. Which leads to
aomedia:2449 "SEGV on unknown address".

BUG=aomedia:2449

STATS_CHANGED

Change-Id: I7335c27fbfe6829677509adfd4f326f81f8e53ab

av1/encoder/encodeframe.c

diff --git a/av1/encoder/encodeframe.c b/av1/encoder/encodeframe.c
index 0c63a5c..7cdb5c9 100644
--- a/av1/encoder/encodeframe.c
+++ b/av1/encoder/encodeframe.c
@@ -4796,6 +4796,7 @@
           for (i = 0; i < RANSAC_NUM_MOTIONS; ++i) {
             memcpy(params_by_motion[i].params, kIdentityParams,
                    (MAX_PARAMDIM - 1) * sizeof(*(params_by_motion[i].params)));
+            params_by_motion[i].num_inliers = 0;
           }
 
           av1_compute_global_motion(