diff options
author | Hugo Beauzée-Luyssen <hugo@beauzee.fr> | 2015-02-01 19:19:45 +0100 |
---|---|---|
committer | Vittorio Giovara <vittorio.giovara@gmail.com> | 2015-02-17 12:16:43 -0500 |
commit | a8765852158ecb2ae34895fa35ff51dc95c186f9 (patch) | |
tree | a2623867071aeac273c774780199b13d94a0d93a /libavformat | |
parent | da7e31a240f276836a0b90ca6c0714181b353cc4 (diff) | |
download | ffmpeg-a8765852158ecb2ae34895fa35ff51dc95c186f9.tar.gz |
movenc: Check memory allocations
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
Diffstat (limited to 'libavformat')
-rw-r--r-- | libavformat/movenc.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 6259a8770f..585b080b5b 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -1167,6 +1167,8 @@ static int mov_write_ctts_tag(AVIOContext *pb, MOVTrack *track) int i; ctts_entries = av_malloc((track->entry + 1) * sizeof(*ctts_entries)); /* worst case */ + if (!ctts_entries) + return AVERROR(ENOMEM); ctts_entries[0].count = 1; ctts_entries[0].duration = track->cluster[0].cts; for (i = 1; i < track->entry; i++) { @@ -1202,6 +1204,8 @@ static int mov_write_stts_tag(AVIOContext *pb, MOVTrack *track) if (track->enc->codec_type == AVMEDIA_TYPE_AUDIO && !track->audio_vbr) { stts_entries = av_malloc(sizeof(*stts_entries)); /* one entry */ + if (!stts_entries) + return AVERROR(ENOMEM); stts_entries[0].count = track->sample_count; stts_entries[0].duration = 1; entries = 1; @@ -1209,6 +1213,8 @@ static int mov_write_stts_tag(AVIOContext *pb, MOVTrack *track) stts_entries = track->entry ? av_malloc(track->entry * sizeof(*stts_entries)) : /* worst case */ NULL; + if (!stts_entries) + return AVERROR(ENOMEM); for (i = 0; i < track->entry; i++) { int duration = get_cluster_duration(track, i); if (i && duration == stts_entries[entries].duration) { @@ -3282,6 +3288,10 @@ int ff_mov_write_packet(AVFormatContext *s, AVPacket *pkt) if (trk->vos_len == 0 && enc->extradata_size > 0) { trk->vos_len = enc->extradata_size; trk->vos_data = av_malloc(trk->vos_len); + if (!trk->vos_data) { + ret = AVERROR(ENOMEM); + goto err; + } memcpy(trk->vos_data, enc->extradata, trk->vos_len); } @@ -3829,6 +3839,8 @@ static int mov_write_header(AVFormatContext *s) else { track->vos_len = st->codec->extradata_size; track->vos_data = av_malloc(track->vos_len); + if (!track->vos_data) + goto error; memcpy(track->vos_data, st->codec->extradata, track->vos_len); } } |