From b80e4f58f6bb82fa0de3c891fb15a54eb33766c1 Mon Sep 17 00:00:00 2001 From: Steve Brown Date: Fri, 22 Dec 2017 05:29:54 -0500 Subject: [PATCH] mesh: meshctl: Correct display security The case of a security command without an argument is not handled and throws a segv. --- mesh/main.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/mesh/main.c b/mesh/main.c index 14298b089..b5ec4bcd5 100644 --- a/mesh/main.c +++ b/mesh/main.c @@ -1669,14 +1669,19 @@ static void cmd_security(int argc, char *argv[]) uint8_t level; char *end; + if (argc == 1) + goto done; + level = strtol(argv[1], &end, 10); if (end == argv[1] || !prov_set_sec_level(level)) { bt_shell_printf("Invalid security level %s\n", argv[1]); return; } - bt_shell_printf("Provision Security Level set to %u (%s)\n", level, - security2str(level)); +done: + bt_shell_printf("Provision Security Level set to %u (%s)\n", + prov_get_sec_level(), + security2str(prov_get_sec_level())); } static void cmd_connect(int argc, char *argv[]) -- 2.47.3