motionpixels: Clip YUV values after applying a gradient.
Prevents illegal reads on truncated and malformed input. CC: libav-stable@libav.org (cherry picked from commitb5da848fac
) Signed-off-by: Reinhard Tartler <siretart@tauware.de> (cherry picked from commitaaa6a66677
) Signed-off-by: Reinhard Tartler <siretart@tauware.de> (cherry picked from commit50073e2395
) Signed-off-by: Reinhard Tartler <siretart@tauware.de> (cherry picked from commit 2134e7f6e88959513ba1713ad6fd7a7c8d5a0f41) Signed-off-by: Reinhard Tartler <siretart@tauware.de>
This commit is contained in:

committed by
Reinhard Tartler

parent
b2ac7e585e
commit
6c9b404dba
@@ -239,10 +239,13 @@ static void mp_decode_line(MotionPixelsContext *mp, GetBitContext *gb, int y)
|
|||||||
p = mp_get_yuv_from_rgb(mp, x - 1, y);
|
p = mp_get_yuv_from_rgb(mp, x - 1, y);
|
||||||
} else {
|
} else {
|
||||||
p.y += mp_gradient(mp, 0, mp_get_vlc(mp, gb));
|
p.y += mp_gradient(mp, 0, mp_get_vlc(mp, gb));
|
||||||
|
p.y = av_clip(p.y, 0, 31);
|
||||||
if ((x & 3) == 0) {
|
if ((x & 3) == 0) {
|
||||||
if ((y & 3) == 0) {
|
if ((y & 3) == 0) {
|
||||||
p.v += mp_gradient(mp, 1, mp_get_vlc(mp, gb));
|
p.v += mp_gradient(mp, 1, mp_get_vlc(mp, gb));
|
||||||
|
p.v = av_clip(p.v, -32, 31);
|
||||||
p.u += mp_gradient(mp, 2, mp_get_vlc(mp, gb));
|
p.u += mp_gradient(mp, 2, mp_get_vlc(mp, gb));
|
||||||
|
p.u = av_clip(p.u, -32, 31);
|
||||||
mp->hpt[((y / 4) * mp->avctx->width + x) / 4] = p;
|
mp->hpt[((y / 4) * mp->avctx->width + x) / 4] = p;
|
||||||
} else {
|
} else {
|
||||||
p.v = mp->hpt[((y / 4) * mp->avctx->width + x) / 4].v;
|
p.v = mp->hpt[((y / 4) * mp->avctx->width + x) / 4].v;
|
||||||
@@ -266,9 +269,12 @@ static void mp_decode_frame_helper(MotionPixelsContext *mp, GetBitContext *gb)
|
|||||||
p = mp_get_yuv_from_rgb(mp, 0, y);
|
p = mp_get_yuv_from_rgb(mp, 0, y);
|
||||||
} else {
|
} else {
|
||||||
p.y += mp_gradient(mp, 0, mp_get_vlc(mp, gb));
|
p.y += mp_gradient(mp, 0, mp_get_vlc(mp, gb));
|
||||||
|
p.y = av_clip(p.y, 0, 31);
|
||||||
if ((y & 3) == 0) {
|
if ((y & 3) == 0) {
|
||||||
p.v += mp_gradient(mp, 1, mp_get_vlc(mp, gb));
|
p.v += mp_gradient(mp, 1, mp_get_vlc(mp, gb));
|
||||||
|
p.v = av_clip(p.v, -32, 31);
|
||||||
p.u += mp_gradient(mp, 2, mp_get_vlc(mp, gb));
|
p.u += mp_gradient(mp, 2, mp_get_vlc(mp, gb));
|
||||||
|
p.u = av_clip(p.u, -32, 31);
|
||||||
}
|
}
|
||||||
mp->vpt[y] = p;
|
mp->vpt[y] = p;
|
||||||
mp_set_rgb_from_yuv(mp, 0, y, &p);
|
mp_set_rgb_from_yuv(mp, 0, y, &p);
|
||||||
|
Reference in New Issue
Block a user