When setting the timestamps of a file using e.g. futimens the flag
HOSTFS_ATTR_(A|M)TIME_SET is set only if explicit timestamps were
specified. If futimens is called without timestamps (i.e. parameter
times is set to NULL), only HOSTFS_ATTR_(A|M)TIME is set. On the other
hand, the latter flag is also set if the timestamps were explicitly
specified.

Signed-off-by: Tobias Brunner <tob...@strongswan.org>
---
 fs/hostfs/hostfs_user.c |   15 ++++-----------
 1 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/fs/hostfs/hostfs_user.c b/fs/hostfs/hostfs_user.c
index b79424f..3da7cdd 100644
--- a/fs/hostfs/hostfs_user.c
+++ b/fs/hostfs/hostfs_user.c
@@ -277,7 +277,8 @@ int set_attr(const char *file, struct hostfs_iattr
*attrs, int fd)
         * times according to the changes to perform, and then call futimes()
         * or utimes() to apply them.
         */
-       ma = (HOSTFS_ATTR_ATIME_SET | HOSTFS_ATTR_MTIME_SET);
+       /* Note: ctime is not handled */
+       ma = (HOSTFS_ATTR_ATIME | HOSTFS_ATTR_MTIME);
        if (attrs->ia_valid & ma) {
                err = stat_file(file, NULL, NULL, NULL, NULL, NULL, NULL,
                                &atime_ts, &mtime_ts, NULL, NULL, NULL, fd);
@@ -289,11 +290,11 @@ int set_attr(const char *file, struct hostfs_iattr
*attrs, int fd)
                times[1].tv_sec = mtime_ts.tv_sec;
                times[1].tv_usec = mtime_ts.tv_nsec / 1000;

-               if (attrs->ia_valid & HOSTFS_ATTR_ATIME_SET) {
+               if (attrs->ia_valid & HOSTFS_ATTR_ATIME) {
                        times[0].tv_sec = attrs->ia_atime.tv_sec;
                        times[0].tv_usec = attrs->ia_atime.tv_nsec / 1000;
                }
-               if (attrs->ia_valid & HOSTFS_ATTR_MTIME_SET) {
+               if (attrs->ia_valid & HOSTFS_ATTR_MTIME) {
                        times[1].tv_sec = attrs->ia_mtime.tv_sec;
                        times[1].tv_usec = attrs->ia_mtime.tv_nsec / 1000;
                }
@@ -306,14 +307,6 @@ int set_attr(const char *file, struct hostfs_iattr
*attrs, int fd)
                }
        }

-       /* Note: ctime is not handled */
-       if (attrs->ia_valid & (HOSTFS_ATTR_ATIME | HOSTFS_ATTR_MTIME)) {
-               err = stat_file(file, NULL, NULL, NULL, NULL, NULL, NULL,
-                               &attrs->ia_atime, &attrs->ia_mtime, NULL,
-                               NULL, NULL, fd);
-               if (err != 0)
-                       return err;
-       }
        return 0;
 }


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Reply via email to