Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 14 Apr 2023 12:46:25 +0200
From:   Jan Kara <jack@suse.cz>
To:     Luca Vizzarro <Luca.Vizzarro@arm.com>
Cc:     linux-kernel@vger.kernel.org,
        Kevin Brodsky <Kevin.Brodsky@arm.com>,
        Szabolcs Nagy <Szabolcs.Nagy@arm.com>,
        Theodore Ts'o <tytso@mit.edu>,
        David Laight <David.Laight@ACULAB.com>,
        Mark Rutland <Mark.Rutland@arm.com>,
        Alexander Viro <viro@zeniv.linux.org.uk>,
        Christian Brauner <brauner@kernel.org>,
        Jeff Layton <jlayton@kernel.org>,
        Chuck Lever <chuck.lever@oracle.com>,
        linux-fsdevel@vger.kernel.org, Jan Kara <jack@suse.cz>,
        Amir Goldstein <amir73il@gmail.com>
Subject: Re: [PATCH 5/5] dnotify: Pass argument of fcntl_dirnotify as int
Message-ID: <20230414104625.gyzuswldwil4jlfw@quack3>
References: <20230414100212.766118-1-Luca.Vizzarro@arm.com>
 <20230414100212.766118-6-Luca.Vizzarro@arm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230414100212.766118-6-Luca.Vizzarro@arm.com>
Precedence: bulk

On Fri 14-04-23 11:02:12, Luca Vizzarro wrote:
> The interface for fcntl expects the argument passed for the command
> F_DIRNOTIFY to be of type int. The current code wrongly treats it as
> a long. In order to avoid access to undefined bits, we should explicitly
> cast the argument to int.
> 
> Cc: Kevin Brodsky <Kevin.Brodsky@arm.com>
> Cc: Szabolcs Nagy <Szabolcs.Nagy@arm.com>
> Cc: "Theodore Ts'o" <tytso@mit.edu>
> Cc: David Laight <David.Laight@ACULAB.com>
> Cc: Mark Rutland <Mark.Rutland@arm.com>
> Cc: Alexander Viro <viro@zeniv.linux.org.uk>
> Cc: Christian Brauner <brauner@kernel.org>
> Cc: Jeff Layton <jlayton@kernel.org>
> Cc: Chuck Lever <chuck.lever@oracle.com>
> Cc: linux-fsdevel@vger.kernel.org
> Cc: Jan Kara <jack@suse.cz>
> Cc: Amir Goldstein <amir73il@gmail.com>
> Signed-off-by: Luca Vizzarro <Luca.Vizzarro@arm.com>

Looks good to me. Do you plan to merge this series together (perhaps
Christian could?) or should I pick up the dnotify patch? In case someone
else will merge the patch feel free to add:

Acked-by: Jan Kara <jack@suse.cz>

								Honza

> ---
>  fs/notify/dnotify/dnotify.c | 4 ++--
>  include/linux/dnotify.h     | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/fs/notify/dnotify/dnotify.c b/fs/notify/dnotify/dnotify.c
> index 190aa717fa32..ebdcc25df0f7 100644
> --- a/fs/notify/dnotify/dnotify.c
> +++ b/fs/notify/dnotify/dnotify.c
> @@ -199,7 +199,7 @@ void dnotify_flush(struct file *filp, fl_owner_t id)
>  }
> 
>  /* this conversion is done only at watch creation */
> -static __u32 convert_arg(unsigned long arg)
> +static __u32 convert_arg(unsigned int arg)
>  {
>         __u32 new_mask = FS_EVENT_ON_CHILD;
> 
> @@ -258,7 +258,7 @@ static int attach_dn(struct dnotify_struct *dn, struct dnotify_mark *dn_mark,
>   * up here.  Allocate both a mark for fsnotify to add and a dnotify_struct to be
>   * attached to the fsnotify_mark.
>   */
> -int fcntl_dirnotify(int fd, struct file *filp, unsigned long arg)
> +int fcntl_dirnotify(int fd, struct file *filp, unsigned int arg)
>  {
>         struct dnotify_mark *new_dn_mark, *dn_mark;
>         struct fsnotify_mark *new_fsn_mark, *fsn_mark;
> diff --git a/include/linux/dnotify.h b/include/linux/dnotify.h
> index b1d26f9f1c9f..9f183a679277 100644
> --- a/include/linux/dnotify.h
> +++ b/include/linux/dnotify.h
> @@ -30,7 +30,7 @@ struct dnotify_struct {
>                             FS_MOVED_FROM | FS_MOVED_TO)
> 
>  extern void dnotify_flush(struct file *, fl_owner_t);
> -extern int fcntl_dirnotify(int, struct file *, unsigned long);
> +extern int fcntl_dirnotify(int, struct file *, unsigned int);
> 
>  #else
> 
> @@ -38,7 +38,7 @@ static inline void dnotify_flush(struct file *filp, fl_owner_t id)
>  {
>  }
> 
> -static inline int fcntl_dirnotify(int fd, struct file *filp, unsigned long arg)
> +static inline int fcntl_dirnotify(int fd, struct file *filp, unsigned int arg)
>  {
>         return -EINVAL;
>  }
> --
> 2.34.1
> 
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR