commit 4052230a93149d6d5d54ba633d9197708287b83f
author chiyotsai <chiyotsai@google.com> Wed Mar 25 13:23:30 2020 -0700
committer Chi Yo Tsai <chiyotsai@google.com> Thu Mar 26 03:33:58 2020 +0000
tree 68baf36603ac0bebc5b06649f9325626aa0a1f36
parent d07f49c30fef5ce1c479fe51f5879af37788da31

Fix an uninitialized memory usage in skip_repeated_full_newmv

BUG=aomedia:2622

STATS_CHANGED

Change-Id: Id75d8b45012f21ac30bd4dfe94afb1f552cd0b2d

av1/encoder/rdopt.c

diff --git a/av1/encoder/rdopt.c b/av1/encoder/rdopt.c
index 2ed7206..b381d95 100644
--- a/av1/encoder/rdopt.c
+++ b/av1/encoder/rdopt.c
@@ -2301,6 +2301,10 @@
   const int base_rate =
       args->ref_frame_cost + args->single_comp_cost + ref_mv_cost;
   for (int ref_mv_idx = 0; ref_mv_idx < ref_set; ++ref_mv_idx) {
+    mode_info[ref_mv_idx].full_search_mv.as_int = INVALID_MV;
+    mode_info[ref_mv_idx].mv.as_int = INVALID_MV;
+    mode_info[ref_mv_idx].rd = INT64_MAX;
+
     if (!mask_check_bit(idx_mask, ref_mv_idx)) {
       // MV did not perform well in simple translation search. Skip it.
       continue;
@@ -2312,10 +2316,6 @@
               cpi->sf.inter_sf.prune_inter_modes_based_on_tpl))
         continue;
     }
-
-    mode_info[ref_mv_idx].full_search_mv.as_int = INVALID_MV;
-    mode_info[ref_mv_idx].mv.as_int = INVALID_MV;
-    mode_info[ref_mv_idx].rd = INT64_MAX;
     av1_init_rd_stats(rd_stats);
 
     mbmi->interinter_comp.type = COMPOUND_AVERAGE;