[PATCH] string: Add tests for unique strerror error strings

Arjun Shankar arjun@redhat.com
Wed Sep 10 23:29:34 GMT 2025


strerror and its variants should return unique strings for each
known/unknown error.  Add tests to verify this for strerror, strerror_r
and strerror_l (for the C locale).

strerror_r is special in that the GNU variant returns "Unknown error"
strings (and only those) in the user supplied buffer, returning
read-only internal strings for known errors.  Additionally test for this
as well.
---
 string/Makefile                 |  3 ++
 string/tst-strerror-strings.c   | 76 +++++++++++++++++++++++++++++++++
 string/tst-strerror_l-strings.c | 38 +++++++++++++++++
 string/tst-strerror_r-strings.c | 48 +++++++++++++++++++++
 4 files changed, 165 insertions(+)
 create mode 100644 string/tst-strerror-strings.c
 create mode 100644 string/tst-strerror_l-strings.c
 create mode 100644 string/tst-strerror_r-strings.c

diff --git a/string/Makefile b/string/Makefile
index c83b195e2e..37719d01a3 100644
--- a/string/Makefile
+++ b/string/Makefile
@@ -197,6 +197,9 @@ tests := \
   tst-inlcall \
   tst-memmove-overflow \
   tst-strerror-fail \
+  tst-strerror-strings \
+  tst-strerror_l-strings \
+  tst-strerror_r-strings \
   tst-strfry \
   tst-strlcat \
   tst-strlcat2 \
diff --git a/string/tst-strerror-strings.c b/string/tst-strerror-strings.c
new file mode 100644
index 0000000000..4b53d9d821
--- /dev/null
+++ b/string/tst-strerror-strings.c
@@ -0,0 +1,76 @@
+/* Test that strerror variants return unique strings for each errnum.
+
+   Copyright (C) 2025 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <https://www.gnu.org/licenses/>.  */
+
+#include <string.h>
+#include <stdlib.h>
+
+#include <support/support.h>
+#include <support/check.h>
+
+static int
+compare_strings (const void *a, const void *b)
+{
+  const char *stra = * (const char **) a;
+  const char *strb = * (const char **) b;
+
+  int ret = strcmp (stra, strb);
+
+  if (!ret)
+    FAIL_EXIT1 ("Found duplicate error strings: \"%s\"\n", stra);
+
+  return ret;
+}
+
+#define NSTRINGS 2049
+
+static int
+do_test (void)
+{
+  char *string[NSTRINGS];
+  /* Convenient indexing for error strings from -1024 to 1024.  */
+  char **err_str = string + 1024;
+
+  unsetenv ("LANGUAGE");
+
+  xsetlocale (LC_ALL, "C");
+
+  for (int i = -1024; i <= 1024; i++)
+    {
+
+#ifdef TEST_STRERROR_VARIANT
+      /* Used for testing strerror_r and strerror_l.  */
+      err_str[i] = TEST_STRERROR_VARIANT (i);
+#else
+      err_str[i] = xstrdup (strerror (i));
+#endif
+
+      /* Negative as well as large positive errnums are unused.  160 allows
+         us to define more errors without needing to update this test.  */
+      int is_unknown_error
+        = (strstr (err_str[i], "Unknown error ") == err_str[i]);
+      TEST_VERIFY_EXIT ((i >= 0 && i < 160) || is_unknown_error);
+    }
+
+  /* We check for and fail on duplicate strings in the comparator.  */
+  qsort (string, NSTRINGS, sizeof (char *), compare_strings);
+
+  return 0;
+}
+
+#include <support/test-driver.c>
diff --git a/string/tst-strerror_l-strings.c b/string/tst-strerror_l-strings.c
new file mode 100644
index 0000000000..186a4b09c7
--- /dev/null
+++ b/string/tst-strerror_l-strings.c
@@ -0,0 +1,38 @@
+/* Test that strerror_l returns unique strings for each errnum.
+
+   Copyright (C) 2025 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <https://www.gnu.org/licenses/>.  */
+
+#include <stdlib.h>
+#include <string.h>
+#include <support/support.h>
+#include <support/check.h>
+
+static locale_t loc = (locale_t) 0;
+
+/* Wrap strerror_l to be plugged into the equivalent strerror test.  */
+static char *
+wrap_strerror_l (int errnum)
+{
+  if (loc == (locale_t) 0)
+    loc = xnewlocale (LC_ALL_MASK, "C", (locale_t) 0);
+
+  return xstrdup (strerror_l (errnum, loc));
+}
+
+#define TEST_STRERROR_VARIANT wrap_strerror_l
+#include "tst-strerror-strings.c"
diff --git a/string/tst-strerror_r-strings.c b/string/tst-strerror_r-strings.c
new file mode 100644
index 0000000000..feaccfbefb
--- /dev/null
+++ b/string/tst-strerror_r-strings.c
@@ -0,0 +1,48 @@
+/* Test that GNU strerror_r returns unique strings for each errnum.
+
+   Copyright (C) 2025 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <https://www.gnu.org/licenses/>.  */
+
+#include <stdlib.h>
+#include <string.h>
+#include <support/support.h>
+#include <support/check.h>
+
+/* Wrap strerror_r into a checked variant that can be plugged into the
+   equivalent strerror test.  */
+static char *
+test_and_return_strerror_r (int errnum)
+{
+  char *buf, *ret;
+
+  buf = xmalloc (1024);
+  ret = strerror_r (errnum, buf, 1024);
+
+  /* "Unknown error" strings are returned in user supplied buffer.  */
+  if (strstr (ret, "Unknown error ") == ret)
+    TEST_VERIFY_EXIT (ret == buf);
+  else
+    {
+      TEST_VERIFY_EXIT (ret != buf);
+      free (buf);
+    }
+
+  return ret;
+}
+
+#define TEST_STRERROR_VARIANT test_and_return_strerror_r
+#include "tst-strerror-strings.c"
-- 
2.50.1



More information about the Libc-alpha mailing list