[PATCH] libio: use PTR_MANGLE/PTR_DEMANGLE for FILE vtables
Kees Cook
keescook@chromium.org
Mon May 23 20:48:00 GMT 2016
On Mon, May 23, 2016 at 1:34 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> On 23 May 2016 13:02, Kees Cook wrote:
>> --- a/libio/libioP.h
>> +++ b/libio/libioP.h
>> +static inline void
>> +__mangle_vtable(const struct _IO_jump_t **vtable, const struct _IO_jump_t *table)
>
> any reason to not use a name like "_IO_mangle_vtable" ?
>
> GNU style says to put a space before the (
>
>> +{
>> + struct _IO_jump_t *ptr = (struct _IO_jump_t *)table;
>> +#ifndef USE_COMPAT_LIBIO
>> + PTR_MANGLE (ptr);
>> +#endif
>> + *vtable = ptr;
>> +}
>
> GNU style says indent with 2 spaces rather than 4
Should I fix these up and resend, or are those trivial enough to get
fixed up during commit?
> otherwise, lgtm
Yay! :)
-Kees
--
Kees Cook
Chrome OS & Brillo Security
More information about the Libc-alpha
mailing list