]> Savannah Git Hosting - gnulib.git/commitdiff
strnlen: avoid undefined memcmp behavior
authorPaul Eggert <eggert@cs.ucla.edu>
Sun, 30 Jun 2024 10:27:48 +0000 (11:27 +0100)
committerPaul Eggert <eggert@cs.ucla.edu>
Sun, 30 Jun 2024 10:28:07 +0000 (11:28 +0100)
Problem reported by Po Lu in:
https://lists.gnu.org/r/bug-gnulib/2024-06/msg00288.html
* config/srclist.txt: Don’t mention strnlen.c even in a comment,
as the Gnulib and glibc implementations have diverged for
portability reasons, and they’re never likely to merge.
* lib/strnlen.c (strnlen): Avoid undefined behavior if
the array S points to has fewer than MAXLEN bytes.

ChangeLog
config/srclist.txt
lib/strnlen.c

index b10396d1fe825d8022677f6a460e799239eb4cfa..d2d81c25a1fd4123782dd6b67db3ffa2b2d2caad 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2024-06-30  Paul Eggert  <eggert@cs.ucla.edu>
+
+       strnlen: avoid undefined memcmp behavior
+       Problem reported by Po Lu in:
+       https://lists.gnu.org/r/bug-gnulib/2024-06/msg00288.html
+       * config/srclist.txt: Don’t mention strnlen.c even in a comment,
+       as the Gnulib and glibc implementations have diverged for
+       portability reasons, and they’re never likely to merge.
+       * lib/strnlen.c (strnlen): Avoid undefined behavior if
+       the array S points to has fewer than MAXLEN bytes.
+
 2024-06-30  Bruno Haible  <bruno@clisp.org>
 
        tests: Avoid warning "is_running_under_virtualbox defined but not used".
index 8930847018075e4ee19e6aa31beca8cc5d3f112d..ce47f3f210b94e493cc9538765cf5b5570a8ea8f 100644 (file)
@@ -205,7 +205,6 @@ $LIBCSRC time/mktime-internal.h             lib
 #$LIBCSRC string/strcasecmp.c                  lib gpl
 #$LIBCSRC string/strchrnul.c                   lib gpl
 #$LIBCSRC string/strerror.c                    lib gpl
-#$LIBCSRC string/strnlen.c                     lib gpl
 #$LIBCSRC sysdeps/posix/gettimeofday.c         lib gpl
 #$LIBCSRC sysdeps/posix/rename.c               lib gpl
 #$LIBCSRC sysdeps/unix/mkdir.c                 lib gpl
index 80857ec22b038fc51f2d2ad304e7aab35af0ff3c..5231e4c595da709406f215a4c9dbd9619e670e61 100644 (file)
@@ -1,6 +1,5 @@
 /* Find the length of STRING, but scan at most MAXLEN characters.
    Copyright (C) 2005-2007, 2009-2024 Free Software Foundation, Inc.
-   Written by Simon Josefsson.
 
    This file is free software: you can redistribute it and/or modify
    it under the terms of the GNU Lesser General Public License as
 
 #include <string.h>
 
-/* Find the length of STRING, but scan at most MAXLEN characters.
-   If no '\0' terminator is found in that many characters, return MAXLEN.  */
+/* Find the length of S, but scan at most MAXLEN bytes.
+   S must be a string if it starts with fewer than MAXLEN initialized bytes.
+   If no '\0' terminator is found in that many bytes, return MAXLEN.  */
 
 size_t
-strnlen (const char *string, size_t maxlen)
+strnlen (const char *s, size_t maxlen)
 {
-  const char *end = memchr (string, '\0', maxlen);
-  return end ? (size_t) (end - string) : maxlen;
+  /* Do not use memchr, because on some platforms memchr has
+     undefined behavior if MAXLEN exceeds the number of bytes in S.  */
+  size_t i;
+  for (i = 0; i < maxlen && s[i]; i++)
+    continue;
+  return i;
 }