powerpc: Fix error message during relocation overflow

Message ID 20171018153824.24225-1-tuliom@linux.vnet.ibm.com
State Committed
Headers

Commit Message

Tulio Magno Quites Machado Filho Oct. 18, 2017, 3:38 p.m. UTC
  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>
---
 sysdeps/powerpc/powerpc64/dl-machine.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
  

Comments

Adhemerval Zanella Oct. 30, 2017, 11:21 a.m. UTC | #1
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)
>      {
>
  
Tulio Magno Quites Machado Filho Jan. 5, 2018, 8:57 p.m. UTC | #2
Adhemerval Zanella <adhemerval.zanella@linaro.org> writes:

> 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>

Pushed as 3a32b7168d9f.
  

Patch

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)
     {