Hi, On Thu, Nov 16, 2017 at 11:50 AM, Michael Niedermayer < mich...@niedermayer.cc> wrote:
> 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 <ceffm...@gmail.com> > > wrote: > > > > > 2017-11-16 4:06 GMT+01:00 Ronald S. Bultje <rsbul...@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. We are re-hashing old arguments here. I'm not really interested in that. My review comment is and remains: please remove the log msg. Otherwise, the patch is perfectly fine. Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel