Skip to contentSkip to conversation

glibc: Fix __memcpy_chk on non-SSE2 CPUs

PATCH New Andre McCurdy 1 participant 1 message Jun 2, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 glibc: Fix __memcpy_chk on non-SSE2 CPUs

Backport from upstream glibc:

https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=132a1328eccd20621b77f7810eebbeec0a1af187

Note that the fix is only required when glibc is built for i686/multiarch, so is not applicable in the default oe-core x86 configuration (which builds glibc for i586 and therefore does not include SSE2 optimised memcpy etc).

---
 .../glibc/Fix-__memcpy_chk-on-non-SSE2-CPUs.patch  | 36 ++++++++++++++++++++++
 meta/recipes-core/glibc/glibc_2.21.bb              |  1 +
 2 files changed, 37 insertions(+)
 create mode 100644 meta/recipes-core/glibc/glibc/Fix-__memcpy_chk-on-non-SSE2-CPUs.patch
meta/recipes-core/glibc/glibc/Fix-__memcpy_chk-on-non-SSE2-CPUs.patch new file +36 -0
new file mode 100644
index 0000000..2e768bb
@@ -0,0 +1,36 @@
+From 132a1328eccd20621b77f7810eebbeec0a1af187 Mon Sep 17 00:00:00 2001
+From: Evangelos Foutras <[email protected]>
+Date: Tue, 10 Feb 2015 03:22:58 +0000
+Subject: [PATCH] Fix __memcpy_chk on non-SSE2 CPUs
+
+In commit 8b4416d, the 1: jump label in __mempcpy_chk was accidentally
+moved.  This resulted in failures of mempcpy on CPU without SSE2.
+
+Upstream-Status: Backport
+
+  https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=132a1328eccd20621b77f7810eebbeec0a1af187
+
+Signed-off-by: Andre McCurdy <[email protected]>
+
+---
+ sysdeps/i386/i686/multiarch/mempcpy_chk.S | 4 ++--
+ 3 files changed, 9 insertions(+), 3 deletions(-)
+
+diff --git a/sysdeps/i386/i686/multiarch/mempcpy_chk.S b/sysdeps/i386/i686/multiarch/mempcpy_chk.S
+index 207b648..b6fa202 100644
+--- a/sysdeps/i386/i686/multiarch/mempcpy_chk.S
++++ b/sysdeps/i386/i686/multiarch/mempcpy_chk.S
+@@ -36,8 +36,8 @@ ENTRY(__mempcpy_chk)
+ 	cmpl	$0, KIND_OFFSET+__cpu_features@GOTOFF(%ebx)
+ 	jne	1f
+ 	call	__init_cpu_features
+-	leal	__mempcpy_chk_ia32@GOTOFF(%ebx), %eax
+-1:	testl	$bit_SSE2, CPUID_OFFSET+index_SSE2+__cpu_features@GOTOFF(%ebx)
++1:	leal	__mempcpy_chk_ia32@GOTOFF(%ebx), %eax
++	testl	$bit_SSE2, CPUID_OFFSET+index_SSE2+__cpu_features@GOTOFF(%ebx)
+ 	jz	2f
+ 	leal	__mempcpy_chk_sse2_unaligned@GOTOFF(%ebx), %eax
+ 	testl	$bit_Fast_Unaligned_Load, FEATURE_OFFSET+index_Fast_Unaligned_Load+__cpu_features@GOTOFF(%ebx)
+-- 
+1.9.1
+
meta/recipes-core/glibc/glibc_2.21.bb +1 -0
index 03c5461..81b5a2b 100644
@@ -27,6 +27,7 @@ SRC_URI = "git://sourceware.org/git/glibc.git;branch=${BRANCH} \
            file://0001-yes-within-the-path-sets-wrong-config-variables.patch \
            file://elf-Makefile-fix-a-typo.patch \
            file://makesyscall.patch \
+           file://Fix-__memcpy_chk-on-non-SSE2-CPUs.patch \
            ${EGLIBCPATCHES} \
            ${CVEPATCHES} \
           "
Signature · 1 line
1.9.1