Commit 3237da12 authored by Roman Smirnov's avatar Roman Smirnov Committed by Luiz Augusto von Dentz

Bluetooth: mgmt: remove NULL check in mgmt_set_connectable_complete()

Remove the cmd pointer NULL check in mgmt_set_connectable_complete()
because it occurs earlier in set_connectable(). This check is also
unnecessary because the pointer is dereferenced just before it.

Found by Linux Verification Center (linuxtesting.org) with Svace.
Signed-off-by: default avatarRoman Smirnov <r.smirnov@omp.ru>
Reviewed-by: default avatarSergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: default avatarLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
parent 1cb63d80
...@@ -1707,7 +1707,6 @@ static void mgmt_set_connectable_complete(struct hci_dev *hdev, void *data, ...@@ -1707,7 +1707,6 @@ static void mgmt_set_connectable_complete(struct hci_dev *hdev, void *data,
new_settings(hdev, cmd->sk); new_settings(hdev, cmd->sk);
done: done:
if (cmd)
mgmt_pending_remove(cmd); mgmt_pending_remove(cmd);
hci_dev_unlock(hdev); hci_dev_unlock(hdev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment