This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Work around GCC 10 warning regression in string/tester.c

string/tester.c contains code that correctly triggers various GCC
warnings about dubious uses of string functions (uses that are being
deliberately tested there), and duly disables those warnings around
the relevant code.

A change in GCC mainline resulted in this code failing to compile with
a -Warray-bounds error, despite the location with the error having
-Warray-bounds already disabled.  This has been reported as
<>.  Since this
doesn't look like being fixed soon, and to keep regression testing
with GCC mainline useful rather than potentially covering up other
issues that might be appearing, this patch adds -Warray-bounds
disabling for a larger region of code that does suffice to avoid the
problem warning.  This is on the basis that this would be removed if
the regression is fixed before GCC 10 is released.

Tested with for aarch64-linux-gnu with GCC

2019-10-01  Joseph Myers  <>

	* string/tester.c (test_strncat) [__GNUC_PREREQ (10, 0)]: Ignore
	-Warray-bounds for more of the function.


Any comments on either the patch itself or the principle of working
around such issues in glibc to allow more effective testing if not
fixed quickly in GCC?

diff --git a/string/tester.c b/string/tester.c
index 24b0dad1cb..9350538996 100644
--- a/string/tester.c
+++ b/string/tester.c
@@ -359,6 +359,14 @@ test_strncat (void)
   /* First test it as strcat, with big counts, then test the count
      mechanism.  */
   it = "strncat";
+#if __GNUC_PREREQ (10, 0)
+  /* A regression in GCC 10
+     <>, present as
+     of 2019-10-01, means this warning needs to be disabled around
+     more code than just the relevant strncat call.  */
+  DIAG_IGNORE_NEEDS_COMMENT (8, "-Warray-bounds");
   (void) strcpy (one, "ijk");
 #if __GNUC_PREREQ (7, 0)
@@ -394,6 +402,10 @@ test_strncat (void)
   DIAG_IGNORE_NEEDS_COMMENT (8, "-Warray-bounds");
   (void) strncat (one, two, 99);
+#if __GNUC_PREREQ (10, 0)
+  /* See the comment above about a regression in GCC 10.  */
   equal (one, "ghef", 5);			/* Basic test encore. */
   equal (two, "ef", 6);			/* Stomped on source? */

Joseph S. Myers

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]