michaelh/zephyr
1
0
Fork 0
Code Releases Activity

bluetooth: audio: Fix logical check

Browse source 
I imagine that expression (req != 0 || req != 1) is always true. Fix
it changing '||' to '&&'.

Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
This commit is contained in:
Andrei Emeltchenko 2023-05-10 16:55:43 +03:00 • committed by Carles Cufí
commit a4341bbc2c
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
subsys/bluetooth/audio/shell/bap_broadcast_assistant.c
View file

@ -801,7 +801,7 @@ static int cmd_bap_broadcast_assistant_add_pa_sync(const struct shell *sh,
shell_error(sh, "failed to parse pa_sync: %d", err);
return -ENOEXEC;
} else if (pa_sync_req != 0U || pa_sync_req != 1U) {
} else if (pa_sync_req != 0U && pa_sync_req != 1U) {
shell_error(sh, "pa_sync_req shall be boolean: %lu", pa_sync_req);
return -ENOEXEC;