From: Ondrej Kozina <okozina@redhat.com>
Date: Wed, 27 Feb 2013 12:49:15 +0000 (+0100)
Subject: - add ctime check only for future consideration
X-Git-Tag: v0.1.3~18^2~22
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=e9e48c0a32ef0dd85a6a8b271f94fdeae4930084;p=thirdparty%2Fsnapper.git

- add ctime check only for future consideration
---

diff --git a/snapper/Compare.cc b/snapper/Compare.cc
index 8c48ead4..e82195d8 100644
--- a/snapper/Compare.cc
+++ b/snapper/Compare.cc
@@ -183,16 +183,18 @@ namespace snapper
         unsigned int status = 0;
 
         /*
-         * NOTE:
+         * NOTE: just for a consideration
          *
          * if both ctimes are in match, files should be same
          * ctime can be altered only by some debugfs tool and
-         * on umounted fs (ext2,3,4...). So this should be safe
+         * on umounted fs (ext2,3,4, xfs). So this should be safe
          * unless root or CAP_SYS_ADMIN played with low-level fs
-         * utilities
+         * utilities, right?
+         *
+         * if ((stat1.st_ctime == stat2.st_ctime))
+         *      return status;
+         *
          */
-	if ((stat1.st_ctime == stat2.st_ctime))
-	    return status;
 
 	if ((stat1.st_mode & S_IFMT) != (stat2.st_mode & S_IFMT))
 	{