This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Avoid undefined behavior in __FD_MASK


> __fd_mask 
> should really be an unsigned type, but changing that is more risky.

That really is the right fix (and then your change is unnecessary).
What risks do you imagine with the type change?

The only thing I can think of is C++ name mangling for function signatures
using the public (but nonstandard) fd_mask type.


Thanks,
Roland


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]