diff options
| author | H.J. Lu <hjl.tools@gmail.com> | 2015-03-06 04:55:56 -0800 |
|---|---|---|
| committer | H.J. Lu <hjl.tools@gmail.com> | 2015-08-01 15:42:23 -0700 |
| commit | 2702b161faaf048300de5c67dad5d1ca2078db60 (patch) | |
| tree | 864010c6eec17bfdb033123c35e8d02ebef908d3 /elf | |
| parent | d8700f7fa9a60def973ccdfad24266d91d9ce667 (diff) | |
| download | glibc-2702b161faaf048300de5c67dad5d1ca2078db60.tar.gz glibc-2702b161faaf048300de5c67dad5d1ca2078db60.tar.xz glibc-2702b161faaf048300de5c67dad5d1ca2078db60.zip | |
Replace __attribute__((visibility("protected"))) hjl/pr17711/2.21
With copy relocation, address of protected data defined in the shared
library may be external. Compiler shouldn't asssume protected data will
be local. But due to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65248
__attribute__((visibility("protected"))) doesn't work correctly, we need
to use asm (".protected xxx") instead.
Cherry-pick from master: e0ed2fb40a0e29c43cf60addc74741dab15f2e05
* elf/ifuncdep2.c (global): Replace
__attribute__((visibility("protected"))) with
asm (".protected global").
* elf/ifuncmod1.c (global): Likewise.
* elf/ifuncmod5.c (global): Likewise.
Diffstat (limited to 'elf')
| -rw-r--r-- | elf/ifuncdep2.c | 8 | ||||
| -rw-r--r-- | elf/ifuncmod1.c | 8 | ||||
| -rw-r--r-- | elf/ifuncmod5.c | 8 |
3 files changed, 21 insertions, 3 deletions
diff --git a/elf/ifuncdep2.c b/elf/ifuncdep2.c index 99d19263ae..6e66d318a6 100644 --- a/elf/ifuncdep2.c +++ b/elf/ifuncdep2.c @@ -2,7 +2,13 @@ #include "ifunc-sel.h" -int global __attribute__ ((visibility ("protected"))) = -1; +int global = -1; +/* Can't use __attribute__((visibility("protected"))) until the GCC bug: + + https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65248 + + is fixed. */ +asm (".protected global"); static int one (void) diff --git a/elf/ifuncmod1.c b/elf/ifuncmod1.c index 2b8195ce55..0b6138056d 100644 --- a/elf/ifuncmod1.c +++ b/elf/ifuncmod1.c @@ -6,7 +6,13 @@ */ #include "ifunc-sel.h" -int global __attribute__ ((visibility ("protected"))) = -1; +int global = -1; +/* Can't use __attribute__((visibility("protected"))) until the GCC bug: + + https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65248 + + is fixed. */ +asm (".protected global"); static int one (void) diff --git a/elf/ifuncmod5.c b/elf/ifuncmod5.c index 9a08e8cf53..0e65a63691 100644 --- a/elf/ifuncmod5.c +++ b/elf/ifuncmod5.c @@ -1,7 +1,13 @@ /* Test STT_GNU_IFUNC symbols without direct function call. */ #include "ifunc-sel.h" -int global __attribute__ ((visibility ("protected"))) = -1; +int global = -1; +/* Can't use __attribute__((visibility("protected"))) until the GCC bug: + + https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65248 + + is fixed. */ +asm (".protected global"); static int one (void) |