From 0a055f463a60af764c083fd0ba3112037557de4b Mon Sep 17 00:00:00 2001 From: Josh de Kock Date: Wed, 26 Sep 2018 10:35:04 +0100 Subject: [PATCH] lavc/h264dec: don't error out when receiving multiple IDR slices This error isn't particularly helpful as checking for mixed IDR/non-IDR NALUs would need to be done at a higher level to actually be accurate. Removing the error allows an API user to send individual slice NALUs (i.e. incomplete frames) so they can take advantage of slice threading. The ticket which this error was added for (#4408) no longer segfaults after removing this error (as the bug was likely fixed more properly elsewhere). --- libavcodec/h264dec.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index 7b4c5c76ea..00d922fbe9 100644 --- a/libavcodec/h264dec.c +++ b/libavcodec/h264dec.c @@ -657,11 +657,6 @@ static int decode_nal_units(H264Context *h, const uint8_t *buf, int buf_size) goto end; } if(!idr_cleared) { - if (h->current_slice && (avctx->active_thread_type & FF_THREAD_SLICE)) { - av_log(h, AV_LOG_ERROR, "invalid mixed IDR / non IDR frames cannot be decoded in slice multithreading mode\n"); - ret = AVERROR_INVALIDDATA; - goto end; - } idr(h); // FIXME ensure we don't lose some frames if there is reordering } idr_cleared = 1;