You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-10 06:10:52 +02:00
libgsmdec: check output buffer size before decoding
This commit is contained in:
@@ -168,18 +168,25 @@ static int libgsm_decode_frame(AVCodecContext *avctx,
|
|||||||
AVPacket *avpkt) {
|
AVPacket *avpkt) {
|
||||||
uint8_t *buf = avpkt->data;
|
uint8_t *buf = avpkt->data;
|
||||||
int buf_size = avpkt->size;
|
int buf_size = avpkt->size;
|
||||||
|
int out_size = avctx->frame_size * av_get_bytes_per_sample(avctx->sample_fmt);
|
||||||
|
|
||||||
|
if (*data_size < out_size) {
|
||||||
|
av_log(avctx, AV_LOG_ERROR, "Output buffer is too small\n");
|
||||||
|
return AVERROR(EINVAL);
|
||||||
|
}
|
||||||
|
|
||||||
*data_size = 0; /* In case of error */
|
*data_size = 0; /* In case of error */
|
||||||
if(buf_size < avctx->block_align) return -1;
|
if(buf_size < avctx->block_align) return -1;
|
||||||
switch(avctx->codec_id) {
|
switch(avctx->codec_id) {
|
||||||
case CODEC_ID_GSM:
|
case CODEC_ID_GSM:
|
||||||
if(gsm_decode(avctx->priv_data,buf,data)) return -1;
|
if(gsm_decode(avctx->priv_data,buf,data)) return -1;
|
||||||
*data_size = GSM_FRAME_SIZE*sizeof(int16_t);
|
|
||||||
break;
|
break;
|
||||||
case CODEC_ID_GSM_MS:
|
case CODEC_ID_GSM_MS:
|
||||||
if(gsm_decode(avctx->priv_data,buf,data) ||
|
if(gsm_decode(avctx->priv_data,buf,data) ||
|
||||||
gsm_decode(avctx->priv_data,buf+33,((int16_t*)data)+GSM_FRAME_SIZE)) return -1;
|
gsm_decode(avctx->priv_data,buf+33,((int16_t*)data)+GSM_FRAME_SIZE)) return -1;
|
||||||
*data_size = GSM_FRAME_SIZE*sizeof(int16_t)*2;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
*data_size = out_size;
|
||||||
return avctx->block_align;
|
return avctx->block_align;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user