[lvc-project] [PATCH] Bluetooth: fix error handling path in __add_adv_patterns_monitor()

[Fedor Pchelkin]

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 at yandex.ru>
> ---

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