diff options
| author | John David Anglin <dave.anglin@bell.net> | 2017-02-20 20:31:57 -0500 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2017-03-15 14:30:48 -0700 |
| commit | 658b0abcff29e817bcb887a6a128aca0226289cd (patch) | |
| tree | d49e05c6ab67e2b75e3e30fe6d7d73a3b0c92c4f | |
| parent | 4d393a8831fc0ab1b7c29e2d5bafbdbfe2102082 (diff) | |
| download | glibc-658b0abcff29e817bcb887a6a128aca0226289cd.tar glibc-658b0abcff29e817bcb887a6a128aca0226289cd.tar.gz glibc-658b0abcff29e817bcb887a6a128aca0226289cd.tar.bz2 glibc-658b0abcff29e817bcb887a6a128aca0226289cd.zip | |
hppa: Fix setting of __libc_stack_end
The binutils package was recently changed to fix -z relro support on hppa.
See ld/21000 for details:
https://sourceware.org/bugzilla/show_bug.cgi?id=21000
This exposed a problem with the _dl_start_user function in the RTLD_START
define. We need to set __libc_stack_end before it is made read only. For
this, we need to define DL_STACK_END. The offset of 0x160 gives the same
stack end as the code in _dl_start_user.
A build log with the attached patch is here:
https://buildd.debian.org/status/fetch.php?pkg=glibc&arch=hppa&ver=2.24-9&stamp=1487639205&raw=0
(cherry picked from commit 5d20a49aaccef5ef7adac93d5ca159f6b7ba0105)
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | sysdeps/hppa/dl-machine.h | 9 |
2 files changed, 9 insertions, 5 deletions
@@ -1,3 +1,8 @@ +2017-03-15 John David Anglin <danglin@gcc.gnu.org> + + * sysdeps/hppa/dl-machine.h (DL_STACK_END): Define. + (RTLD_START): Don't record stack end address in _dl_start_user. + 2017-01-30 H.J. Lu <hongjiu.lu@intel.com> [BZ #21081] diff --git a/sysdeps/hppa/dl-machine.h b/sysdeps/hppa/dl-machine.h index 9404211819..01bd5bf197 100644 --- a/sysdeps/hppa/dl-machine.h +++ b/sysdeps/hppa/dl-machine.h @@ -302,6 +302,10 @@ elf_machine_runtime_setup (struct link_map *l, int lazy, int profile) #define ARCH_LA_PLTENTER hppa_gnu_pltenter #define ARCH_LA_PLTEXIT hppa_gnu_pltexit +/* Adjust DL_STACK_END to get value we want in __libc_stack_end. */ +#define DL_STACK_END(cookie) \ + ((void *) (((long) (cookie)) + 0x160)) + /* Initial entry point code for the dynamic linker. The C function `_dl_start' is the real entry point; its return value is the user program's entry point. */ @@ -401,11 +405,6 @@ asm ( \ /* Save the entry point in %r3. */ \ " copy %ret0,%r3\n" \ \ - /* Remember the lowest stack address. */ \ -" addil LT'__libc_stack_end,%r19\n" \ -" ldw RT'__libc_stack_end(%r1),%r20\n" \ -" stw %sp,0(%r20)\n" \ - \ /* See if we were called as a command with the executable file \ name as an extra leading argument. */ \ " addil LT'_dl_skip_args,%r19\n" \ |