[FFmpeg-devel] [PATCH] lavfi/format: fix crash in case no format is specified

Stefano Sabatini stefasab at gmail.com
Mon Dec 9 20:20:48 CET 2013


On date Monday 2013-12-09 19:03:03 +0100, wm4 encoded:
> On Mon,  9 Dec 2013 15:45:56 +0100
> Stefano Sabatini <stefasab at gmail.com> wrote:
> 
> > Also extend logic so that if no format is specified then all formats are
> > accepted. This is probably silly but is consitent with the (undocumented)
> > behavior of aformat.
> > 
> > Fix trac ticket #3210.
> 

> If the format list is explicitly set to empty (like in #3210), it
> definitely should always fail at format negotiation, instead of
> accepting all formats.

Inconsistent with what aformat does.

Since that's not even documented and rather counter-intuitive we
should probably change the behavior, and add a special value "all" to
make format/aformat accept all the formats. This means to introduce an
incompatibility with the fork.
-- 
FFmpeg = Fostering Frightening Martial Power Energized Gangster


More information about the ffmpeg-devel mailing list