aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2015-11-14 14:29:02 +0100
committerMichael Niedermayer <michael@niedermayer.cc>2015-11-19 03:51:39 +0100
commit7ab1f5b0d4b621dbd78bc901d38fcad6e19c375e (patch)
tree544eb7c7540ce0dc00c57f068706142d86925bc4
parent1c73f9d84c8c0726fca4627381b77b99f09af99a (diff)
downloadffmpeg-7ab1f5b0d4b621dbd78bc901d38fcad6e19c375e.tar.gz
avcodec/dpx: Move need_align to act per line
Fixes out of array read Fixes: 61cf123c081ee2bb774d307c75bdb99e/asan_heap-oob_1224f76_5546_bee833ffae73f752b489b9eeaac52db7.dpx Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> (cherry picked from commit c8aaae8e0f1519bc99bd717ea3067c9cfdb68def) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavcodec/dpx.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c
index 66d8428951..3b45b8eb98 100644
--- a/libavcodec/dpx.c
+++ b/libavcodec/dpx.c
@@ -348,11 +348,11 @@ static int decode_frame(AVCodecContext *avctx,
// For 12 bit, ignore alpha
if (elements == 4)
buf += 2;
- // Jump to next aligned position
- buf += need_align;
}
for (i = 0; i < 3; i++)
ptr[i] += p->linesize[i];
+ // Jump to next aligned position
+ buf += need_align;
}
break;
case 16: