Bug 17153, PowerPC64le shared libraries with multiple tocs resolve plt to function local entry

Message ID 20140714114344.GS8039@bubble.grove.modra.org
State Committed
Delegated to: Adhemerval Zanella Netto
Headers

Commit Message

Alan Modra July 14, 2014, 11:43 a.m. UTC
  Applying as obvious.  We have four DT_PPC64 values now but left
DT_PPC64_NUM at 3, with effects as described in the bugzilla.

2014-07-14  Alan Modra  <amodra@gmail.com>

	[BZ #17153]
	* elf/elf.h (DT_PPC64_NUM): Correct value.
  

Comments

Adhemerval Zanella Netto July 14, 2014, 12:23 p.m. UTC | #1
On 14-07-2014 08:43, Alan Modra wrote:
> Applying as obvious.  We have four DT_PPC64 values now but left
> DT_PPC64_NUM at 3, with effects as described in the bugzilla.
>
> 2014-07-14  Alan Modra  <amodra@gmail.com>
>
> 	[BZ #17153]
> 	* elf/elf.h (DT_PPC64_NUM): Correct value.
>
> diff --git a/elf/elf.h b/elf/elf.h
> index 40e87b2..78815e8 100644
> --- a/elf/elf.h
> +++ b/elf/elf.h
> @@ -2283,7 +2283,7 @@ typedef Elf32_Addr Elf32_Conflict;
>  #define DT_PPC64_OPD	(DT_LOPROC + 1)
>  #define DT_PPC64_OPDSZ	(DT_LOPROC + 2)
>  #define DT_PPC64_OPT	(DT_LOPROC + 3)
> -#define DT_PPC64_NUM    3
> +#define DT_PPC64_NUM    4
>
>  /* PowerPC64 specific values for the DT_PPC64_OPT Dyn entry.  */
>  #define PPC64_OPT_TLS		1
>
Thanks Alan, how serious is this issue? Should we backport it to previous releases?
  

Patch

diff --git a/elf/elf.h b/elf/elf.h
index 40e87b2..78815e8 100644
--- a/elf/elf.h
+++ b/elf/elf.h
@@ -2283,7 +2283,7 @@  typedef Elf32_Addr Elf32_Conflict;
 #define DT_PPC64_OPD	(DT_LOPROC + 1)
 #define DT_PPC64_OPDSZ	(DT_LOPROC + 2)
 #define DT_PPC64_OPT	(DT_LOPROC + 3)
-#define DT_PPC64_NUM    3
+#define DT_PPC64_NUM    4
 
 /* PowerPC64 specific values for the DT_PPC64_OPT Dyn entry.  */
 #define PPC64_OPT_TLS		1