Messages in this thread | | | Date | Tue, 22 Nov 2016 11:44:34 +0100 (CET) | From | Jiri Kosina <> | Subject | Re: [PATCH] HID: lg: fix noderef.cocci warnings |
| |
On Mon, 21 Nov 2016, Benjamin Tissoires wrote:
> > Generated by: scripts/coccinelle/misc/noderef.cocci > > > > CC: Benjamin Tissoires <benjamin.tissoires@redhat.com> > > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com> > > --- > > > > hid-lg.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > --- a/drivers/hid/hid-lg.c > > +++ b/drivers/hid/hid-lg.c > > @@ -777,8 +777,10 @@ static int lg_probe(struct hid_device *h > > buf[1] = 0xB2; > > get_random_bytes(&buf[2], 2); > > > > - ret = hid_hw_raw_request(hdev, buf[0], buf, sizeof(buf), > > - HID_FEATURE_REPORT, HID_REQ_SET_REPORT); > > + ret = hid_hw_raw_request(hdev, buf[0], buf, > > + sizeof(*buf), > > This is wrong. I messed up and should have used "sizeof(cbuf)", but the > coccinelle script failed at detecting the correct solution (I guess it > couldn't).
Fengguang, is there anything that could be done to improve this?
> Jiri, do you want me to send a v2 of the series or will you just amend > the patch while applying?
I'll fix that up, no worries. Thanks,
-- Jiri Kosina SUSE Labs
| |