mach: strerror must not return NULL (bug 30555)
authorSamuel Thibault <samuel.thibault@ens-lyon.org>
Sun, 2 Jul 2023 11:27:51 +0000 (11:27 +0000)
committerSamuel Thibault <samuel.thibault@ens-lyon.org>
Sun, 2 Jul 2023 11:27:51 +0000 (11:27 +0000)
This follows 1d44530a5be2 ("string: strerror must not return NULL (bug 30555)"):

«
    For strerror, this fixes commit 28aff047818eb1726394296d27b ("string:
    Implement strerror in terms of strerror_l").  This commit avoids
    returning NULL for strerror_l as well, although POSIX allows this
    behavior for strerror_l.
»

sysdeps/mach/strerror_l.c

index 412e38ba5276ed6fd040fccae0ef305dbb465754..6555185da7e03e7da8a03ccaebc81ebb53e0da47 100644 (file)
@@ -61,11 +61,18 @@ __strerror_l (int errnum, locale_t loc)
       free (tls_internal->strerror_l_buf);
       if (__asprintf (&tls_internal->strerror_l_buf, "%s%X",
                      translate ("Error in unknown error system: ", loc),
-                     errnum) == -1)
-       tls_internal->strerror_l_buf = NULL;
+                     errnum) > 0)
+       err = tls_internal->strerror_l_buf;
+      else
+       {
+         /* The memory was freed above.  */
+         tls_internal->strerror_l_buf = NULL;
+         /* Provide a fallback translation.  */
+         err = (char *) translate ("Unknown error", loc);
+       }
 
       __set_errno (saved_errno);
-      return tls_internal->strerror_l_buf;
+      return err;
     }
 
   es = &__mach_error_systems[system];
@@ -79,10 +86,15 @@ __strerror_l (int errnum, locale_t loc)
       if (__asprintf (&tls_internal->strerror_l_buf, "%s%s %d",
                      translate ("Unknown error ", loc),
                      translate (es->subsystem[sub].subsys_name, loc),
-                     errnum) == -1)
-       tls_internal->strerror_l_buf = NULL;
-
-      err = tls_internal->strerror_l_buf;
+                     errnum) > 0)
+       err = tls_internal->strerror_l_buf;
+      else
+       {
+         /* The memory was freed above.  */
+         tls_internal->strerror_l_buf = NULL;
+         /* Provide a fallback translation.  */
+         err = (char *) translate ("Unknown error", loc);
+       }
     }
   else
     err = (char *) translate (es->subsystem[sub].codes[code], loc);