Merge pull request #390 from trapexit/futimesat

abstract futimesat

src/fs_base_futimesat.cpp

@@ -0,0 +1,23 @@
+/*
+  ISC License
+
+  Copyright (c) 2017, Antonio SJ Musumeci <trapexit@spawn.link>
+
+  Permission to use, copy, modify, and/or distribute this software for any
+  purpose with or without fee is hereby granted, provided that the above
+  copyright notice and this permission notice appear in all copies.
+
+  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+*/
+
+#if __APPLE__
+# include "fs_base_futimesat_osx.icpp"
+#else
+# include "fs_base_futimesat_generic.icpp"
+#endif

src/fs_base_futimesat.hpp

@@ -0,0 +1,30 @@
+/*
+  ISC License
+
+  Copyright (c) 2017, Antonio SJ Musumeci <trapexit@spawn.link>
+
+  Permission to use, copy, modify, and/or distribute this software for any
+  purpose with or without fee is hereby granted, provided that the above
+  copyright notice and this permission notice appear in all copies.
+
+  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+*/
+
+#ifndef __FS_BASE_FUTIMESAT_HPP__
+#define __FS_BASE_FUTIMESAT_HPP__
+
+namespace fs
+{
+  int
+  futimesat(const int             dirfd,
+            const char           *pathname,
+            const struct timeval  times[2]);
+}
+
+#endif

src/fs_base_futimesat_generic.icpp

@@ -0,0 +1,31 @@
+/*
+  ISC License
+
+  Copyright (c) 2017, Antonio SJ Musumeci <trapexit@spawn.link>
+
+  Permission to use, copy, modify, and/or distribute this software for any
+  purpose with or without fee is hereby granted, provided that the above
+  copyright notice and this permission notice appear in all copies.
+
+  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+*/
+
+#include <fcntl.h>
+#include <sys/time.h>
+
+namespace fs
+{
+  int
+  futimesat(const int             dirfd,
+            const char           *pathname,
+            const struct timeval  times[2])
+  {
+    return ::futimesat(dirfd,pathname,times);
+  }
+}

src/fs_base_futimesat_osx.icpp

@@ -0,0 +1,84 @@
+/*
+  ISC License
+
+  Copyright (c) 2017, Antonio SJ Musumeci <trapexit@spawn.link>
+
+  Permission to use, copy, modify, and/or distribute this software for any
+  purpose with or without fee is hereby granted, provided that the above
+  copyright notice and this permission notice appear in all copies.
+
+  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+*/
+
+#include <err.h>
+#include <fcntl.h>
+#include <stdio.h>
+#include <string.h>
+#include <sys/errno.h>
+#include <sys/param.h>
+#include <sys/stat.h>
+#include <sys/time.h>
+#include <unistd.h>
+
+static
+int
+getpath(const int   dirfd,
+        const char *path,
+        char       *fullpath)
+{
+  int rv;
+  struct stat st;
+
+  rv = ::fstat(dirfd,&st);
+  if(rv == -1)
+    return -1;
+
+  if(!S_ISDIR(st.st_mode))
+    return (errno=ENOTDIR,-1);
+
+  rv = ::fcntl(dirfd,F_GETPATH,fullpath);
+  if(rv == -1)
+    return -1;
+
+  rv = ::strlcat(fullpath,"/",MAXPATHLEN);
+  if(rv > MAXPATHLEN)
+    return (errno=ENAMETOOLONG,-1);
+
+  rv = ::strlcat(fullpath,path,MAXPATHLEN);
+  if(rv > MAXPATHLEN)
+    return (errno=ENAMETOOLONG,-1);
+
+  return 0;
+}
+
+namespace fs
+{
+  int
+  futimesat(const int             dirfd,
+            const char           *pathname,
+            const struct timeval  times[2])
+  {
+    int rv;
+    char fullpath[MAXPATHLEN];
+
+    if((dirfd == AT_FDCWD) ||
+       ((pathname != NULL) &&
+        (pathname[0] == '/')))
+      return ::utimes(pathname,times);
+
+    if(dirfd < 0)
+      return (errno=EBADF,-1);
+
+    rv = getpath(dirfd,pathname,fullpath);
+    if(rv == -1)
+      return -1;
+
+    return ::utimes(fullpath,times);
+  }
+}

src/fs_base_utime_generic.hpp

@@ -25,6 +25,7 @@
 #include <sys/stat.h>
 #include <sys/time.h>
 
+#include "fs_base_futimesat.hpp"
 #include "fs_base_stat.hpp"
 
 #ifndef UTIME_NOW
@@ -169,7 +170,7 @@ _set_utime_omit_to_current_value(const int             fd,
   mtime = fs::stat_mtime(st);
 
   if(ts[0].tv_nsec == UTIME_OMIT)
-    TIMESPEC_TO_TIMEVAL(&tv[0],atime;
+    TIMESPEC_TO_TIMEVAL(&tv[0],atime);
   if(ts[1].tv_nsec == UTIME_OMIT)
     TIMESPEC_TO_TIMEVAL(&tv[1],mtime);
 
@@ -282,7 +283,7 @@ namespace fs
       return -1;
 
     if((flags & AT_SYMLINK_NOFOLLOW) == 0)
-      return ::futimesat(dirfd,path.c_str(),tvp);
+      return fs::futimesat(dirfd,path.c_str(),tvp);
     if(_can_call_lutimes(dirfd,path,flags))
       return ::lutimes(path.c_str(),tvp);