summary refs log tree commit diff
diff options
context:
space:
mode:
authorH.J. Lu <hjl.tools@gmail.com>2017-08-04 10:56:40 -0700
committerH.J. Lu <hjl.tools@gmail.com>2017-08-04 10:56:51 -0700
commitc8a0e6ec03b2646842fe3c7c59955f55175f3669 (patch)
treeaa61a56df0011fd21a06a2893f23b7fc9a484900
parent2b34e2716f1e84b2c3457ffc868c3dc775b55845 (diff)
downloadglibc-c8a0e6ec03b2646842fe3c7c59955f55175f3669.tar.gz
glibc-c8a0e6ec03b2646842fe3c7c59955f55175f3669.tar.xz
glibc-c8a0e6ec03b2646842fe3c7c59955f55175f3669.zip
x86: Remove __memset_zero_constant_len_parameter [BZ #21790]
__memset_zero_constant_len_parameter should be removed by

commit 61062f56304750c367c5c1533351621353c112a7
Author: Ulrich Drepper <drepper@redhat.com>
Date:   Tue Mar 1 00:35:23 2005 +0000

    2005-02-24  Roland McGrath  <roland@redhat.com>

            * debug/Versions (libc: GLIBC_2.4): Remove
            __memset_zero_constant_len_parameter.
            * sysdeps/generic/memset_chk.c: Remove alias and warning.
            * misc/sys/cdefs.h (__warndecl): New macro.
            * debug/warning-nop.c: New file.
            * string/bits/string3.h (memset): Call __warn_memset_zero_len with no
            arguments, instead of calling __memset_zero_constant_len_parameter.
            Use __warndecl for __warn_memset_zero_len.
            * debug/Makefile (routines): Add $(static-only-routines).
            (static-only-routines): New variable.

This patch removes the last emaining pieces of it.  Tested it on i586,
i686 and x86-64.

	[BZ #21790]
	* sysdeps/i386/i586/memset.S
	(__memset_zero_constant_len_parameter): Removed.
	* sysdeps/i386/i686/memset.S
	(__memset_zero_constant_len_parameter): Likewise.
	* sysdeps/i386/i686/multiarch/memset_chk.S
	(__memset_zero_constant_len_parameter): Likewise.
	* sysdeps/x86_64/memset.S (__memset_zero_constant_len_parameter):
	Likewise.
-rw-r--r--ChangeLog12
-rw-r--r--sysdeps/i386/i586/memset.S7
-rw-r--r--sysdeps/i386/i686/memset.S7
-rw-r--r--sysdeps/i386/i686/multiarch/memset_chk.S6
-rw-r--r--sysdeps/x86_64/memset.S6
5 files changed, 13 insertions, 25 deletions
diff --git a/ChangeLog b/ChangeLog
index 1d336afb25..7683341150 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2017-08-04  H.J. Lu  <hongjiu.lu@intel.com>
+
+	[BZ #21790]
+	* sysdeps/i386/i586/memset.S
+	(__memset_zero_constant_len_parameter): Removed.
+	* sysdeps/i386/i686/memset.S
+	(__memset_zero_constant_len_parameter): Likewise.
+	* sysdeps/i386/i686/multiarch/memset_chk.S
+	(__memset_zero_constant_len_parameter): Likewise.
+	* sysdeps/x86_64/memset.S (__memset_zero_constant_len_parameter):
+	Likewise.
+
 2017-08-03  Aurelien Jarno  <aurelien@aurel32.net>
 
 	* stdlib/getentropy.c (getentropy): Change return type to int.
diff --git a/sysdeps/i386/i586/memset.S b/sysdeps/i386/i586/memset.S
index 4f8f1bcf94..a55e8c68cf 100644
--- a/sysdeps/i386/i586/memset.S
+++ b/sysdeps/i386/i586/memset.S
@@ -112,10 +112,3 @@ L(2):	shrl	$2, %ecx	/* convert byte count to longword count */
 	ret
 END (memset)
 libc_hidden_builtin_def (memset)
-
-#if defined SHARED && IS_IN (libc) && !defined __memset_chk \
-    && !defined USE_AS_BZERO
-strong_alias (__memset_chk, __memset_zero_constant_len_parameter)
-	.section .gnu.warning.__memset_zero_constant_len_parameter
-	.string "memset used with constant zero length parameter; this could be due to transposed parameters"
-#endif
diff --git a/sysdeps/i386/i686/memset.S b/sysdeps/i386/i686/memset.S
index 24d06178d2..a29b87ba09 100644
--- a/sysdeps/i386/i686/memset.S
+++ b/sysdeps/i386/i686/memset.S
@@ -91,10 +91,3 @@ ENTRY (memset)
 	ret
 END (memset)
 libc_hidden_builtin_def (memset)
-
-#if defined SHARED && IS_IN (libc) && !defined __memset_chk \
-    && !defined USE_AS_BZERO
-strong_alias (__memset_chk, __memset_zero_constant_len_parameter)
-	.section .gnu.warning.__memset_zero_constant_len_parameter
-	.string "memset used with constant zero length parameter; this could be due to transposed parameters"
-#endif
diff --git a/sysdeps/i386/i686/multiarch/memset_chk.S b/sysdeps/i386/i686/multiarch/memset_chk.S
index 573cf4208a..0580991ce4 100644
--- a/sysdeps/i386/i686/multiarch/memset_chk.S
+++ b/sysdeps/i386/i686/multiarch/memset_chk.S
@@ -37,11 +37,7 @@ ENTRY(__memset_chk)
 2:	ret
 END(__memset_chk)
 
-# ifdef SHARED
-strong_alias (__memset_chk, __memset_zero_constant_len_parameter)
-	.section .gnu.warning.__memset_zero_constant_len_parameter
-	.string "memset used with constant zero length parameter; this could be due to transposed parameters"
-# else
+# ifndef SHARED
 	.text
 	.type __memset_chk_sse2, @function
 	.p2align 4;
diff --git a/sysdeps/x86_64/memset.S b/sysdeps/x86_64/memset.S
index 41278787fe..f1245aec60 100644
--- a/sysdeps/x86_64/memset.S
+++ b/sysdeps/x86_64/memset.S
@@ -59,9 +59,3 @@ libc_hidden_def (__wmemset)
 weak_alias (__wmemset, wmemset)
 libc_hidden_weak (wmemset)
 #endif
-
-#if defined SHARED && IS_IN (libc) && !defined USE_MULTIARCH
-strong_alias (__memset_chk, __memset_zero_constant_len_parameter)
-	.section .gnu.warning.__memset_zero_constant_len_parameter
-	.string "memset used with constant zero length parameter; this could be due to transposed parameters"
-#endif