From: Paul Eggert <eggert@cs.ucla.edu>
Date: Wed, 16 Aug 2006 17:24:35 +0000 (+0000)
Subject: (memcoll): Set errno = 0 in the shortcut case, too.
X-Git-Tag: v6.1~36
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f048f049e7dd531e207c630088d050c376b39feb;p=thirdparty%2Fcoreutils.git

(memcoll): Set errno = 0 in the shortcut case, too.
---

diff --git a/lib/ChangeLog b/lib/ChangeLog
index c65dde33e5..00c19a3c7f 100644
--- a/lib/ChangeLog
+++ b/lib/ChangeLog
@@ -1,3 +1,9 @@
+2006-08-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+	* memcoll.c (memcoll): Set errno = 0 in the shortcut case, too.
+	Problem and fix reported by Pádraig Brady in
+	<http://lists.gnu.org/archive/html/bug-coreutils/2006-08/msg00099.html>.
+
 2006-08-16  Jim Meyering  <jim@meyering.net>
 
 	* fts.c (fts_children): Remove obsolete "// FIXME ..." comment.
diff --git a/lib/memcoll.c b/lib/memcoll.c
index 7f61a6b58c..457d1104a9 100644
--- a/lib/memcoll.c
+++ b/lib/memcoll.c
@@ -43,7 +43,10 @@ memcoll (char *s1, size_t s1len, char *s2, size_t s2len)
      the buffers using strcoll on each substring.  */
 
   if (s1len == s2len && memcmp (s1, s2, s1len) == 0)
-    diff = 0;
+    {
+      errno = 0;
+      diff = 0;
+    }
   else
     {
       char n1 = s1[s1len];