Skip to content

arm: mc: Fix out of bounds reads/writes in 8tap/bilin w2/w4 for vertical OBMC

Martin Storsjö requested to merge mstorsjo/dav1d:arm-mc-bounds-checking into master

This should fix issue #380 (closed).

I tried to add bounds checking for all cases of pixel rects where a stride can be specified and where the checks can be added easily. This uncovered a couple more cases where the MC resize functions overwrite past the right edge by a little, in resize_16bpc_ssse3, resize_8bpc_avx2 and resize_16bpc_avx2. If this is expected and tolerable, it's easy to tweak the test to use checkasm_check_pixel_padded_align to allow a little overwrite.

Merge request reports