[FFmpeg-cvslog] avformat/sccdec: Simplify cleanup after read_header failure
Andreas Rheinhardt
git at videolan.org
Thu Jul 8 16:45:17 EEST 2021
ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Sat Mar 21 18:31:06 2020 +0100| [9ef7582446bf792e11ff55cc5f7dbaf7a1d0d89c] | committer: Andreas Rheinhardt
avformat/sccdec: Simplify cleanup after read_header failure
by setting the FF_FMT_INIT_CLEANUP flag.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=9ef7582446bf792e11ff55cc5f7dbaf7a1d0d89c
---
libavformat/sccdec.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/libavformat/sccdec.c b/libavformat/sccdec.c
index 5ad21a1f8b..e59e015bbc 100644
--- a/libavformat/sccdec.c
+++ b/libavformat/sccdec.c
@@ -145,7 +145,7 @@ static int scc_read_header(AVFormatContext *s)
sub = ff_subtitles_queue_insert(&scc->q, out, i, 0);
if (!sub)
- goto fail;
+ return AVERROR(ENOMEM);
sub->pos = pos;
pos += i;
@@ -164,7 +164,7 @@ static int scc_read_header(AVFormatContext *s)
sub = ff_subtitles_queue_insert(&scc->q, out, i, 0);
if (!sub)
- goto fail;
+ return AVERROR(ENOMEM);
sub->pos = pos;
sub->pts = ts;
@@ -175,9 +175,6 @@ static int scc_read_header(AVFormatContext *s)
ff_subtitles_queue_finalize(s, &scc->q);
return 0;
-fail:
- ff_subtitles_queue_clean(&scc->q);
- return AVERROR(ENOMEM);
}
static int scc_read_packet(AVFormatContext *s, AVPacket *pkt)
@@ -205,6 +202,7 @@ const AVInputFormat ff_scc_demuxer = {
.name = "scc",
.long_name = NULL_IF_CONFIG_SMALL("Scenarist Closed Captions"),
.priv_data_size = sizeof(SCCContext),
+ .flags_internal = FF_FMT_INIT_CLEANUP,
.read_probe = scc_probe,
.read_header = scc_read_header,
.read_packet = scc_read_packet,
More information about the ffmpeg-cvslog
mailing list