Use pa_safe_streq instead of pa_streq in subscribe-observer

[Version] 5.0.124
[Profile] Common
[Issue Type] SVACE

Change-Id: Idab8b9861331562a3fd77b00f8841fdbc70bc2ed

diff --git a/src/subscribe-observer.c b/src/subscribe-observer.c
index 7b13092..366d531 100644 (file)
--- a/src/subscribe-observer.c
+++ b/src/subscribe-observer.c
@@ -443,7 +443,7 @@ static DBusHandlerResult dbus_filter_audio_client_handler(DBusConnection *c, DBu
 
     dbus_error_init(&error);
 
-    if (!pa_streq(dbus_message_get_interface(m), INTERFACE_AUDIO_CLIENT))
+    if (!pa_safe_streq(dbus_message_get_interface(m), INTERFACE_AUDIO_CLIENT))
         return DBUS_HANDLER_RESULT_NOT_YET_HANDLED;
 
 
@@ -459,7 +459,7 @@ static DBusHandlerResult dbus_filter_audio_client_handler(DBusConnection *c, DBu
         if (get_sender_pid(c, m, &sender_pid) < 0)
             goto fail;
 
-        if (!pa_streq(dbus_message_get_signature(m), "sub")) {
+        if (!pa_safe_streq(dbus_message_get_signature(m), "sub")) {
             pa_log_error("Invalid signature for ClientSubscribed, %s", dbus_message_get_signature(m));
             goto fail;
         }
@@ -489,7 +489,7 @@ static DBusHandlerResult dbus_filter_audio_client_handler(DBusConnection *c, DBu
         if (get_sender_pid(c, m, &sender_pid) < 0)
             goto fail;
 
-        if (!dbus_message_iter_init(m, &arg_i) || !pa_streq(dbus_message_get_signature(m), "usuv")) {
+        if (!dbus_message_iter_init(m, &arg_i) || !pa_safe_streq(dbus_message_get_signature(m), "usuv")) {
             pa_log_error("Invalid signature for ClientSignalHandled, %s", dbus_message_get_signature(m));
             goto fail;
         }
@@ -518,7 +518,7 @@ static DBusHandlerResult dbus_filter_audio_client_handler(DBusConnection *c, DBu
     } else if (dbus_message_is_signal(m, INTERFACE_AUDIO_CLIENT, "EmergentExit")) {
         /* Client will be exit emergently, we should care about that */
         dbus_int32_t client_pid;
-        if (!pa_streq(dbus_message_get_signature(m), "i")) {
+        if (!pa_safe_streq(dbus_message_get_signature(m), "i")) {
             pa_log_error("Invalid signature for EmergentExit, %s", dbus_message_get_signature(m));
             goto fail;
         }