mirror of
https://git.ffmpeg.org/ffmpeg.git
synced 2024-12-24 16:22:37 +00:00
fftools/ffmpeg_demux: honor -ch_layout options for overriding input stream channel layout
The code only set the channel layout of the AVFormatContext, so the user could
not override the channel layout if the demuxer did not have such parameter.
This used to work via the respective AVCodecContext option, but since
639c2f0049
it no longer gets passed to the
decoders. It is actually better if we set it manually, instead of using the
codec option because that way we can also override it on the stream level, so
it will also work for stream copy or bitstream filtering.
We don't allow changing the number of channels, because that can cause
unexpected results. We disable layout guessing, if a channel layout is
specified.
Fixes ticket #11016.
Signed-off-by: Marton Balint <cus@passwd.hu>
This commit is contained in:
parent
44f7f6e010
commit
4bfecde9b9
@ -1386,9 +1386,30 @@ static int ist_add(const OptionsContext *o, Demuxer *d, AVStream *st)
|
|||||||
|
|
||||||
break;
|
break;
|
||||||
case AVMEDIA_TYPE_AUDIO: {
|
case AVMEDIA_TYPE_AUDIO: {
|
||||||
int guess_layout_max = INT_MAX;
|
char *ch_layout_str = NULL;
|
||||||
MATCH_PER_STREAM_OPT(guess_layout_max, i, guess_layout_max, ic, st);
|
MATCH_PER_STREAM_OPT(audio_ch_layouts, str, ch_layout_str, ic, st);
|
||||||
guess_input_channel_layout(ist, par, guess_layout_max);
|
if (ch_layout_str) {
|
||||||
|
AVChannelLayout ch_layout;
|
||||||
|
ret = av_channel_layout_from_string(&ch_layout, ch_layout_str);
|
||||||
|
if (ret < 0) {
|
||||||
|
av_log(ist, AV_LOG_ERROR, "Error parsing channel layout %s.\n", ch_layout_str);
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
if (par->ch_layout.nb_channels <= 0 || par->ch_layout.nb_channels == ch_layout.nb_channels) {
|
||||||
|
av_channel_layout_uninit(&par->ch_layout);
|
||||||
|
par->ch_layout = ch_layout;
|
||||||
|
} else {
|
||||||
|
av_log(ist, AV_LOG_ERROR,
|
||||||
|
"Specified channel layout '%s' has %d channels, but input has %d channels.\n",
|
||||||
|
ch_layout_str, ch_layout.nb_channels, par->ch_layout.nb_channels);
|
||||||
|
av_channel_layout_uninit(&ch_layout);
|
||||||
|
return AVERROR(EINVAL);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
int guess_layout_max = INT_MAX;
|
||||||
|
MATCH_PER_STREAM_OPT(guess_layout_max, i, guess_layout_max, ic, st);
|
||||||
|
guess_input_channel_layout(ist, par, guess_layout_max);
|
||||||
|
}
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
case AVMEDIA_TYPE_DATA:
|
case AVMEDIA_TYPE_DATA:
|
||||||
|
Loading…
Reference in New Issue
Block a user