Commit 6ad42b3e authored by Michael Niedermayer's avatar Michael Niedermayer

avformat/mp3dec: properly allocate dummy AVCodecContext

Fixes (harmless) use of uninitialized variable

Found-by: jamrial
Signed-off-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parent 51b7ab2e
...@@ -61,7 +61,7 @@ static int mp3_read_probe(AVProbeData *p) ...@@ -61,7 +61,7 @@ static int mp3_read_probe(AVProbeData *p)
int fsize, frames; int fsize, frames;
uint32_t header; uint32_t header;
const uint8_t *buf, *buf0, *buf2, *end; const uint8_t *buf, *buf0, *buf2, *end;
AVCodecContext avctx; AVCodecContext *avctx = avcodec_alloc_context3(NULL);
buf0 = p->buf; buf0 = p->buf;
end = p->buf + p->buf_size - sizeof(uint32_t); end = p->buf + p->buf_size - sizeof(uint32_t);
...@@ -79,7 +79,7 @@ static int mp3_read_probe(AVProbeData *p) ...@@ -79,7 +79,7 @@ static int mp3_read_probe(AVProbeData *p)
for(frames = 0; buf2 < end; frames++) { for(frames = 0; buf2 < end; frames++) {
int dummy; int dummy;
header = AV_RB32(buf2); header = AV_RB32(buf2);
fsize = avpriv_mpa_decode_header(&avctx, header, &dummy, &dummy, &dummy, &dummy); fsize = avpriv_mpa_decode_header(avctx, header, &dummy, &dummy, &dummy, &dummy);
if(fsize < 0) if(fsize < 0)
break; break;
buf2 += fsize; buf2 += fsize;
...@@ -88,6 +88,7 @@ static int mp3_read_probe(AVProbeData *p) ...@@ -88,6 +88,7 @@ static int mp3_read_probe(AVProbeData *p)
if(buf == buf0) if(buf == buf0)
first_frames= frames; first_frames= frames;
} }
avcodec_free_context(&avctx);
// keep this in sync with ac3 probe, both need to avoid // keep this in sync with ac3 probe, both need to avoid
// issues with MPEG-files! // issues with MPEG-files!
if (first_frames>=4) return AVPROBE_SCORE_EXTENSION + 1; if (first_frames>=4) return AVPROBE_SCORE_EXTENSION + 1;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment