aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2013-01-11 04:44:20 +0100
committerMichael Niedermayer <michaelni@gmx.at>2013-01-11 04:46:49 +0100
commit23318a57358358e7a4dc551e830e4503f0638cfe (patch)
tree30a1f391c92520f4786057a2850b65ebe566cc6e
parent8b47058c61af83c28231b860d46ee754ed7a9310 (diff)
downloadffmpeg-23318a57358358e7a4dc551e830e4503f0638cfe.tar.gz
error_concealment: Check that the picture is not in a half setup state.
Fixes state becoming inconsistent Fixes a null pointer dereference Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavcodec/error_resilience.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/libavcodec/error_resilience.c b/libavcodec/error_resilience.c
index 01f7424904..2b6bc42ade 100644
--- a/libavcodec/error_resilience.c
+++ b/libavcodec/error_resilience.c
@@ -924,6 +924,12 @@ void ff_er_frame_end(MpegEncContext *s)
return;
};
+ if ( s->picture_structure == PICT_FRAME
+ && s->current_picture.f.linesize[0] != s->current_picture_ptr->f.linesize[0]) {
+ av_log(s->avctx, AV_LOG_ERROR, "Error concealment not possible, frame not fully initialized\n");
+ return;
+ }
+
if (s->current_picture.f.motion_val[0] == NULL) {
av_log(s->avctx, AV_LOG_ERROR, "Warning MVs not available\n");