Messages in this thread | | | Subject | Re: [RESEND][PATCH] hwmon: scmi: fix potential buffer overflow in scmi_hwmon_probe() | From | Guenter Roeck <> | Date | Wed, 15 Jul 2020 07:55:52 -0700 |
| |
On 7/15/20 6:00 AM, Sudeep Holla wrote: > On Wed, Jul 15, 2020 at 01:13:38PM +0100, Cristian Marussi wrote: >> SMATCH detected a potential buffer overflow in the manipulation of >> hwmon_attributes array inside the scmi_hwmon_probe function: >> >> drivers/hwmon/scmi-hwmon.c:226 >> scmi_hwmon_probe() error: buffer overflow 'hwmon_attributes' 6 <= 9 >> >> Fix it by statically declaring the size of the array as the maximum >> possible as defined by hwmon_max define. >> > > Makes sense to me, > > Reviewed-by: Sudeep Holla <sudeep.holla@arm.com> > > There may be other such instances. I am not sure if Guenter has ignored > them intentionally or just no one has fixed them so far.
I am not perfect. No, I have not intentionally ignored anything, and I don't recall seeing smatch reports (or this patch) before.
Guenter
| |