Re: [PATCH v2 04/13] io_[gs]etxattr_prep(): just use getname()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Al Viro <viro@xxxxxxxxxxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH v2 04/13] io_[gs]etxattr_prep(): just use getname()
From: Jens Axboe <axboe@xxxxxxxxx>
Date: Sat, 2 Nov 2024 08:44:08 -0600
Cc: Christian Brauner <brauner@xxxxxxxxxx>, io-uring@xxxxxxxxxxxxxxx, Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
In-reply-to: <20241102073149.2457240-4-viro@zeniv.linux.org.uk>
User-agent: Mozilla Thunderbird

On 11/2/24 1:31 AM, Al Viro wrote:
> getname_flags(pathname, LOOKUP_FOLLOW) is obviously bogus - following
> trailing symlinks has no impact on how to copy the pathname from userland...

Reviewed-by: Jens Axboe <axboe@xxxxxxxxx>

-- 
Jens Axboe

References:
- [RFC][PATCHES v2] xattr stuff and interactions with io_uring
  - From: Al Viro
- [PATCH v2 01/13] teach filename_lookup() to treat NULL filename as ""
  - From: Al Viro
- [PATCH v2 04/13] io_[gs]etxattr_prep(): just use getname()
  - From: Al Viro

Prev by Date: Re: [RFC][PATCHES v2] xattr stuff and interactions with io_uring
Next by Date: Re: [PATCH] fs: binfmt: Fix a typo
Previous by thread: [PATCH v2 04/13] io_[gs]etxattr_prep(): just use getname()
Next by thread: [PATCH v2 06/13] fs: rename struct xattr_ctx to kernel_xattr_ctx
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]

Powered by Linux