You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-04 22:03:09 +02:00
avcodec/avcodec: Check sample_rate generically
Check that all audio codecs except decoders with the AV_CODEC_CAP_CHANNEL_CONF flag have a positive sample rate set. Fixes: AVERROR_BUG return Fixes: 413997604/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HCA_fuzzer-5188382613635072 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
This commit is contained in:
@ -255,7 +255,11 @@ int attribute_align_arg avcodec_open2(AVCodecContext *avctx, const AVCodec *code
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (avctx->sample_rate < 0) {
|
/* AV_CODEC_CAP_CHANNEL_CONF is a decoder-only flag; so the code below
|
||||||
|
* in particular checks that sample_rate is set for all audio encoders. */
|
||||||
|
if (avctx->sample_rate < 0 ||
|
||||||
|
avctx->sample_rate == 0 && avctx->codec_type == AVMEDIA_TYPE_AUDIO &&
|
||||||
|
!(codec->capabilities & AV_CODEC_CAP_CHANNEL_CONF)) {
|
||||||
av_log(avctx, AV_LOG_ERROR, "Invalid sample rate: %d\n", avctx->sample_rate);
|
av_log(avctx, AV_LOG_ERROR, "Invalid sample rate: %d\n", avctx->sample_rate);
|
||||||
ret = AVERROR(EINVAL);
|
ret = AVERROR(EINVAL);
|
||||||
goto free_and_end;
|
goto free_and_end;
|
||||||
|
@ -633,11 +633,6 @@ static int encode_preinit_audio(AVCodecContext *avctx)
|
|||||||
avctx->sample_fmt);
|
avctx->sample_fmt);
|
||||||
return AVERROR(EINVAL);
|
return AVERROR(EINVAL);
|
||||||
}
|
}
|
||||||
if (avctx->sample_rate <= 0) {
|
|
||||||
av_log(avctx, AV_LOG_ERROR, "Invalid audio sample rate: %d\n",
|
|
||||||
avctx->sample_rate);
|
|
||||||
return AVERROR(EINVAL);
|
|
||||||
}
|
|
||||||
|
|
||||||
ret = avcodec_get_supported_config(avctx, NULL, AV_CODEC_CONFIG_SAMPLE_FORMAT,
|
ret = avcodec_get_supported_config(avctx, NULL, AV_CODEC_CONFIG_SAMPLE_FORMAT,
|
||||||
0, (const void **) &sample_fmts,
|
0, (const void **) &sample_fmts,
|
||||||
|
Reference in New Issue
Block a user