diff options
| author | Zev Weiss <[email protected]> | 2023-02-01 18:18:25 -0800 |
|---|---|---|
| committer | Guenter Roeck <[email protected]> | 2023-02-03 07:27:25 -0800 |
| commit | f00093608fa790580da309bb9feb5108fbe7c331 (patch) | |
| tree | 462d19b64dfe38d71caf22ce20341624f16a424d | |
| parent | 2fbb848b65cde5b876cce52ebcb34de4aaa5a94a (diff) | |
hwmon: (peci/cputemp) Fix off-by-one in coretemp_label allocation
The find_last_bit() call produces the index of the highest-numbered
core in core_mask; because cores are numbered from zero, the number of
elements we need to allocate is one more than that.
Signed-off-by: Zev Weiss <[email protected]>
Cc: [email protected] # v5.18
Fixes: bf3608f338e9 ("hwmon: peci: Add cputemp driver")
Reviewed-by: Iwona Winiarska <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Guenter Roeck <[email protected]>
| -rw-r--r-- | drivers/hwmon/peci/cputemp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/peci/cputemp.c b/drivers/hwmon/peci/cputemp.c index 57470fda5f6c..30850a479f61 100644 --- a/drivers/hwmon/peci/cputemp.c +++ b/drivers/hwmon/peci/cputemp.c @@ -402,7 +402,7 @@ static int create_temp_label(struct peci_cputemp *priv) unsigned long core_max = find_last_bit(priv->core_mask, CORE_NUMS_MAX); int i; - priv->coretemp_label = devm_kzalloc(priv->dev, core_max * sizeof(char *), GFP_KERNEL); + priv->coretemp_label = devm_kzalloc(priv->dev, (core_max + 1) * sizeof(char *), GFP_KERNEL); if (!priv->coretemp_label) return -ENOMEM; |