Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 28 Apr 2015 15:55:06 +0200
From: Szabolcs Nagy <>
Subject: Re: Pending patches & other changes

* Rich Felker <> [2015-04-28 09:46:47 -0400]:
> On Tue, Apr 28, 2015 at 11:43:21AM +0200, Szabolcs Nagy wrote:
> > i forgot to do this one in elf.h:
> > 
> > -#define SHF_EXCLUDE         (1 << 31)
> > +#define SHF_EXCLUDE         (1U << 31)
> > 
> > (there is no other invalid <<31)
> This was discussed quite a bit on libc-alpha (glibc list) and it seems
> the consensus there is that changing the type to unsigned is
> potentially dangerous and it would be better to use (-0x7fffffff-1) or
> perhaps ((1<<30) * -2) if it's more clear. What do you think?

the glibc bikeshed ended with florian saying

"I thought some more about this, and have changed my opinion completely.
 Making the constant unsigned is less risky than making it negative
 because of potential sign extension issues."

and they committed the original 1U<<31 patch today

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.