aboutsummaryrefslogtreecommitdiffstats
path: root/libavformat/anm.c
diff options
context:
space:
mode:
authorLaurent Aimar <fenrir@videolan.org>2011-10-01 00:44:57 +0200
committerMichael Niedermayer <michaelni@gmx.at>2011-10-01 02:41:39 +0200
commitd39d7122e34d2cf994d6dc474fe0c8bee2f7fcfd (patch)
tree90b7c2fdc3dd6db907d5b1771893edeecc892f4a /libavformat/anm.c
parent7afe9e5638242a3210a0fc378e34e3af41e29176 (diff)
downloadffmpeg-d39d7122e34d2cf994d6dc474fe0c8bee2f7fcfd.tar.gz
Fix double free on error in Deluxe Paint Animation demuxer.
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavformat/anm.c')
-rw-r--r--libavformat/anm.c23
1 files changed, 8 insertions, 15 deletions
diff --git a/libavformat/anm.c b/libavformat/anm.c
index e35dc1517d..09e7dc7217 100644
--- a/libavformat/anm.c
+++ b/libavformat/anm.c
@@ -134,18 +134,17 @@ static int read_header(AVFormatContext *s,
/* color cycling and palette data */
st->codec->extradata_size = 16*8 + 4*256;
st->codec->extradata = av_mallocz(st->codec->extradata_size + FF_INPUT_BUFFER_PADDING_SIZE);
- if (!st->codec->extradata) {
- ret = AVERROR(ENOMEM);
- goto close_and_return;
- }
+ if (!st->codec->extradata)
+ return AVERROR(ENOMEM);
+
ret = avio_read(pb, st->codec->extradata, st->codec->extradata_size);
if (ret < 0)
- goto close_and_return;
+ return ret;
/* read page table */
ret = avio_seek(pb, anm->page_table_offset, SEEK_SET);
if (ret < 0)
- goto close_and_return;
+ return ret;
for (i = 0; i < MAX_PAGES; i++) {
Page *p = &anm->pt[i];
@@ -156,21 +155,15 @@ static int read_header(AVFormatContext *s,
/* find page of first frame */
anm->page = find_record(anm, 0);
- if (anm->page < 0) {
- ret = anm->page;
- goto close_and_return;
- }
+ if (anm->page < 0)
+ return anm->page;
anm->record = -1;
return 0;
invalid:
av_log_ask_for_sample(s, NULL);
- ret = AVERROR_INVALIDDATA;
-
-close_and_return:
- av_close_input_stream(s);
- return ret;
+ return AVERROR_INVALIDDATA;
}
static int read_packet(AVFormatContext *s,