* [PATCH 3/10] iio: at91_adc: Cocci spatch "noderef" [not found] <1379604755850-858421494-0-diffsplit-thomas@m3y3r.de> @ 2013-09-19 21:42 ` Thomas Meyer [not found] ` <1379604755852-604229709-3-diffsplit-thomas-VsYtu1Qij5c@public.gmane.org> 0 siblings, 1 reply; 5+ messages in thread From: Thomas Meyer @ 2013-09-19 21:42 UTC (permalink / raw) To: jic23, grant.likely, rob.herring, linux-iio, linux-kernel, devicetree sizeof when applied to a pointer typed expression gives the size of the pointer. Found by coccinelle spatch "misc/noderef.cocci" Signed-off-by: Thomas Meyer <thomas@m3y3r.de> --- diff -u -p a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c --- a/drivers/iio/adc/at91_adc.c +++ b/drivers/iio/adc/at91_adc.c @@ -279,7 +279,7 @@ static int at91_adc_trigger_init(struct int i, ret; st->trig = devm_kzalloc(&idev->dev, - st->trigger_number * sizeof(st->trig), + st->trigger_number * sizeof(*st->trig), GFP_KERNEL); if (st->trig == NULL) { ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <1379604755852-604229709-3-diffsplit-thomas-VsYtu1Qij5c@public.gmane.org>]
* Re: [PATCH 3/10] iio: at91_adc: Cocci spatch "noderef" [not found] ` <1379604755852-604229709-3-diffsplit-thomas-VsYtu1Qij5c@public.gmane.org> @ 2013-09-20 7:26 ` Nicolas Ferre [not found] ` <523BF89E.4000508-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org> 0 siblings, 1 reply; 5+ messages in thread From: Nicolas Ferre @ 2013-09-20 7:26 UTC (permalink / raw) To: Thomas Meyer, jic23-KWPb1pKIrIJaa/9Udqfwiw, grant.likely-QSEj5FYQhm4dnm+yROfE0A, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, linux-iio-u79uwXL29TY76Z2rM5mHXA Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA On 19/09/2013 23:42, Thomas Meyer : > sizeof when applied to a pointer typed expression gives the size of the > pointer. > Found by coccinelle spatch "misc/noderef.cocci" > > Signed-off-by: Thomas Meyer <thomas-VsYtu1Qij5c@public.gmane.org> Acked-by: Nicolas Ferre <nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org> > --- > > diff -u -p a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c > --- a/drivers/iio/adc/at91_adc.c > +++ b/drivers/iio/adc/at91_adc.c > @@ -279,7 +279,7 @@ static int at91_adc_trigger_init(struct > int i, ret; > > st->trig = devm_kzalloc(&idev->dev, > - st->trigger_number * sizeof(st->trig), > + st->trigger_number * sizeof(*st->trig), > GFP_KERNEL); > > if (st->trig == NULL) { > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Nicolas Ferre ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <523BF89E.4000508-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>]
* Re: [PATCH 3/10] iio: at91_adc: Cocci spatch "noderef" [not found] ` <523BF89E.4000508-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org> @ 2013-09-20 18:36 ` Jonathan Cameron [not found] ` <523C95B0.6070906-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> 0 siblings, 1 reply; 5+ messages in thread From: Jonathan Cameron @ 2013-09-20 18:36 UTC (permalink / raw) To: Nicolas Ferre Cc: Thomas Meyer, jic23-KWPb1pKIrIJaa/9Udqfwiw, grant.likely-QSEj5FYQhm4dnm+yROfE0A, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, linux-iio-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA On 09/20/13 08:26, Nicolas Ferre wrote: > On 19/09/2013 23:42, Thomas Meyer : >> sizeof when applied to a pointer typed expression gives the size of the >> pointer. >> Found by coccinelle spatch "misc/noderef.cocci" >> >> Signed-off-by: Thomas Meyer <thomas-VsYtu1Qij5c@public.gmane.org> > > Acked-by: Nicolas Ferre <nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org> Applied to the fixes-togreg branch of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git Thanks, > >> --- >> >> diff -u -p a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c >> --- a/drivers/iio/adc/at91_adc.c >> +++ b/drivers/iio/adc/at91_adc.c >> @@ -279,7 +279,7 @@ static int at91_adc_trigger_init(struct >> int i, ret; >> >> st->trig = devm_kzalloc(&idev->dev, >> - st->trigger_number * sizeof(st->trig), >> + st->trigger_number * sizeof(*st->trig), >> GFP_KERNEL); >> >> if (st->trig == NULL) { >> -- >> To unsubscribe from this list: send the line "unsubscribe devicetree" in >> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > > ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <523C95B0.6070906-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]
* Re: [PATCH 3/10] iio: at91_adc: Cocci spatch "noderef" [not found] ` <523C95B0.6070906-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> @ 2013-09-20 18:42 ` Jonathan Cameron [not found] ` <523C971E.5090809-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> 0 siblings, 1 reply; 5+ messages in thread From: Jonathan Cameron @ 2013-09-20 18:42 UTC (permalink / raw) To: Nicolas Ferre Cc: Thomas Meyer, jic23-KWPb1pKIrIJaa/9Udqfwiw, grant.likely-QSEj5FYQhm4dnm+yROfE0A, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, linux-iio-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA On 09/20/13 19:36, Jonathan Cameron wrote: > On 09/20/13 08:26, Nicolas Ferre wrote: >> On 19/09/2013 23:42, Thomas Meyer : >>> sizeof when applied to a pointer typed expression gives the size of the >>> pointer. >>> Found by coccinelle spatch "misc/noderef.cocci" >>> >>> Signed-off-by: Thomas Meyer <thomas-VsYtu1Qij5c@public.gmane.org> >> >> Acked-by: Nicolas Ferre <nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org> > Applied to the fixes-togreg branch of > git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git Actually, change of plan. Applied to the togreg branch of that tree. Whilst technically more correct to have it as you say, we are dealing with the size of a struct iio_trig ** vs a struct iio_trig * so it isn't actually a bug, just a less than ideal bit of code ;) It was so obviously a fix I didn't initially check if it was a 'real' bug or not. oops. Jonathan > > Thanks, >> >>> --- >>> >>> diff -u -p a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c >>> --- a/drivers/iio/adc/at91_adc.c >>> +++ b/drivers/iio/adc/at91_adc.c >>> @@ -279,7 +279,7 @@ static int at91_adc_trigger_init(struct >>> int i, ret; >>> >>> st->trig = devm_kzalloc(&idev->dev, >>> - st->trigger_number * sizeof(st->trig), >>> + st->trigger_number * sizeof(*st->trig), >>> GFP_KERNEL); >>> >>> if (st->trig == NULL) { >>> -- >>> To unsubscribe from this list: send the line "unsubscribe devicetree" in >>> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >> >> ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <523C971E.5090809-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]
* Re: [PATCH 3/10] iio: at91_adc: Cocci spatch "noderef" [not found] ` <523C971E.5090809-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> @ 2013-09-23 12:16 ` Nicolas Ferre 0 siblings, 0 replies; 5+ messages in thread From: Nicolas Ferre @ 2013-09-23 12:16 UTC (permalink / raw) To: Jonathan Cameron Cc: Thomas Meyer, jic23-KWPb1pKIrIJaa/9Udqfwiw, grant.likely-QSEj5FYQhm4dnm+yROfE0A, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, linux-iio-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, devicetree-u79uwXL29TY76Z2rM5mHXA On 20/09/2013 20:42, Jonathan Cameron : > On 09/20/13 19:36, Jonathan Cameron wrote: >> On 09/20/13 08:26, Nicolas Ferre wrote: >>> On 19/09/2013 23:42, Thomas Meyer : >>>> sizeof when applied to a pointer typed expression gives the size of the >>>> pointer. >>>> Found by coccinelle spatch "misc/noderef.cocci" >>>> >>>> Signed-off-by: Thomas Meyer <thomas-VsYtu1Qij5c@public.gmane.org> >>> >>> Acked-by: Nicolas Ferre <nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org> >> Applied to the fixes-togreg branch of >> git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git > > Actually, change of plan. Applied to the togreg branch of that > tree. Whilst technically more correct to have it as you say, > we are dealing with the size of a struct iio_trig ** vs > a struct iio_trig * so it isn't actually a bug, just a > less than ideal bit of code ;) Absolutely. > It was so obviously a fix I didn't initially check if it > was a 'real' bug or not. oops. Yes, I took this path myself as well ;-) Bye, >>>> --- >>>> >>>> diff -u -p a/drivers/iio/adc/at91_adc.c b/drivers/iio/adc/at91_adc.c >>>> --- a/drivers/iio/adc/at91_adc.c >>>> +++ b/drivers/iio/adc/at91_adc.c >>>> @@ -279,7 +279,7 @@ static int at91_adc_trigger_init(struct >>>> int i, ret; >>>> >>>> st->trig = devm_kzalloc(&idev->dev, >>>> - st->trigger_number * sizeof(st->trig), >>>> + st->trigger_number * sizeof(*st->trig), >>>> GFP_KERNEL); >>>> >>>> if (st->trig == NULL) { >>>> -- >>>> To unsubscribe from this list: send the line "unsubscribe devicetree" in >>>> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >>>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>>> >>> >>> > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Nicolas Ferre -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-09-23 12:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1379604755850-858421494-0-diffsplit-thomas@m3y3r.de>
2013-09-19 21:42 ` [PATCH 3/10] iio: at91_adc: Cocci spatch "noderef" Thomas Meyer
[not found] ` <1379604755852-604229709-3-diffsplit-thomas-VsYtu1Qij5c@public.gmane.org>
2013-09-20 7:26 ` Nicolas Ferre
[not found] ` <523BF89E.4000508-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2013-09-20 18:36 ` Jonathan Cameron
[not found] ` <523C95B0.6070906-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2013-09-20 18:42 ` Jonathan Cameron
[not found] ` <523C971E.5090809-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2013-09-23 12:16 ` Nicolas Ferre
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).