power: supply: lp8788: prevent out of bounds array access
commitbdd9968d35
upstream. val might become 7 in which case stime[7] (array of length 7) would be accessed during the scnprintf call later and that will cause issues. Obviously, string concatenation is not intended here so just a comma needs to be added to fix the issue. Fixes:98a2766493
("power_supply: Add new lp8788 charger driver") Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@gmail.com> Acked-by: Milo Kim <milo.kim@ti.com> Signed-off-by: Sebastian Reichel <sre@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
ccef31d22e
commit
4f39fcce99
|
@ -654,7 +654,7 @@ static ssize_t lp8788_show_eoc_time(struct device *dev,
|
|||
{
|
||||
struct lp8788_charger *pchg = dev_get_drvdata(dev);
|
||||
char *stime[] = { "400ms", "5min", "10min", "15min",
|
||||
"20min", "25min", "30min" "No timeout" };
|
||||
"20min", "25min", "30min", "No timeout" };
|
||||
u8 val;
|
||||
|
||||
lp8788_read_byte(pchg->lp, LP8788_CHG_EOC, &val);
|
||||
|
|
Loading…
Reference in a new issue