[FFmpeg-devel] [PATCH 9/9] boadec: prevent overflow during block alignment calculation

Clément Bœsch u at pkh.me
Tue Jan 31 15:57:05 EET 2017


On Tue, Jan 31, 2017 at 07:45:56AM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Mon, Jan 30, 2017 at 8:52 PM, Andreas Cadhalpun <
> andreas.cadhalpun at googlemail.com> wrote:
> 
> > Marton proposed a way to mitigate this [1], but you haven't commented
> > on it so far.
> 
> 
> It doesn't mitigate it. Source code is still an issue, as is binary size if
> CONFIG_SMALL is off, which is the default for release builds.
> 
> Even if you change CONFIG_SMALL to NDEBUG, the source code issue is still
> there.
> 
> I don't want this patch. I also don't want to discuss it further. Please
> remove the log message. Thank you.
> 

I stayed silent on the issue so far, but I'm with Ronald on this one and
don't want to pollute functional code with elaborated error code paths
unlikely to happen.

I understand every con and pro from both sides and don't plan to comment
again on the topic.

Regards,

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170131/91c7a53b/attachment.sig>


More information about the ffmpeg-devel mailing list