| Message ID | 20220825203903.3526-1-ceggers@arri.de |
|---|---|
| State | New |
| Headers | show |
| Series | client: Add missing return statement in error path | expand |
Hello: This patch was applied to bluetooth/bluez.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Thu, 25 Aug 2022 22:39:03 +0200 you wrote: > Client will SEGFAULT if default_ctrl==NULL > > Fixes: 936ad5ef7b56 ("client: Quit when done with command") > Signed-off-by: Christian Eggers <ceggers@arri.de> > --- > client/main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Here is the summary with links: - client: Add missing return statement in error path https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=9b93d958845b You are awesome, thank you!
diff --git a/client/main.c b/client/main.c index 88c7ad799888..754157b169c6 100644 --- a/client/main.c +++ b/client/main.c @@ -2546,7 +2546,7 @@ static void cmd_advertise(int argc, char *argv[]) if (!default_ctrl || !default_ctrl->ad_proxy) { bt_shell_printf("LEAdvertisingManager not found\n"); - bt_shell_noninteractive_quit(EXIT_FAILURE); + return bt_shell_noninteractive_quit(EXIT_FAILURE); } if (enable == TRUE)
Client will SEGFAULT if default_ctrl==NULL Fixes: 936ad5ef7b56 ("client: Quit when done with command") Signed-off-by: Christian Eggers <ceggers@arri.de> --- client/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)