[FFmpeg-devel] [libav-devel] [PATCH] nuv: reject negative fps rate

[Andreas Cadhalpun]

On 18.12.2015 15:46, Anton Khirnov wrote:
> Quoting Luca Barbato (2015-12-18 15:36:33)
>> On 18/12/15 15:25, Andreas Cadhalpun wrote:
>>> On 18.12.2015 12:00, Luca Barbato wrote:
>>>> b- mark the framerate as invalid
>>>
>>> How does one do that?
>>>
>>>> and let the user decide what to do,
>>>> e.g. override it with a value that works for them.
>>>> c- a or b depending on explode.
>>>>
>>>> A framerate of 0 is the same as a negative framerate for all the
>>>> intended purposes.
>>>
>>> So should a negative framerate simply be set to 0?
>>>
>>
>> I'd just have an explode mode: either error out or otherwise set it to
>> invalid (0 is fine as invalid) and let the user decide what to do.

OK, patch doing that attached.

> To elaborate, check if AV_EF_EXPLODE is set in
> AVFormatContext.error_recognition

And thanks for the reminder.

Best regards,
Andreas

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-nuv-sanitize-negative-fps-rate.patch
Type: text/x-diff
Size: 1119 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151218/d2928c63/attachment.patch>