diff mbox series

[BlueZ,v1,2/5] shared/shell: prevent integer overflow in bt_shell_init()

Message ID 20240704180752.94264-3-r.smirnov@omp.ru
State New
Headers show
Series fix errors found by SVACE static analyzer #2 | expand

Commit Message

Roman Smirnov July 4, 2024, 6:07 p.m. UTC 
An integer overflow will occur if index < offest. It is necessary
to prevent this case.

Found with the SVACE static analysis tool.
---
 src/shared/shell.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/src/shared/shell.c b/src/shared/shell.c
index add4fa131..ea0985815 100644
--- a/src/shared/shell.c
+++ b/src/shared/shell.c
@@ -1333,7 +1333,7 @@  void bt_shell_init(int argc, char **argv, const struct bt_shell_opt *opt)
 				}
 			}
 
-			if (opt) {
+			if (opt && index >= offset) {
 				if (c != opt->options[index - offset].val) {
 					usage(argc, argv, opt);
 					exit(EXIT_SUCCESS);