* [PATCH] x86_64: fix static build of __memcpy_chk for compilers defaulting to PIC/PIE
@ 2016-11-24 11:24 Aurelien Jarno
2016-11-24 11:37 ` Florian Weimer
0 siblings, 1 reply; 2+ messages in thread
From: Aurelien Jarno @ 2016-11-24 11:24 UTC (permalink / raw)
To: libc-alpha; +Cc: Aurelien Jarno
When glibc is compiled with gcc 6.2 that has been configured with
to default to PIC/PIE, the static version of __memcpy_chk is not built,
as the test is done on PIC instead of SHARED. Fix the test to check for
SHARED, like it is done for similar functions like memmove_chk.
Changelog
* sysdeps/x86_64/memcpy_chk.S (__memcpy_chk): Check SHARED
instead of PIC.
---
ChangeLog | 5 +++++
sysdeps/x86_64/memcpy_chk.S | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index b6fc831..db1fb2a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2016-11-24 Aurelien Jarno <aurelien@aurel32.net>
+
+ * sysdeps/x86_64/memcpy_chk.S (__memcpy_chk): Check SHARED
+ instead of PIC.
+
2016-11-23 Joseph Myers <joseph@codesourcery.com>
[BZ #20859]
diff --git a/sysdeps/x86_64/memcpy_chk.S b/sysdeps/x86_64/memcpy_chk.S
index 2296b55..a95b3ad 100644
--- a/sysdeps/x86_64/memcpy_chk.S
+++ b/sysdeps/x86_64/memcpy_chk.S
@@ -19,7 +19,7 @@
#include <sysdep.h>
#include "asm-syntax.h"
-#ifndef PIC
+#ifndef SHARED
/* For libc.so this is defined in memcpy.S.
For libc.a, this is a separate source to avoid
memcpy bringing in __chk_fail and all routines
--
2.10.2
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] x86_64: fix static build of __memcpy_chk for compilers defaulting to PIC/PIE
2016-11-24 11:24 [PATCH] x86_64: fix static build of __memcpy_chk for compilers defaulting to PIC/PIE Aurelien Jarno
@ 2016-11-24 11:37 ` Florian Weimer
0 siblings, 0 replies; 2+ messages in thread
From: Florian Weimer @ 2016-11-24 11:37 UTC (permalink / raw)
To: Aurelien Jarno, libc-alpha
On 11/24/2016 12:24 PM, Aurelien Jarno wrote:
> When glibc is compiled with gcc 6.2 that has been configured with
> to default to PIC/PIE, the static version of __memcpy_chk is not built,
> as the test is done on PIC instead of SHARED. Fix the test to check for
> SHARED, like it is done for similar functions like memmove_chk.
>
> Changelog
> * sysdeps/x86_64/memcpy_chk.S (__memcpy_chk): Check SHARED
> instead of PIC.
Okay.
Thanks,
Florian
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-11-24 11:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-24 11:24 [PATCH] x86_64: fix static build of __memcpy_chk for compilers defaulting to PIC/PIE Aurelien Jarno
2016-11-24 11:37 ` Florian Weimer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).