HID: hid-sensor-custom: set fixed size for custom attributes
This is no bugfix (so no Fixes: tag is necessary) as it is taken care of in hid_sensor_custom_add_attributes(). The motivation for this patch is that: hid_sensor_custom_field.attr_name and hid_sensor_custom_field.attrs has the size of HID_CUSTOM_TOTAL_ATTRS and used in same context. We compare against HID_CUSTOM_TOTAL_ATTRS when looping through hid_custom_attrs. We will silent the smatch error: hid_sensor_custom_add_attributes() error: buffer overflow 'hid_custom_attrs' 8 <= 10 Signed-off-by: Marcus Folkesson <marcus.folkesson@gmail.com> Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
This commit is contained in:
parent
9861a25fc2
commit
9d013910df
@ -62,7 +62,7 @@ struct hid_sensor_sample {
|
|||||||
u32 raw_len;
|
u32 raw_len;
|
||||||
} __packed;
|
} __packed;
|
||||||
|
|
||||||
static struct attribute hid_custom_attrs[] = {
|
static struct attribute hid_custom_attrs[HID_CUSTOM_TOTAL_ATTRS] = {
|
||||||
{.name = "name", .mode = S_IRUGO},
|
{.name = "name", .mode = S_IRUGO},
|
||||||
{.name = "units", .mode = S_IRUGO},
|
{.name = "units", .mode = S_IRUGO},
|
||||||
{.name = "unit-expo", .mode = S_IRUGO},
|
{.name = "unit-expo", .mode = S_IRUGO},
|
||||||
|
Loading…
Reference in New Issue
Block a user