[pulseaudio-discuss] [PATCH 1/2] bluetooth: fix possible segfault in bluetooth backend-native.c module

Georg Chini georg at chini.tk
Thu Apr 13 09:41:10 UTC 2017


On 13.04.2017 07:22, Andrew Hlynskyi wrote:
> The pa_bluetooth_transport_unlink function already
> called by the pa_bluetooth_transport_free function.
> ---
>   src/modules/bluetooth/backend-native.c | 1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/src/modules/bluetooth/backend-native.c b/src/modules/bluetooth/backend-native.c
> index 6eb4e168..cbafaff6 100644
> --- a/src/modules/bluetooth/backend-native.c
> +++ b/src/modules/bluetooth/backend-native.c
> @@ -415,7 +415,6 @@ static void rfcomm_io_callback(pa_mainloop_api *io, pa_io_event *e, int fd, pa_i
>       return;
>   
>   fail:
> -    pa_bluetooth_transport_unlink(t);
>       pa_bluetooth_transport_free(t);
>   }
>   

mh, the patch is correct, but I do not see how it prevents a segfault. 
What exactly happens?



More information about the pulseaudio-discuss mailing list