Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Organization: Red Hat UK Ltd. Registered Address: Red Hat UK Ltd, Amberley
        Place, 107-111 Peascod Street, Windsor, Berkshire, SI4 1TE, United
        Kingdom.
        Registered in England and Wales under Company Registration No. 3798903
From:   David Howells <dhowells@redhat.com>
To:     torvalds@linux-foundation.org
cc:     dhowells@redhat.com, Jeffrey Altman <jaltman@auristor.com>,
        Marc Dionne <marc.dionne@auristor.com>,
        linux-afs@lists.infradead.org, linux-fsdevel@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: [PATCH] afs: Fix setting of mtime when creating a file/dir/symlink
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <2004569.1686127633.1@warthog.procyon.org.uk>
Content-Transfer-Encoding: quoted-printable
Date:   Wed, 07 Jun 2023 09:47:13 +0100
Message-ID: <2004570.1686127633@warthog.procyon.org.uk>
Precedence: bulk

kafs incorrectly passes a zero mtime (ie. 1st Jan 1970) to the server when
creating a file, dir or symlink because the mtime recorded in the
afs_operation struct gets passed to the server by the marshalling routines=
,
but the afs_mkdir(), afs_create() and afs_symlink() functions don't set it=
.

This gets masked if a file or directory is subsequently modified.

Fix this by filling in op->mtime before calling the create op.

Fixes: e49c7b2f6de7 ("afs: Build an abstraction around an "operation" conc=
ept")
Signed-off-by: David Howells <dhowells@redhat.com>
Reviewed-by: Jeffrey Altman <jaltman@auristor.com>
Reviewed-by: Marc Dionne <marc.dionne@auristor.com>
cc: linux-afs@lists.infradead.org
cc: linux-fsdevel@vger.kernel.org
---
 fs/afs/dir.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/fs/afs/dir.c b/fs/afs/dir.c
index 4dd97afa536c..5219182e52e1 100644
--- a/fs/afs/dir.c
+++ b/fs/afs/dir.c
@@ -1358,6 +1358,7 @@ static int afs_mkdir(struct mnt_idmap *idmap, struct=
 inode *dir,
 	op->dentry	=3D dentry;
 	op->create.mode	=3D S_IFDIR | mode;
 	op->create.reason =3D afs_edit_dir_for_mkdir;
+	op->mtime	=3D current_time(dir);
 	op->ops		=3D &afs_mkdir_operation;
 	return afs_do_sync_operation(op);
 }
@@ -1661,6 +1662,7 @@ static int afs_create(struct mnt_idmap *idmap, struc=
t inode *dir,
 	op->dentry	=3D dentry;
 	op->create.mode	=3D S_IFREG | mode;
 	op->create.reason =3D afs_edit_dir_for_create;
+	op->mtime	=3D current_time(dir);
 	op->ops		=3D &afs_create_operation;
 	return afs_do_sync_operation(op);
 =

@@ -1796,6 +1798,7 @@ static int afs_symlink(struct mnt_idmap *idmap, stru=
ct inode *dir,
 	op->ops			=3D &afs_symlink_operation;
 	op->create.reason	=3D afs_edit_dir_for_symlink;
 	op->create.symlink	=3D content;
+	op->mtime		=3D current_time(dir);
 	return afs_do_sync_operation(op);
 =

 error: