mirrors/pulseaudio
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git synced 2025-11-02 09:01:46 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

bluetooth: Fix unregistering of the message handler

Browse source 
In cases, where codec initialisation may fail, we still try
unregistering the message handler. Do not try unregistering
the handler if it was not registered in the first place.

Part-of: <https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/483>
This commit is contained in:
Sanchayan Maity 2021-01-21 18:41:25 +05:30
parent 3a4038f3ff
commit 8816b6b05b
1 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/modules/bluetooth/module-bluez5-device.c
View file

@ -145,6 +145,8 @@ struct userdata {
pa_sample_spec decoder_sample_spec;
void *decoder_buffer; /* Codec transfer buffer */
size_t decoder_buffer_size; /* Size of the buffer */
bool message_handler_registered;
};
typedef enum pa_bluetooth_form_factor {
@ -2492,6 +2494,7 @@ int pa__init(pa_module* m) {
m->userdata = u = pa_xnew0(struct userdata, 1);
u->module = m;
u->core = m->core;
u->message_handler_registered = false;
if (!(ma = pa_modargs_new(m->argument, valid_modargs))) {
pa_log_error("Failed to parse module arguments");
@ -2562,6 +2565,7 @@ int pa__init(pa_module* m) {
bluez5_device_message_handler, (void *) u);
pa_log_info("Bluez5 device message handler registered at path: %s", message_handler_path);
pa_xfree(message_handler_path);
u->message_handler_registered = true;
return 0;
@ -2593,9 +2597,11 @@ void pa__done(pa_module *m) {
if (!(u = m->userdata))
return;
message_handler_path = make_message_handler_path(u->card->name);
pa_message_handler_unregister(m->core, message_handler_path);
pa_xfree(message_handler_path);
if (u->message_handler_registered) {
message_handler_path = make_message_handler_path(u->card->name);
pa_message_handler_unregister(m->core, message_handler_path);
pa_xfree(message_handler_path);
}
stop_thread(u);