Merge pull request #312 from yuchen0cc/main

untar: mv store dirs into extract_file
containerd · Jan 26, 2024 · 41bc0db · 41bc0db
2 parents 08847cf + 7aa2ffe
commit 41bc0db
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/overlaybd/tar/libtar.cpp b/src/overlaybd/tar/libtar.cpp
@@ -137,9 +137,6 @@ int UnTar::extract_all() {
         if (extract_file(filename.c_str()) != 0) {
             LOG_ERRNO_RETURN(0, -1, "extract failed, filename `", filename);
         }
-        if (TH_ISDIR(header)) {
-            dirs.emplace_back(std::make_pair(filename, header.get_mtime()));
-        }
         count++;
     }
 
@@ -230,7 +227,11 @@ int UnTar::extract_file(const char *filename) {
     if (i != 0) {
         return i;
     }
-
+    // Directory mtimes must be handled at the end to avoid further
+    // file creation in them to modify the directory mtime
+    if (TH_ISDIR(header)) {
+        dirs.emplace_back(std::make_pair(filename, header.get_mtime()));
+    }
     unpackedPaths.insert(filename);
     return 0;
 }