[FFmpeg-cvslog] Handle an invalid extra mpeg2 picture header following a frame-encoded picture .
Joseph Artsimovich
git at videolan.org
Fri Mar 1 16:05:14 CET 2013
ffmpeg | branch: master | Joseph Artsimovich <joseph at mirriad.com> | Fri Mar 1 13:37:28 2013 +0000| [4a6bd346373b63d13c95cc2303346b0e971f0358] | committer: Michael Niedermayer
Handle an invalid extra mpeg2 picture header following a frame-encoded picture.
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=4a6bd346373b63d13c95cc2303346b0e971f0358
---
libavcodec/mpeg12.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/libavcodec/mpeg12.c b/libavcodec/mpeg12.c
index 5e95ad7..9d2743a 100644
--- a/libavcodec/mpeg12.c
+++ b/libavcodec/mpeg12.c
@@ -2269,6 +2269,7 @@ static int decode_chunks(AVCodecContext *avctx,
const uint8_t *buf_end = buf + buf_size;
int ret, input_size;
int last_code = 0;
+ int picture_start_code_seen = 0;
for (;;) {
/* find next start code */
@@ -2319,6 +2320,14 @@ static int decode_chunks(AVCodecContext *avctx,
break;
case PICTURE_START_CODE:
+ if (picture_start_code_seen && s2->picture_structure == PICT_FRAME) {
+ /* If it's a frame picture, there can't be more than one picture header.
+ Yet, it does happen and we need to handle it. */
+ av_log(avctx, AV_LOG_WARNING, "ignoring extra picture following a frame-picture\n");
+ break;
+ }
+ picture_start_code_seen = 1;
+
if (s2->width <= 0 || s2->height <= 0) {
av_log(avctx, AV_LOG_ERROR, "%dx%d is invalid\n", s2->width, s2->height);
return AVERROR_INVALIDDATA;
More information about the ffmpeg-cvslog
mailing list