diff mbox series

tools/btmgmt: add missing return statement

Message ID 20220825203848.3499-1-ceggers@arri.de
State New
Headers show
Series tools/btmgmt: add missing return statement | expand

Commit Message

Christian Eggers Aug. 25, 2022, 8:38 p.m. UTC 
Leave function on error instead of printing (possibly) invalid flags.

Fixes: d70618e49461 ("tools/btmgmt: Add device flags commands")
Signed-off-by: Christian Eggers <ceggers@arri.de>
---
 tools/btmgmt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

patchwork-bot+bluetooth@kernel.org Aug. 29, 2022, 8:10 p.m. UTC | #1 
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:38:48 +0200 you wrote:
> Leave function on error instead of printing (possibly) invalid flags.
> 
> Fixes: d70618e49461 ("tools/btmgmt: Add device flags commands")
> Signed-off-by: Christian Eggers <ceggers@arri.de>
> ---
>  tools/btmgmt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Here is the summary with links:
  - tools/btmgmt: add missing return statement
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=150bbff449c7

You are awesome, thank you!
diff mbox series

Patch

diff --git a/tools/btmgmt.c b/tools/btmgmt.c
index 42ef9acefaea..132ebb5f5b44 100644
--- a/tools/btmgmt.c
+++ b/tools/btmgmt.c
@@ -2183,7 +2183,7 @@  static void get_flags_rsp(uint8_t status, uint16_t len, const void *param,
 	if (status != 0) {
 		error("Get device flags failed with status 0x%02x (%s)",
 						status, mgmt_errstr(status));
-		bt_shell_noninteractive_quit(EXIT_FAILURE);
+		return bt_shell_noninteractive_quit(EXIT_FAILURE);
 	}
 
 	print("Supported Flags: 0x%08x", rp->supported_flags);