[FFmpeg-devel] [PATCH 1/2] avcodec/snowdec: Check intra block dc differences.

Michael Niedermayer michael at niedermayer.cc
Thu Nov 16 18:50:32 EET 2017


On Thu, Nov 16, 2017 at 06:26:06AM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Wed, Nov 15, 2017 at 10:15 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com>
> wrote:
> 
> > 2017-11-16 4:06 GMT+01:00 Ronald S. Bultje <rsbultje at gmail.com>:
> >
> > > So, commit it without the error message? I really don't see the issue.
> >
> > As explained, the issue is that without an error message, it
> > is impossible to parse any related bug report.
> 
> 
> We've been OK with that situation so far. Since it only happens for fuzzed
> files, it's OK to continue going like that.

Thats not the case, the snow spec contains no limit in the place where
we need to check. Its a natural and expected limit so likely all files
will be within that but a file outside would still be arguably valid.

So a valid file could potentially be outside this range and the
maintainer (that being me) need to know about this.

Please dont see every change that originated from a fuzzer generated
report as only related to fuzzed files.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171116/9475f936/attachment.sig>


More information about the ffmpeg-devel mailing list