diff options
| author | Michael Niedermayer <michaelni@gmx.at> | 2011-08-27 21:24:13 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2011-08-27 21:39:52 +0200 |
| commit | 941bb552c6e08b40eb7d7842df19285cd650edd0 (patch) | |
| tree | a75f1e7e033974c17221b5446084630f55a3dee0 /libavformat/utils.c | |
| parent | 24ddfb2bdb8a02971c3932901bc87f233b23a12f (diff) | |
| download | ffmpeg-941bb552c6e08b40eb7d7842df19285cd650edd0.tar.gz | |
Fix memory corruption in case of memory allocation failure in av_probe_input_buffer()
Reported-by: Tanami Ohad
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavformat/utils.c')
| -rw-r--r-- | libavformat/utils.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/libavformat/utils.c b/libavformat/utils.c index 7a9d69a63b..a60fcd9706 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -527,13 +527,19 @@ int av_probe_input_buffer(AVIOContext *pb, AVInputFormat **fmt, probe_size = FFMIN(probe_size<<1, FFMAX(max_probe_size, probe_size+1))) { int score = probe_size < max_probe_size ? AVPROBE_SCORE_MAX/4 : 0; int buf_offset = (probe_size == PROBE_BUF_MIN) ? 0 : probe_size>>1; + void *buftmp; if (probe_size < offset) { continue; } /* read probe data */ - buf = av_realloc(buf, probe_size + AVPROBE_PADDING_SIZE); + buftmp = av_realloc(buf, probe_size + AVPROBE_PADDING_SIZE); + if(!buftmp){ + av_free(buf); + return AVERROR(ENOMEM); + } + buf=buftmp; if ((ret = avio_read(pb, buf + buf_offset, probe_size - buf_offset)) < 0) { /* fail if error was not end of file, otherwise, lower score */ if (ret != AVERROR_EOF) { |