b3097fa24a
Upstream commit: 97bb89668d7171164975f3dc895e38343a2f3a95 - Force DT_RPATH for --enable-hardcoded-path-in-tests - elf: Only process multiple tunable once (BZ 31686) - Add a test to check for duplicate definitions in the static library - i686: Fix multiple definitions of __memmove_chk and __memset_chk - i586: Fix multiple definitions of __memcpy_chk and __mempcpy_chk - time: Allow later version licensing. - nscd: Use time_t for return type of addgetnetgrentX - login: structs utmp, utmpx, lastlog _TIME_BITS independence (bug 30701) - login: Check default sizes of structs utmp, utmpx, lastlog
58 lines
2.6 KiB
Diff
58 lines
2.6 KiB
Diff
commit 8b005d7869debac4d5cd67f65e49a0fad89da9ad
|
|
Author: Gabi Falk <gabifalk@gmx.com>
|
|
Date: Tue Apr 30 20:05:03 2024 +0000
|
|
|
|
i686: Fix multiple definitions of __memmove_chk and __memset_chk
|
|
|
|
Commit c73c96a4a1af1326df7f96eec58209e1e04066d8 updated memcpy.S and
|
|
mempcpy.S, but omitted memmove.S and memset.S. As a result, the static
|
|
library built as PIC, whether with or without multiarch support,
|
|
contains two definitions for each of the __memmove_chk and __memset_chk
|
|
symbols.
|
|
|
|
/usr/lib/gcc/i686-pc-linux-gnu/14/../../../../i686-pc-linux-gnu/bin/ld: /usr/lib/gcc/i686-pc-linux-gnu/14/../../../../lib/libc.a(memset-ia32.o): in function `__memset_chk':
|
|
/var/tmp/portage/sys-libs/glibc-2.39-r3/work/glibc-2.39/string/../sysdeps/i386/i686/memset.S:32: multiple definition of `__memset_chk'; /usr/lib/gcc/i686-pc-linux-gnu/14/../../../../lib/libc.a(memset_chk.o):/var/tmp/portage/sys-libs/glibc-2.39-r3/work/glibc-2.39/debug/../sysdeps/i386/i686/multiarch/memset_chk.c:24: first defined here
|
|
|
|
After this change, regardless of PIC options, the static library, built
|
|
for i686 with multiarch contains implementations of these functions
|
|
respectively from debug/memmove_chk.c and debug/memset_chk.c, and
|
|
without multiarch contains implementations of these functions
|
|
respectively from sysdeps/i386/memmove_chk.S and
|
|
sysdeps/i386/memset_chk.S. This ensures that memmove and memset won't
|
|
pull in __chk_fail and the routines it calls.
|
|
|
|
Reported-by: Sam James <sam@gentoo.org>
|
|
Tested-by: Sam James <sam@gentoo.org>
|
|
Fixes: c73c96a4a1 ("i686: Fix build with --disable-multiarch")
|
|
Signed-off-by: Gabi Falk <gabifalk@gmx.com>
|
|
Reviewed-by: H.J. Lu <hjl.tools@gmail.com>
|
|
Reviewed-by: Dmitry V. Levin <ldv@altlinux.org>
|
|
(cherry picked from commit 5a2cf833f5772d6c37c7adac388dd9af9cc1c4b9)
|
|
|
|
diff --git a/sysdeps/i386/i686/memmove.S b/sysdeps/i386/i686/memmove.S
|
|
index f230359ad62b2443..effd958120082b04 100644
|
|
--- a/sysdeps/i386/i686/memmove.S
|
|
+++ b/sysdeps/i386/i686/memmove.S
|
|
@@ -29,7 +29,7 @@
|
|
#define SRC DEST+4
|
|
#define LEN SRC+4
|
|
|
|
-#if defined PIC && IS_IN (libc)
|
|
+#if defined SHARED && IS_IN (libc)
|
|
ENTRY_CHK (__memmove_chk)
|
|
movl 12(%esp), %eax
|
|
cmpl %eax, 16(%esp)
|
|
diff --git a/sysdeps/i386/i686/memset.S b/sysdeps/i386/i686/memset.S
|
|
index f02f5a6df763d4e9..ab06771ea0ca071f 100644
|
|
--- a/sysdeps/i386/i686/memset.S
|
|
+++ b/sysdeps/i386/i686/memset.S
|
|
@@ -27,7 +27,7 @@
|
|
#define LEN CHR+4
|
|
|
|
.text
|
|
-#if defined PIC && IS_IN (libc)
|
|
+#if defined SHARED && IS_IN (libc)
|
|
ENTRY_CHK (__memset_chk)
|
|
movl 12(%esp), %eax
|
|
cmpl %eax, 16(%esp)
|