diff options
| author | David Howells <dhowells@redhat.com> | 2024-09-14 22:40:02 +0200 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2024-09-27 18:29:19 +0200 |
| commit | 2cf36327ee1e47733aba96092d7bd082a4056ff5 (patch) | |
| tree | 535561ae2f827bfb59533e7891c4edde02144c5a | |
| parent | cachefiles: fix dentry leak in cachefiles_open_file() (diff) | |
| download | linux-2cf36327ee1e47733aba96092d7bd082a4056ff5.tar.xz linux-2cf36327ee1e47733aba96092d7bd082a4056ff5.zip | |
afs: Fix missing wire-up of afs_retry_request()
afs_retry_request() is supposed to be pointed to by the afs_req_ops netfs
operations table, but the pointer got lost somewhere. The function is used
during writeback to rotate through the authentication keys that were in
force when the file was modified locally.
Fix this by adding the pointer to the function.
Fixes: 1ecb146f7cd8 ("netfs, afs: Use writeback retry to deal with alternate keys")
Reported-by: Dr. David Alan Gilbert <linux@treblig.org>
Signed-off-by: David Howells <dhowells@redhat.com>
Link: https://lore.kernel.org/r/1690847.1726346402@warthog.procyon.org.uk
cc: Marc Dionne <marc.dionne@auristor.com>
cc: Jeff Layton <jlayton@kernel.org>
cc: linux-afs@lists.infradead.org
cc: netfs@lists.linux.dev
cc: linux-fsdevel@vger.kernel.org
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | fs/afs/file.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/afs/file.c b/fs/afs/file.c index 492d857a3fa0..6762eff97517 100644 --- a/fs/afs/file.c +++ b/fs/afs/file.c @@ -420,6 +420,7 @@ const struct netfs_request_ops afs_req_ops = { .begin_writeback = afs_begin_writeback, .prepare_write = afs_prepare_write, .issue_write = afs_issue_write, + .retry_request = afs_retry_request, }; static void afs_add_open_mmap(struct afs_vnode *vnode) |