Re: [FFmpeg-devel] [PATCH v5 2/2] lavf/mpegenc: fix termination on error conditions

2022-04-01 Thread Nicolas Gaullier
>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

Re: [FFmpeg-devel] [PATCH v5 2/2] lavf/mpegenc: fix termination on error conditions

2022-03-31 Thread Andreas Rheinhardt
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..e11

[FFmpeg-devel] [PATCH v5 2/2] lavf/mpegenc: fix termination on error conditions

2022-03-25 Thread 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/mpe