You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-15 14:13:16 +02:00
electronicarts: Check packet sizes before reading
Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind CC: libav-stable@libav.org Signed-off-by: Martin Storsjö <martin@martin.st>
This commit is contained in:
@@ -555,10 +555,16 @@ static int ea_read_packet(AVFormatContext *s, AVPacket *pkt)
|
|||||||
case AV_CODEC_ID_ADPCM_EA_R1:
|
case AV_CODEC_ID_ADPCM_EA_R1:
|
||||||
case AV_CODEC_ID_ADPCM_EA_R2:
|
case AV_CODEC_ID_ADPCM_EA_R2:
|
||||||
case AV_CODEC_ID_ADPCM_IMA_EA_EACS:
|
case AV_CODEC_ID_ADPCM_IMA_EA_EACS:
|
||||||
pkt->duration = AV_RL32(pkt->data);
|
|
||||||
break;
|
|
||||||
case AV_CODEC_ID_ADPCM_EA_R3:
|
case AV_CODEC_ID_ADPCM_EA_R3:
|
||||||
pkt->duration = AV_RB32(pkt->data);
|
if (pkt->size < 4) {
|
||||||
|
av_log(s, AV_LOG_ERROR, "Packet is too short\n");
|
||||||
|
av_free_packet(pkt);
|
||||||
|
return AVERROR_INVALIDDATA;
|
||||||
|
}
|
||||||
|
if (ea->audio_codec == AV_CODEC_ID_ADPCM_EA_R3)
|
||||||
|
pkt->duration = AV_RB32(pkt->data);
|
||||||
|
else
|
||||||
|
pkt->duration = AV_RL32(pkt->data);
|
||||||
break;
|
break;
|
||||||
case AV_CODEC_ID_ADPCM_IMA_EA_SEAD:
|
case AV_CODEC_ID_ADPCM_IMA_EA_SEAD:
|
||||||
pkt->duration = ret * 2 / ea->num_channels;
|
pkt->duration = ret * 2 / ea->num_channels;
|
||||||
|
Reference in New Issue
Block a user