Make std.file.timeLastModified with two arguments safe

dlang · Aug 29, 2014 · f7fc11e · f7fc11e
1 parent 6cd2e43
commit f7fc11e
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/std/file.d b/std/file.d
@@ -905,25 +905,29 @@ else
 }
 --------------------
 +/
-SysTime timeLastModified(in char[] name, SysTime returnIfMissing)
+SysTime timeLastModified(in char[] name, SysTime returnIfMissing) @safe
 {
     version(Windows)
     {
         if(!exists(name))
             return returnIfMissing;
 
-        SysTime dummy = void;
-        SysTime ftm = void;
+        SysTime dummy;
+        SysTime ftm;
 
         getTimesWin(name, dummy, dummy, ftm);
 
         return ftm;
     }
     else version(Posix)
     {
+        static auto trustedStat(in char[] path, ref stat_t buf) @trusted
+        {
+            return stat(path.tempCString(), &buf);
+        }
         stat_t statbuf = void;
 
-        return stat(name.tempCString(), &statbuf) != 0 ?
+        return trustedStat(name, statbuf) != 0 ?
                returnIfMissing :
                SysTime(unixTimeToStdTime(statbuf.st_mtime));
     }