utimens: fix use of futimens on older Linux

Using UTIME_OMIT with non-zero seconds was already worked around
for utimensat, but I missed the futimens case.

* lib/utimens.c (fdutimens): Use updated, rather than original,
timespec to avoid bug in older Linux kernel.
Reported by Simon Josefsson.

Signed-off-by: Eric Blake <ebb9@byu.net>

diff --git a/ChangeLog b/ChangeLog
index 58635bdaa5d5da8dfb6bdec8d9d57128828c016b..57a7fdb32bd189bee59031d7a99ba5ccad29f253 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2009-11-04  Eric Blake  <ebb9@byu.net>
+
+       utimens: fix use of futimens on older Linux
+       * lib/utimens.c (fdutimens): Use updated, rather than original,
+       timespec to avoid bug in older Linux kernel.
+       Reported by Simon Josefsson.
+
 2009-11-04  Bruno Haible  <bruno@clisp.org>
 
        Make num_processors more flexible and consistent.

diff --git a/lib/utimens.c b/lib/utimens.c
index ffc60b699a68e014ffb1fa12e5596e971ec512af..ce759a1d09cd3db87c108eb421d6484801a5a311 100644 (file)
--- a/lib/utimens.c
+++ b/lib/utimens.c
@@ -227,7 +227,7 @@ fdutimens (char const *file, int fd, struct timespec const timespec[2])
 # endif /* HAVE_UTIMENSAT */
 # if HAVE_FUTIMENS
       {
-        int result = futimens (fd, timespec);
+        int result = futimens (fd, ts);
 #  ifdef __linux__
         /* Work around the same bug as above.  */
         if (0 < result)