Re: Fix UBSAN issue

2017-11-08 Thread Niels Möller
Tim Rühsen writes: > And of course you might be right and this is a bug in the sanitizer. > I leave that discussion to the experts and open a bug at oss-fuzz as soon as > I > find the time. Ok. Please add me to cc me on any related bugs, my personal address if that works,

Re: Release plan?

2017-11-08 Thread Niels Möller
Nikos Mavrogiannopoulos writes: > I suggest including the attached patches to the CI. The first makes > sure that random errors due to installation of fedora packages don't > get in the way by using the pre-build images used in gnutls. The second > it includes gnutls build in

Re: Fix UBSAN issue

2017-11-08 Thread Tim Rühsen
On Mittwoch, 8. November 2017 12:44:28 CET Niels Möller wrote: > Tim Rühsen writes: > > And of course you might be right and this is a bug in the sanitizer. > > I leave that discussion to the experts and open a bug at oss-fuzz as soon > > as I find the time. > > Ok. Please

Re: Fix UBSAN issue

2017-11-08 Thread Tim Rühsen
On Sonntag, 5. November 2017 23:20:27 CET Niels Möller wrote: > Tim Rühsen writes: > > Nothing with a real impact, just to silence sanitizers. > > Can you explain precisely what's undefined behavior in this code ? > > > diff --git a/sec-tabselect.c b/sec-tabselect.c > >

Re: API fixes

2017-11-08 Thread Niels Möller
ni...@lysator.liu.se (Niels Möller) writes: > ni...@lysator.liu.se (Niels Möller) writes: > >> Applied. I'd expect the gnutls build to fail at the moment, problem not >> yet fixed. > > And it did. Tentative fix now pushed to the branch api-opaque-fix. And it seems to work, see