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] powerpc: Fix error message during relocation overflow



On 18/10/2017 13:38, Tulio Magno Quites Machado Filho wrote:
> The function _itoa_word() writes characters from the higher address to
> the lower address, requiring the destination string to reserve that size
> before calling it.
> 
> 2017-10-18  Tulio Magno Quites Machado Filho  <tuliom@linux.vnet.ibm.com>
> 
> 	* sysdeps/powerpc/powerpc64/dl-machine.c (_dl_reloc_overflow):
> 	Reserve 16 chars to reloc_addr before calling _itoa_word.
> 
> Signed-off-by: Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>

> ---
>  sysdeps/powerpc/powerpc64/dl-machine.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/sysdeps/powerpc/powerpc64/dl-machine.c b/sysdeps/powerpc/powerpc64/dl-machine.c
> index 2cfd43b..4c1b556 100644
> --- a/sysdeps/powerpc/powerpc64/dl-machine.c
> +++ b/sysdeps/powerpc/powerpc64/dl-machine.c
> @@ -31,7 +31,10 @@ _dl_reloc_overflow (struct link_map *map,
>    char buffer[1024];
>    char *t;
>    t = stpcpy (buffer, name);
> -  t = stpcpy (t, " reloc at 0x");
> +  /* Notice that _itoa_word() writes characters from the higher address to the
> +     lower address, requiring the destination string to reserve all the
> +     required size before the call.  */
> +  t = stpcpy (t, " reloc at 0x0000000000000000");
>    _itoa_word ((unsigned long) reloc_addr, t, 16, 0);
>    if (refsym)
>      {
> 


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