Re: [PATCH v3] vfs: support statx(..., NULL, AT_EMPTY_PATH, ...)

From: Xi Ruoyao
Date: Tue Jun 25 2024 - 23:00:08 EST


On Tue, 2024-06-25 at 17:18 +0200, Mateusz Guzik wrote:
> + if ((sx->flags & (AT_EMPTY_PATH | AT_STATX_SYNC_TYPE)) ==
> +     (AT_EMPTY_PATH | AT_STATX_SYNC_TYPE) &&
> +     vfs_empty_path(sx->dfd, path)) {
>   sx->filename = NULL;
> - return ret;

AT_STATX_SYNC_TYPE == AT_STATX_FORCE_SYNC | AT_STATX_DONT_SYNC but
AT_STATX_FORCE_SYNC and AT_STATX_DONT_SYNC obviously contradicts with
each other. Thus valid uses of statx won't satisfy this condition.

And I guess the condition here should be same as the condition in
SYSCALL_DEFINE5(statx) or am I wrong?

--
Xi Ruoyao <xry111@xxxxxxxxxxx>
School of Aerospace Science and Technology, Xidian University