Avoid reading uninitialized data in decodemv.c
Avoid reading uninitialized data in decodemv.c

The existing code has a case where we set a variable to equal
xd->ref_mv_stack[mbmi->ref_frame[0]][1 + mbmi->ref_mv_idx]
even for compound blocks. However, the range of allowable
values for mbmi->ref_mv_idx is determined by the ref_mv_count
for the *combined* ref frame, not for the first single ref frame.

This means that, if we have more ref-mv candidates for the combined
ref frame than for the first single ref frame, then we can sometimes
fetch uninitialized data.
In every case where this happens, we immediately overwrite
the destination with the correct mv, but it is still preferable
to avoid reading uninitialized data.

This patch moves the code block to avoid this bug. In addition,
the variable (nearmv[0]) is only used when the mode equals NEARMV,
so the condition on its assignment is changed to reflect that.

Change-Id: I3bd268dc80d8065d5189999232b8a0f826d40a95

GrokImageCompression/aom,GrokImageCompression/aom,GrokImageCompression/aom,GrokImageCompression/aom,GrokImageCompression/aom,GrokImageCompression/aom