android/android_kernel_samsung_sm8650
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

gpio: sim: fix memory corruption when adding named lines and unnamed hogs

Browse Source 
[ Upstream commit 95ae9979bfe3174c2ee8d64409c44532f2881907 ]

When constructing the sim, gpio-sim constructs an array of named lines,
sized based on the largest offset of any named line, and then initializes
that array with the names of all lines, including unnamed hogs with higher
offsets.  In doing so it writes NULLs beyond the extent of the array.

Add a check that only named lines are used to initialize the array.

Fixes: cb8c474e79 ("gpio: sim: new testing module")
Signed-off-by: Kent Gibson<warthog618@gmail.com>
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Kent Gibson 2023-06-06 20:00:34 +08:00 committed by Greg Kroah-Hartman
parent 4106894328
commit c53f2e8462
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/gpio/gpio-sim.c
View File

@ -717,8 +717,10 @@ static char **gpio_sim_make_line_names(struct gpio_sim_bank *bank,
if (!line_names) if (!line_names)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
list_for_each_entry(line, &bank->line_list, siblings) list_for_each_entry(line, &bank->line_list, siblings) {
line_names[line->offset] = line->name; if (line->name && (line->offset <= max_offset))
line_names[line->offset] = line->name;
}
return line_names; return line_names;
} }