From: Bruno Haible <bruno@clisp.org>
Date: Thu, 14 Aug 2025 20:05:37 +0000 (+0200)
Subject: chown: Avoid a redundant stat() call.
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=64ac89d38d76688b53edb26ca38720d6cc9cad36;p=thirdparty%2Fgnulib.git

chown: Avoid a redundant stat() call.

* lib/chown.c (rpl_chown): Set stat_valid after stat() succeeded.
---

diff --git a/ChangeLog b/ChangeLog
index b072f949d7..b69ff7406c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2025-08-14  Bruno Haible  <bruno@clisp.org>
+
+	chown: Avoid a redundant stat() call.
+	* lib/chown.c (rpl_chown): Set stat_valid after stat() succeeded.
+
 2025-08-14  Bruno Haible  <bruno@clisp.org>
 
 	lchown: Use issymlink.
diff --git a/lib/chown.c b/lib/chown.c
index 2ad9972100..97cacb82da 100644
--- a/lib/chown.c
+++ b/lib/chown.c
@@ -75,6 +75,7 @@ rpl_chown (const char *file, uid_t uid, gid_t gid)
       /* Stat file to get id(s) that should remain unchanged.  */
       if (!stat_valid && stat (file, &st))
         return -1;
+      stat_valid = true;
       if (gid == (gid_t) -1)
         gid = st.st_gid;
       if (uid == (uid_t) -1)