| Message ID | 20241112202552.3393751-1-viro@zeniv.linux.org.uk (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [1/5] io_statx_prep(): use getname_uflags() | expand |
On Tue, Nov 12, 2024 at 08:25:48PM +0000, Al Viro wrote: > the only thing in flags getname_flags() ever cares about is > LOOKUP_EMPTY; anything else is none of its damn business. > > Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> > --- Reviewed-by: Christian Brauner <brauner@kernel.org>
Looks fine to me:
Reviewed-by: Jens Axboe <axboe@kernel.dk>
diff --git a/io_uring/statx.c b/io_uring/statx.c index f7f9b202eec0..6bc4651700a2 100644 --- a/io_uring/statx.c +++ b/io_uring/statx.c @@ -36,8 +36,7 @@ int io_statx_prep(struct io_kiocb *req, const struct io_uring_sqe *sqe) sx->buffer = u64_to_user_ptr(READ_ONCE(sqe->addr2)); sx->flags = READ_ONCE(sqe->statx_flags); - sx->filename = getname_flags(path, - getname_statx_lookup_flags(sx->flags)); + sx->filename = getname_uflags(path, sx->flags); if (IS_ERR(sx->filename)) { int ret = PTR_ERR(sx->filename);
the only thing in flags getname_flags() ever cares about is LOOKUP_EMPTY; anything else is none of its damn business. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> --- io_uring/statx.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)