lkml.org 
[lkml]   [2024]   [Mar]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] afs: Revert "afs: Hide silly-rename files from userspace"
From


On 3/13/24 04:08, David Howells wrote:
>
> This reverts commit 57e9d49c54528c49b8bffe6d99d782ea051ea534.
>
> This undoes the hiding of .__afsXXXX silly-rename files. The problem with
> hiding them is that rm can't then manually delete them.
>
> This also reverts commit 5f7a07646655fb4108da527565dcdc80124b14c4 ("afs: Fix
> endless loop in directory parsing") as that's a bugfix for the above.
>
> Fixes: 57e9d49c5452 ("afs: Hide silly-rename files from userspace")
> Reported-by: Markus Suvanto <markus.suvanto@gmail.com>
> Link: https://lists.infradead.org/pipermail/linux-afs/2024-February/008102html

Not Found

The requested URL was not found on this server.

> Signed-off-by: David Howells <dhowells@redhat.com>
> cc: Marc Dionne <marc.dionne@auristor.com>
> cc: linux-afs@lists.infradead.org
> ---
> fs/afs/dir.c | 10 ----------
> 1 file changed, 10 deletions(-)
>
> diff --git a/fs/afs/dir.c b/fs/afs/dir.c
> index 8a67fc427e74..67afe68972d5 100644
> --- a/fs/afs/dir.c
> +++ b/fs/afs/dir.c
> @@ -474,16 +474,6 @@ static int afs_dir_iterate_block(struct afs_vnode *dvnode,
> continue;
> }
>
> - /* Don't expose silly rename entries to userspace. */
> - if (nlen > 6 &&
> - dire->u.name[0] == '.' &&
> - ctx->actor != afs_lookup_filldir &&
> - ctx->actor != afs_lookup_one_filldir &&
> - memcmp(dire->u.name, ".__afs", 6) == 0) {
> - ctx->pos = blkoff + next * sizeof(union afs_xdr_dirent);
> - continue;
> - }
> -
> /* found the next entry */
> if (!dir_emit(ctx, dire->u.name, nlen,
> ntohl(dire->u.vnode),
>
>

--
#Randy

\
 
 \ /
  Last update: 2024-03-13 16:16    [W:0.060 / U:0.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site