[FFmpeg-devel] [PATCH v5 2/2] lavf/mpegenc: fix termination on error conditions
Avoid an infinite 'retry' loop in output_packet when flushing. Signed-off-by: Nicolas Gaullier --- libavformat/mpegenc.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c index e0955a7d33..e113a42867 100644 --- a/libavformat/mpegenc.c +++ b/libavformat/mpegenc.c @@ -1002,7 +1002,7 @@ static int output_packet(AVFormatContext *ctx, int flush) MpegMuxContext *s = ctx->priv_data; AVStream *st; StreamInfo *stream; -int i, avail_space = 0, es_size, trailer_size; +int i, has_avail_data = 0, avail_space = 0, es_size, trailer_size; int best_i = -1; int best_score = INT_MIN; int ignore_constraints = 0; @@ -1028,6 +1028,7 @@ retry: if (avail_data == 0) continue; av_assert0(avail_data > 0); +has_avail_data = 1; if (space < s->packet_size && !ignore_constraints) continue; @@ -1048,6 +1049,8 @@ retry: int64_t best_dts = INT64_MAX; int has_premux = 0; +if (!has_avail_data) +return 0; for (i = 0; i < ctx->nb_streams; i++) { AVStream *st = ctx->streams[i]; StreamInfo *stream = st->priv_data; -- 2.34.0.windows.1 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH v5 2/2] lavf/mpegenc: fix termination on error conditions
Nicolas Gaullier: > Avoid an infinite 'retry' loop in output_packet when flushing. > > Signed-off-by: Nicolas Gaullier > --- > libavformat/mpegenc.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c > index e0955a7d33..e113a42867 100644 > --- a/libavformat/mpegenc.c > +++ b/libavformat/mpegenc.c > @@ -1002,7 +1002,7 @@ static int output_packet(AVFormatContext *ctx, int > flush) > MpegMuxContext *s = ctx->priv_data; > AVStream *st; > StreamInfo *stream; > -int i, avail_space = 0, es_size, trailer_size; > +int i, has_avail_data = 0, avail_space = 0, es_size, trailer_size; > int best_i = -1; > int best_score = INT_MIN; > int ignore_constraints = 0; > @@ -1028,6 +1028,7 @@ retry: > if (avail_data == 0) > continue; > av_assert0(avail_data > 0); > +has_avail_data = 1; > > if (space < s->packet_size && !ignore_constraints) > continue; > @@ -1048,6 +1049,8 @@ retry: > int64_t best_dts = INT64_MAX; > int has_premux = 0; > > +if (!has_avail_data) > +return 0; > for (i = 0; i < ctx->nb_streams; i++) { > AVStream *st = ctx->streams[i]; > StreamInfo *stream = st->priv_data; in case of errors, the context is left in an inconsistent state: The PacketDesc linked-list claims that there is data in the FIFO although this is wrong. I always prefer avoiding such scenarios over fixing them lateron. In this case, fixing them would mean growing the FIFO before allocating the new PacketDesc (if the FIFO needs growing at all). Shall I do this or do you want to? (In any case, thanks for reporting this issue.) - Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH v5 2/2] lavf/mpegenc: fix termination on error conditions
>in case of errors, the context is left in an inconsistent state: The >PacketDesc linked-list claims that there is data in the FIFO although this is >wrong. I always prefer avoiding such scenarios over fixing them lateron. In >this case, fixing them would mean growing the FIFO before >allocating the new >PacketDesc (if the FIFO needs growing at all). Shall I do this or do you want >to? >(In any case, thanks for reporting this issue.) > >- Andreas I fully understand. I think the most time-effective is to let you do that if you are ok, you also know this code better than me. Thanks to you Nicolas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".