From: David Howells Date: Mon, 16 Dec 2024 20:41:02 +0000 (+0000) Subject: afs: Fix EEXIST error returned from afs_rmdir() to be ENOTEMPTY X-Git-Tag: v6.14-rc1~218^2^2~19 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b49194da2aff2c879dec9c59ef8dec0f2b0809ef;p=thirdparty%2Flinux.git afs: Fix EEXIST error returned from afs_rmdir() to be ENOTEMPTY AFS servers pass back a code indicating EEXIST when they're asked to remove a directory that is not empty rather than ENOTEMPTY because not all the systems that an AFS server can run on have the latter error available and AFS preexisted the addition of that error in general. Fix afs_rmdir() to translate EEXIST to ENOTEMPTY. Fixes: 260a980317da ("[AFS]: Add "directory write" support.") Signed-off-by: David Howells Link: https://lore.kernel.org/r/20241216204124.3752367-13-dhowells@redhat.com cc: Marc Dionne cc: linux-afs@lists.infradead.org Signed-off-by: Christian Brauner --- diff --git a/fs/afs/dir.c b/fs/afs/dir.c index ada363af5aab8..50edd1cae28ac 100644 --- a/fs/afs/dir.c +++ b/fs/afs/dir.c @@ -1472,7 +1472,12 @@ static int afs_rmdir(struct inode *dir, struct dentry *dentry) op->file[1].vnode = vnode; } - return afs_do_sync_operation(op); + ret = afs_do_sync_operation(op); + + /* Not all systems that can host afs servers have ENOTEMPTY. */ + if (ret == -EEXIST) + ret = -ENOTEMPTY; + return ret; error: return afs_put_operation(op);