Nothing with a real impact, just to silence sanitizers.
From OSS-Fuzz:
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior
/src/libfuzzer/FuzzerSHA1.cpp:132:14 in
sec-tabselect.c:58:24: runtime error: negation of 1 cannot be
represented in type 'mp_limb_t' (aka 'unsigned long')
Patch
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
> index e6bf2282..942c4247 100644
> --- a/sec-tabselect.c
> +++ b/sec-tabselect.c
> @@ -55,7 +55,7 @@
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
> > index e6bf2282..942c4247
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, otherwise my work add
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 add me to cc me on any
On Mittwoch, 8. November 2017 21:29:57 CET Tim Rühsen wrote:
> 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