Commit eb205eda authored by Andreas Cadhalpun's avatar Andreas Cadhalpun

oggparsespeex: validate sample_rate

A negative sample rate doesn't make sense and triggers assertions in
av_rescale_rnd.
Signed-off-by: 's avatarAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
parent f748e3b5
...@@ -68,6 +68,10 @@ static int speex_header(AVFormatContext *s, int idx) { ...@@ -68,6 +68,10 @@ static int speex_header(AVFormatContext *s, int idx) {
} }
st->codecpar->sample_rate = AV_RL32(p + 36); st->codecpar->sample_rate = AV_RL32(p + 36);
if (st->codecpar->sample_rate <= 0) {
av_log(s, AV_LOG_ERROR, "Invalid sample rate %d\n", st->codecpar->sample_rate);
return AVERROR_INVALIDDATA;
}
st->codecpar->channels = AV_RL32(p + 48); st->codecpar->channels = AV_RL32(p + 48);
if (st->codecpar->channels < 1 || st->codecpar->channels > 2) { if (st->codecpar->channels < 1 || st->codecpar->channels > 2) {
av_log(s, AV_LOG_ERROR, "invalid channel count. Speex must be mono or stereo.\n"); av_log(s, AV_LOG_ERROR, "invalid channel count. Speex must be mono or stereo.\n");
......
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