diff options
| author | Dan Carpenter <[email protected]> | 2025-04-12 10:15:00 +0000 |
|---|---|---|
| committer | Bartosz Golaszewski <[email protected]> | 2025-04-14 20:25:20 +0000 |
| commit | eebfcb98cdc0228f5e1b7407f9db1c602bd8e545 (patch) | |
| tree | 86e4e1462512d44954b1bf1f7abfbbbfacd67cd2 /drivers/gpio/gpio-aggregator.c | |
| parent | selftests: gpio: add test cases for gpio-aggregator (diff) | |
| download | kernel-eebfcb98cdc0228f5e1b7407f9db1c602bd8e545.tar.gz kernel-eebfcb98cdc0228f5e1b7407f9db1c602bd8e545.zip | |
gpio: aggregator: fix "_sysfs" prefix check in gpio_aggregator_make_group()
This code is intended to reject strings that start with "_sysfs" but the
strcmp() limit is wrong so checks the whole string instead of the prefix.
Fixes: 83c8e3df642f ("gpio: aggregator: expose aggregator created via legacy sysfs to configfs")
Signed-off-by: Dan Carpenter <[email protected]>
Reviewed-by: Geert Uytterhoeven <[email protected]>
Acked-by: Koichiro Den <[email protected]>
Link: https://lore.kernel.org/r/30210ed77b40b4b6629de659cb56b9ec7832c447.1744452787.git.dan.carpenter@linaro.org
Signed-off-by: Bartosz Golaszewski <[email protected]>
Diffstat (limited to 'drivers/gpio/gpio-aggregator.c')
| -rw-r--r-- | drivers/gpio/gpio-aggregator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-aggregator.c b/drivers/gpio/gpio-aggregator.c index dde969f29ee2..b4c9e373a6ec 100644 --- a/drivers/gpio/gpio-aggregator.c +++ b/drivers/gpio/gpio-aggregator.c @@ -1016,7 +1016,7 @@ gpio_aggregator_make_group(struct config_group *group, const char *name) * for devices create via legacy sysfs interface. */ if (strncmp(name, AGGREGATOR_LEGACY_PREFIX, - sizeof(AGGREGATOR_LEGACY_PREFIX)) == 0) + sizeof(AGGREGATOR_LEGACY_PREFIX) - 1) == 0) return ERR_PTR(-EINVAL); /* arg space is unneeded */ |