On 13.04.2017 07:22, Andrew Hlynskyi wrote:
A bluetooth transport will be garbage collected automatically
by the pa_bluetooth_discovery_unref function.
---
  src/modules/bluetooth/backend-ofono.c | 3 ---
  1 file changed, 3 deletions(-)

diff --git a/src/modules/bluetooth/backend-ofono.c 
b/src/modules/bluetooth/backend-ofono.c
index 755df9e8..5fd7e4ce 100644
--- a/src/modules/bluetooth/backend-ofono.c
+++ b/src/modules/bluetooth/backend-ofono.c
@@ -111,9 +111,6 @@ static struct hf_audio_card 
*hf_audio_card_new(pa_bluetooth_backend *backend, co
  static void hf_audio_card_free(struct hf_audio_card *card) {
      pa_assert(card);
- if (card->transport)
-        pa_bluetooth_transport_free(card->transport);
-
      pa_xfree(card->path);
      pa_xfree(card->remote_address);
      pa_xfree(card->local_address);

This one does not look correct because the pa_bluetooth_discovery_unref() function is only called when the modules are unloaded. What exactly happens when you are getting a segfault? The transport itself must be valid, otherwise the function would not
get called.

Regards
             Georg

_______________________________________________
pulseaudio-discuss mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to