aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2013-06-17 01:04:33 +0200
committerMichael Niedermayer <michaelni@gmx.at>2013-06-17 01:04:33 +0200
commit95275551ef280deadb73d04bad0472486e497115 (patch)
tree908ac496d72c0f95bb456053bf3a3e3ebf2b8b54
parente99f324c143a2d825b2be260cb6a799dbfe97a10 (diff)
parentee16a0ced01e6a33b7b01a0b21a0e07c1e1c7884 (diff)
downloadffmpeg-95275551ef280deadb73d04bad0472486e497115.tar.gz
Merge remote-tracking branch 'qatar/master'
* qatar/master: smacker: check frame size validity Conflicts: libavformat/smacker.c Merged-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavformat/smacker.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/libavformat/smacker.c b/libavformat/smacker.c
index 7132a85565..b4c1bf4335 100644
--- a/libavformat/smacker.c
+++ b/libavformat/smacker.c
@@ -306,12 +306,14 @@ static int smacker_read_packet(AVFormatContext *s, AVPacket *pkt)
/* if audio chunks are present, put them to stack and retrieve later */
for(i = 0; i < 7; i++) {
if(flags & 1) {
- unsigned int size;
+ uint32_t size;
uint8_t *tmpbuf;
size = avio_rl32(s->pb) - 4;
- if(size + 4L > frame_size)
+ if (!size || size + 4L > frame_size) {
+ av_log(s, AV_LOG_ERROR, "Invalid audio part size\n");
return AVERROR_INVALIDDATA;
+ }
frame_size -= size;
frame_size -= 4;
smk->curstream++;