[FFmpeg-devel] [PATCH]lavf/riff: Do not use a rogue twocc for adpcm_swf

Ronald S. Bultje rsbultje at gmail.com
Wed Sep 14 19:05:45 EEST 2016


Hi,

On Wed, Sep 14, 2016 at 12:01 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com>
wrote:

> 2016-09-12 12:52 GMT+02:00 Carl Eugen Hoyos <cehoyos at ag.or.at>:
>
> > Attached patch "fixes" ticket #5829, I am happy if a better
> > solution can be found.
>
> Ping.


What makes you believe this is not the right solution? As in: why was the
rogue twocc ever added?

IMO patch is OK, and other rogue twoccs should be removed also, if they
allow creating invalid and unplayable files (like vorbis in wav).

Ronald


More information about the ffmpeg-devel mailing list