Hi Luiz,
On Tue, 22. Oct 14:54, Dmitry Antipov wrote:
> Add missing call to 'mgmt_pending_remove()' on 'hci_cmd_sync_queue()'
> error handling path in '__add_adv_patterns_monitor()'. Compile tested
> only.
>
> Fixes: b747a83690c8 ("Bluetooth: hci_sync: Refactor add Adv Monitor")
> Signed-off-by: Dmitry Antipov <dmantipov@xxxxxxxxx>
> ---
Would you consider the patch, please?
It looks reasonable and is still applicable to the mainline AFAICS.
Thanks!
> net/bluetooth/mgmt.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
> index a429661b676a..92c9808e419b 100644
> --- a/net/bluetooth/mgmt.c
> +++ b/net/bluetooth/mgmt.c
> @@ -5301,6 +5301,7 @@ static int __add_adv_patterns_monitor(struct sock *sk, struct hci_dev *hdev,
> else
> status = MGMT_STATUS_FAILED;
>
> + mgmt_pending_remove(cmd);
> goto unlock;
> }
>
> --
> 2.47.0
