[PATCH v5 05/11] stdio-common: Convert vfprintf and related functions to buffers
Florian Weimer
fweimer@redhat.com
Fri Dec 16 17:58:29 GMT 2022
* Adhemerval Zanella Netto:
> Patch looks good, there is only a small nit on the libio vtable removal
> that in turn showed an issue on elf/tst-relro-symbols.py.
>
> LGTM, thanks.
>
> Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Thanks.
>> -const struct _IO_jump_t _IO_strn_jumps libio_vtable attribute_hidden =
>
> We need to remove both _IO_strn_jumps definition from libio/strfile.h and
> from tst-relro-libc.out.
Yes, fixed.
>> -#ifdef COMPILE_WPRINTF
>> -static const struct _IO_jump_t _IO_helper_jumps libio_vtable =
>> -{
>> -#else
>> -static const struct _IO_jump_t _IO_helper_jumps libio_vtable =
>> -{
> The _IO_helper_jumps should be removed from tst-relro-symbols.py
> required list, and tst-relro-symbols.py not showing a regression
> raised some questions. It turned out that it is not really testing
> the required symbols due a bug, with the fix below:
Right, thanks.
Florian
More information about the Libc-alpha
mailing list