Unverified Commit 34603c90 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!12294 nfsd: return -EINVAL when namelen is 0 

Merge Pull Request from: @ci-robot 
 
PR sync from: Li Lingfeng <lilingfeng3@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/WSRFAYXURKZY4J6IRWHCMUU64P5LGEQU/ 
 
https://gitee.com/openeuler/kernel/issues/IAYQ6P 
 
Link:https://gitee.com/openeuler/kernel/pulls/12294

 

Reviewed-by: default avatarYang Yingliang <yangyingliang@huawei.com>
Signed-off-by: default avatarYang Yingliang <yangyingliang@huawei.com>
parents 5faf5aae 4fc1d598

fs/nfsd/nfs4recover.c

+8 −0
Original line number Diff line number Diff line
@@ -806,6 +806,10 @@ __cld_pipe_inprogress_downcall(const struct cld_msg_v2 __user *cmsg, 
			ci = &cmsg->cm_u.cm_clntinfo;

			if (get_user(namelen, &ci->cc_name.cn_len))

				return -EFAULT;

			if (!namelen) {

				dprintk("%s: namelen should not be zero", __func__);

				return -EINVAL;

			}

			name.data = memdup_user(&ci->cc_name.cn_id, namelen);

			if (IS_ERR_OR_NULL(name.data))

				return -EFAULT;
@@ -828,6 +832,10 @@ __cld_pipe_inprogress_downcall(const struct cld_msg_v2 __user *cmsg, 
			cnm = &cmsg->cm_u.cm_name;

			if (get_user(namelen, &cnm->cn_len))

				return -EFAULT;

			if (!namelen) {

				dprintk("%s: namelen should not be zero", __func__);

				return -EINVAL;

			}

			name.data = memdup_user(&cnm->cn_id, namelen);

			if (IS_ERR_OR_NULL(name.data))

				return -EFAULT;