On 10/02/2014, 02:05 PM, Michael Ikey Doherty wrote: > From: Jiri Kosina <jkosina@xxxxxxx> > > The report passed to us from transport driver could potentially be > arbitrarily large, therefore we better sanity-check it so that > magicmouse_emit_touch() gets only valid values of raw_id. > > Cc: stable@xxxxxxxxxxxxxxx > Reported-by: Steven Vittitoe <scvitti@xxxxxxxxxx> > Signed-off-by: Jiri Kosina <jkosina@xxxxxxx> > --- > drivers/hid/hid-picolcd_core.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/hid/hid-picolcd_core.c b/drivers/hid/hid-picolcd_core.c > index acbb021..8aeba62 100644 > --- a/drivers/hid/hid-picolcd_core.c > +++ b/drivers/hid/hid-picolcd_core.c > @@ -350,6 +350,12 @@ static int picolcd_raw_event(struct hid_device *hdev, > if (!data) > return 1; > > + if (size > 64) { > + hid_warn(hdev, "invalid size value (%d) for picolcd raw event\n", > + size); > + return 0; > + } > + > if (report->id == REPORT_KEY_STATE) { > if (data->input_keys) > ret = picolcd_raw_keypad(data, report, raw_data+1, size-1); > Hi, what is this, please? Do you want it applied to stable or somewhere? -- js suse labs -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html