ref: 818b10396dce15081b6cb74d76a3b050a3296910
parent: 488ba1ab9ca875e48e6ea8f0f30a6bca6b730664
author: Scott LaVarnway <slavarnway@google.com>
date: Tue Sep 11 05:14:02 EDT 2012
valgrind found motion vectors which exceeded frame boundaries Bug introduced in I02d034c70cd97b65025d59dd67c695e1db529f0b Change-Id: Ifb74ab659f3b3cf5f5b95046bacffb202d5b4d64
--- a/vp8/decoder/decodemv.c
+++ b/vp8/decoder/decodemv.c
@@ -293,25 +293,23 @@
blockmv.as_mv.row += best_mv.as_mv.row;
blockmv.as_mv.col = read_mvcomponent(bc, &mvc[1]) << 1;
blockmv.as_mv.col += best_mv.as_mv.col;
-
- mbmi->need_to_clamp_mvs |= vp8_check_mv_bounds(&blockmv,
- mb_to_left_edge,
- mb_to_right_edge,
- mb_to_top_edge,
- mb_to_bottom_edge);
}
}
else
{blockmv.as_int = abovemv.as_int;
- mbmi->need_to_clamp_mvs |= above_mb->mbmi.need_to_clamp_mvs;
}
}
else
{blockmv.as_int = leftmv.as_int;
- mbmi->need_to_clamp_mvs |= left_mb->mbmi.need_to_clamp_mvs;
}
+
+ mbmi->need_to_clamp_mvs |= vp8_check_mv_bounds(&blockmv,
+ mb_to_left_edge,
+ mb_to_right_edge,
+ mb_to_top_edge,
+ mb_to_bottom_edge);
{/* Fill (uniform) modes, mvs of jth subset.