[v7,01/13] switch-on-port-available: Fix null pointer dereference

Submitted by Pali Rohár on Feb. 23, 2019, 9:45 a.m.

Details

Message ID 20190223094544.14999-2-pali.rohar@gmail.com
State Superseded
Headers show
Series "New API for Bluetooth A2DP codecs" ( rev: 39 38 37 36 35 34 33 32 31 30 29 28 27 ) in PulseAudio

Not browsing as part of any series.

Commit Message

Pali Rohár Feb. 23, 2019, 9:45 a.m.
Sometimes on pulseaudio startup card->active_profile is NULL.
---
 src/modules/module-switch-on-port-available.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/src/modules/module-switch-on-port-available.c b/src/modules/module-switch-on-port-available.c
index 2fa7e4d62..9f344d0dc 100644
--- a/src/modules/module-switch-on-port-available.c
+++ b/src/modules/module-switch-on-port-available.c
@@ -343,7 +343,7 @@  static pa_hook_result_t card_profile_available_hook_callback(pa_core *c, pa_card
     if (profile->available != PA_AVAILABLE_NO)
         return PA_HOOK_OK;
 
-    if (!pa_streq(profile->name, card->active_profile->name))
+    if (!card->active_profile || !pa_streq(profile->name, card->active_profile->name))
         return PA_HOOK_OK;
 
     pa_log_debug("Active profile %s on card %s became unavailable, switching to another profile", profile->name, card->name);