Le 08/08/2021 à 23:28, Eric Covener a écrit :
I finally got unlazy and got a local build w/ the UBISAN checking and
your patch works!

env CC=clang LD=clang CFLAGS=-fsanitize=undefined ./config.nice

Can you commit to trunk?

Now tested on my side as well and pushed as r1892185.

CJ




On Sun, Aug 8, 2021 at 5:03 PM Christophe JAILLET
<christophe.jail...@wanadoo.fr> wrote:

Hi,

attached, my proposal. (compile tested only, but you'll get the idea)
It applies on top of current trunk (so after 1892038,1892063).

I'm not sure that the "(check > APR_INT64_MAX || check < tout)" check in
your patches is enough to catch overflows.

CJ

Reply via email to