* linux-next: build warning after merge of the char-misc tree
@ 2017-12-12 3:39 Stephen Rothwell
2017-12-12 10:40 ` [PATCH v2] misc: mic: Use memdup_user() as a cleanup Vasyl Gomonovych
2017-12-12 11:49 ` linux-next: build warning after merge of the char-misc tree Greg KH
0 siblings, 2 replies; 8+ messages in thread
From: Stephen Rothwell @ 2017-12-12 3:39 UTC (permalink / raw)
To: Greg KH, Arnd Bergmann
Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Vasyl Gomonovych
Hi all,
After merging the char-misc tree, today's linux-next build
(x86_64_allmodconfig) produced this warning:
drivers/misc/mic/vop/vop_vringh.c: In function 'vop_ioctl':
drivers/misc/mic/vop/vop_vringh.c:1001:1: warning: label 'done' defined but not used [-Wunused-label]
done:
^
Introduced by commit
30b7a2c19e29 ("misc: mic: Use memdup_user() as a cleanup")
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH v2] misc: mic: Use memdup_user() as a cleanup 2017-12-12 3:39 linux-next: build warning after merge of the char-misc tree Stephen Rothwell @ 2017-12-12 10:40 ` Vasyl Gomonovych 2017-12-12 11:49 ` Greg KH 2017-12-12 11:49 ` linux-next: build warning after merge of the char-misc tree Greg KH 1 sibling, 1 reply; 8+ messages in thread From: Vasyl Gomonovych @ 2017-12-12 10:40 UTC (permalink / raw) To: sfr, greg, arnd, sudeep.dutt, ashutosh.dixit, gomonovych, dan.carpenter Cc: linux-kernel, linux-next Fix coccicheck warning which recommends to use memdup_user(): drivers/misc/mic/vop/vop_vringh.c:940:14-21: WARNING opportunity for memdup_user drivers/misc/mic/vop/vop_vringh.c:998:8-15: WARNING opportunity for memdup_user Generated by: scripts/coccinelle/memdup_user/memdup_user.cocci Changelog: - v1: - Replace kzalloc + copy_from_user on memdup_user - v2: - Clear forgotten done label After merging the char-misc tree, today's linux-next build (x86_64_allmodconfig) produced this warning: drivers/misc/mic/vop/vop_vringh.c: In function 'vop_ioctl': drivers/misc/mic/vop/vop_vringh.c:1001:1: warning: label 'done' defined but not used [-Wunused-label] Signed-off-by: Vasyl Gomonovych <gomonovych@gmail.com> --- drivers/misc/mic/vop/vop_vringh.c | 22 +++++++--------------- 1 file changed, 7 insertions(+), 15 deletions(-) diff --git a/drivers/misc/mic/vop/vop_vringh.c b/drivers/misc/mic/vop/vop_vringh.c index fed992e..27db64e 100644 --- a/drivers/misc/mic/vop/vop_vringh.c +++ b/drivers/misc/mic/vop/vop_vringh.c @@ -937,13 +937,10 @@ static long vop_ioctl(struct file *f, unsigned int cmd, unsigned long arg) dd.num_vq > MIC_MAX_VRINGS) return -EINVAL; - dd_config = kzalloc(mic_desc_size(&dd), GFP_KERNEL); - if (!dd_config) - return -ENOMEM; - if (copy_from_user(dd_config, argp, mic_desc_size(&dd))) { - ret = -EFAULT; - goto free_ret; - } + dd_config = memdup_user(argp, mic_desc_size(&dd)); + if (IS_ERR(dd_config)) + return PTR_ERR(dd_config); + /* Ensure desc has not changed between the two reads */ if (memcmp(&dd, dd_config, sizeof(dd))) { ret = -EINVAL; @@ -995,17 +992,12 @@ static long vop_ioctl(struct file *f, unsigned int cmd, unsigned long arg) ret = vop_vdev_inited(vdev); if (ret) goto __unlock_ret; - buf = kzalloc(vdev->dd->config_len, GFP_KERNEL); - if (!buf) { - ret = -ENOMEM; + buf = memdup_user(argp, vdev->dd->config_len); + if (IS_ERR(buf)) { + ret = PTR_ERR(buf); goto __unlock_ret; } - if (copy_from_user(buf, argp, vdev->dd->config_len)) { - ret = -EFAULT; - goto done; - } ret = vop_virtio_config_change(vdev, buf); -done: kfree(buf); __unlock_ret: mutex_unlock(&vdev->vdev_mutex); -- 1.9.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2] misc: mic: Use memdup_user() as a cleanup 2017-12-12 10:40 ` [PATCH v2] misc: mic: Use memdup_user() as a cleanup Vasyl Gomonovych @ 2017-12-12 11:49 ` Greg KH 2017-12-12 13:22 ` Gomonovych, Vasyl 0 siblings, 1 reply; 8+ messages in thread From: Greg KH @ 2017-12-12 11:49 UTC (permalink / raw) To: Vasyl Gomonovych Cc: sfr, arnd, sudeep.dutt, ashutosh.dixit, dan.carpenter, linux-kernel, linux-next On Tue, Dec 12, 2017 at 11:40:58AM +0100, Vasyl Gomonovych wrote: > Fix coccicheck warning which recommends to use memdup_user(): > > drivers/misc/mic/vop/vop_vringh.c:940:14-21: WARNING opportunity for memdup_user > drivers/misc/mic/vop/vop_vringh.c:998:8-15: WARNING opportunity for memdup_user > > Generated by: scripts/coccinelle/memdup_user/memdup_user.cocci > > Changelog: > - v1: > - Replace kzalloc + copy_from_user on memdup_user > - v2: > - Clear forgotten done label > After merging the char-misc tree, today's linux-next build > (x86_64_allmodconfig) produced this warning: > drivers/misc/mic/vop/vop_vringh.c: In function 'vop_ioctl': > drivers/misc/mic/vop/vop_vringh.c:1001:1: warning: label 'done' defined but not used [-Wunused-label] Will not work as I already have taken v1. Also, put the changelog below the --- line please. thanks, greg k-h ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] misc: mic: Use memdup_user() as a cleanup 2017-12-12 11:49 ` Greg KH @ 2017-12-12 13:22 ` Gomonovych, Vasyl 2017-12-13 9:30 ` Greg KH 0 siblings, 1 reply; 8+ messages in thread From: Gomonovych, Vasyl @ 2017-12-12 13:22 UTC (permalink / raw) To: Greg KH Cc: Stephen Rothwell, Arnd Bergmann, sudeep.dutt, ashutosh.dixit, dan.carpenter, linux-kernel, linux-next Hi, Thanks. Should I prepare fix patch only for missed label? Regards Vasyl. On Tue, Dec 12, 2017 at 12:49 PM, Greg KH <greg@kroah.com> wrote: > On Tue, Dec 12, 2017 at 11:40:58AM +0100, Vasyl Gomonovych wrote: >> Fix coccicheck warning which recommends to use memdup_user(): >> >> drivers/misc/mic/vop/vop_vringh.c:940:14-21: WARNING opportunity for memdup_user >> drivers/misc/mic/vop/vop_vringh.c:998:8-15: WARNING opportunity for memdup_user >> >> Generated by: scripts/coccinelle/memdup_user/memdup_user.cocci >> >> Changelog: >> - v1: >> - Replace kzalloc + copy_from_user on memdup_user >> - v2: >> - Clear forgotten done label >> After merging the char-misc tree, today's linux-next build >> (x86_64_allmodconfig) produced this warning: >> drivers/misc/mic/vop/vop_vringh.c: In function 'vop_ioctl': >> drivers/misc/mic/vop/vop_vringh.c:1001:1: warning: label 'done' defined but not used [-Wunused-label] > > Will not work as I already have taken v1. > > Also, put the changelog below the --- line please. > > thanks, > > greg k-h -- Доброї вам пори дня. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] misc: mic: Use memdup_user() as a cleanup 2017-12-12 13:22 ` Gomonovych, Vasyl @ 2017-12-13 9:30 ` Greg KH 2017-12-13 22:01 ` [PATCH v3] " Vasyl Gomonovych 0 siblings, 1 reply; 8+ messages in thread From: Greg KH @ 2017-12-13 9:30 UTC (permalink / raw) To: Gomonovych, Vasyl Cc: Stephen Rothwell, Arnd Bergmann, sudeep.dutt, ashutosh.dixit, dan.carpenter, linux-kernel, linux-next On Tue, Dec 12, 2017 at 02:22:41PM +0100, Gomonovych, Vasyl wrote: > Hi, > Thanks. > Should I prepare fix patch only for missed label? I have no context here at all, sorry... ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v3] misc: mic: Use memdup_user() as a cleanup 2017-12-13 9:30 ` Greg KH @ 2017-12-13 22:01 ` Vasyl Gomonovych 2017-12-19 9:17 ` Greg KH 0 siblings, 1 reply; 8+ messages in thread From: Vasyl Gomonovych @ 2017-12-13 22:01 UTC (permalink / raw) To: gregkh, sfr, arnd, sudeep.dutt, ashutosh.dixit, dan.carpenter Cc: Vasyl Gomonovych, linux-kernel Fix coccicheck warning which recommends to use memdup_user(): drivers/misc/mic/vop/vop_vringh.c:940:14-21: WARNING opportunity for memdup_user drivers/misc/mic/vop/vop_vringh.c:998:8-15: WARNING opportunity for memdup_user Generated by: scripts/coccinelle/memdup_user/memdup_user.cocci Signed-off-by: Vasyl Gomonovych <gomonovych@gmail.com> --- I'm not sure if this still valid, maybe warning was fixed already. In that case sorry for this disturbing noise. Changelog: - v1: - Replace kzalloc + copy_from_user by memdup_user - v2: - Sorry for this disturb - Fix warning: label done defined but not used Introduced by previous commit misc: mic: Use memdup_user() as a cleanup drivers/misc/mic/vop/vop_vringh.c:1001:1: warning: label 'done' defined but not used [-Wunused-label] drivers/misc/mic/vop/vop_vringh.c | 22 +++++++--------------- 1 file changed, 7 insertions(+), 15 deletions(-) diff --git a/drivers/misc/mic/vop/vop_vringh.c b/drivers/misc/mic/vop/vop_vringh.c index fed992e2c258..27db64ec9efe 100644 --- a/drivers/misc/mic/vop/vop_vringh.c +++ b/drivers/misc/mic/vop/vop_vringh.c @@ -937,13 +937,10 @@ static long vop_ioctl(struct file *f, unsigned int cmd, unsigned long arg) dd.num_vq > MIC_MAX_VRINGS) return -EINVAL; - dd_config = kzalloc(mic_desc_size(&dd), GFP_KERNEL); - if (!dd_config) - return -ENOMEM; - if (copy_from_user(dd_config, argp, mic_desc_size(&dd))) { - ret = -EFAULT; - goto free_ret; - } + dd_config = memdup_user(argp, mic_desc_size(&dd)); + if (IS_ERR(dd_config)) + return PTR_ERR(dd_config); + /* Ensure desc has not changed between the two reads */ if (memcmp(&dd, dd_config, sizeof(dd))) { ret = -EINVAL; @@ -995,17 +992,12 @@ static long vop_ioctl(struct file *f, unsigned int cmd, unsigned long arg) ret = vop_vdev_inited(vdev); if (ret) goto __unlock_ret; - buf = kzalloc(vdev->dd->config_len, GFP_KERNEL); - if (!buf) { - ret = -ENOMEM; + buf = memdup_user(argp, vdev->dd->config_len); + if (IS_ERR(buf)) { + ret = PTR_ERR(buf); goto __unlock_ret; } - if (copy_from_user(buf, argp, vdev->dd->config_len)) { - ret = -EFAULT; - goto done; - } ret = vop_virtio_config_change(vdev, buf); -done: kfree(buf); __unlock_ret: mutex_unlock(&vdev->vdev_mutex); -- 1.9.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v3] misc: mic: Use memdup_user() as a cleanup 2017-12-13 22:01 ` [PATCH v3] " Vasyl Gomonovych @ 2017-12-19 9:17 ` Greg KH 0 siblings, 0 replies; 8+ messages in thread From: Greg KH @ 2017-12-19 9:17 UTC (permalink / raw) To: Vasyl Gomonovych Cc: sfr, arnd, sudeep.dutt, ashutosh.dixit, dan.carpenter, linux-kernel On Wed, Dec 13, 2017 at 11:01:08PM +0100, Vasyl Gomonovych wrote: > Fix coccicheck warning which recommends to use memdup_user(): > > drivers/misc/mic/vop/vop_vringh.c:940:14-21: WARNING opportunity for memdup_user > drivers/misc/mic/vop/vop_vringh.c:998:8-15: WARNING opportunity for memdup_user > Generated by: scripts/coccinelle/memdup_user/memdup_user.cocci > > Signed-off-by: Vasyl Gomonovych <gomonovych@gmail.com> > --- > I'm not sure if this still valid, maybe warning was fixed already. > In that case sorry for this disturbing noise. Seems to already be in the tree. thanks, greg k-h ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: linux-next: build warning after merge of the char-misc tree 2017-12-12 3:39 linux-next: build warning after merge of the char-misc tree Stephen Rothwell 2017-12-12 10:40 ` [PATCH v2] misc: mic: Use memdup_user() as a cleanup Vasyl Gomonovych @ 2017-12-12 11:49 ` Greg KH 1 sibling, 0 replies; 8+ messages in thread From: Greg KH @ 2017-12-12 11:49 UTC (permalink / raw) To: Stephen Rothwell Cc: Arnd Bergmann, Linux-Next Mailing List, Linux Kernel Mailing List, Vasyl Gomonovych On Tue, Dec 12, 2017 at 02:39:10PM +1100, Stephen Rothwell wrote: > Hi all, > > After merging the char-misc tree, today's linux-next build > (x86_64_allmodconfig) produced this warning: > > drivers/misc/mic/vop/vop_vringh.c: In function 'vop_ioctl': > drivers/misc/mic/vop/vop_vringh.c:1001:1: warning: label 'done' defined but not used [-Wunused-label] > done: > ^ > > Introduced by commit > > 30b7a2c19e29 ("misc: mic: Use memdup_user() as a cleanup") Thanks, Colin sent a patch for this, will queue it up later today. greg k-h ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-12-19 9:17 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-12-12 3:39 linux-next: build warning after merge of the char-misc tree Stephen Rothwell 2017-12-12 10:40 ` [PATCH v2] misc: mic: Use memdup_user() as a cleanup Vasyl Gomonovych 2017-12-12 11:49 ` Greg KH 2017-12-12 13:22 ` Gomonovych, Vasyl 2017-12-13 9:30 ` Greg KH 2017-12-13 22:01 ` [PATCH v3] " Vasyl Gomonovych 2017-12-19 9:17 ` Greg KH 2017-12-12 11:49 ` linux-next: build warning after merge of the char-misc tree Greg KH
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.