On Wed, Apr 08, 2020 at 08:48:05AM -0400, Jeff Layton wrote:
>
> Sound ok?
Sounds good.
> fs/ceph/mds_client.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/ceph/mds_client.h b/fs/ceph/mds_client.h
> index 1b40f30e0a8e..754e0682398e 100644
> --- a/fs/ceph/mds_client.h
> +++ b/fs/ceph/mds_client.h
> @@ -531,7 +531,7 @@ extern void ceph_mdsc_pre_umount(struct ceph_mds_client *mdsc);
>
> static inline void ceph_mdsc_free_path(char *path, int len)
> {
> - if (path)
> + if (path && !IS_ERR(path))
if (!IS_ERR_OR_NULL(path)
> __putname(path - (PATH_MAX - 1 - len));
> }
>
> --
> 2.25.2
>
regards,
dan carpenter
