sound: check for audio_sink and also unref after using the object

1 files changed, 11 insertions, 7 deletions

diff --git a/src/sound.cpp b/src/sound.cpp
index 0b7894a..c6a7b0a 100644
--- a/src/sound.cpp
+++ b/src/sound.cpp
@@ -114,15 +114,19 @@ private:
         {
             /* Set the media role if audio sink is pulsesink */
             GstElement *audio_sink = NULL;
-            GstPluginFeature *feature = NULL;
             g_object_get(self->m_play, "audio-sink", &audio_sink, NULL);
-            feature = GST_PLUGIN_FEATURE_CAST(GST_ELEMENT_GET_CLASS(audio_sink)->elementfactory);
-            if (feature && g_strcmp0(gst_plugin_feature_get_name(feature), "pulsesink") == 0)
+            if (audio_sink)
             {
-                std::string role_str("props,media.role=alarm");
-                GstStructure *props = gst_structure_from_string(role_str.c_str(), NULL);
-                g_object_set(audio_sink, "stream-properties", props, NULL);
-                gst_structure_free(props);
+                GstPluginFeature *feature = NULL;
+                feature = GST_PLUGIN_FEATURE_CAST(GST_ELEMENT_GET_CLASS(audio_sink)->elementfactory);
+                if (feature && g_strcmp0(gst_plugin_feature_get_name(feature), "pulsesink") == 0)
+                {
+                    std::string role_str("props,media.role=alarm");
+                    GstStructure *props = gst_structure_from_string(role_str.c_str(), NULL);
+                    g_object_set(audio_sink, "stream-properties", props, NULL);
+                    gst_structure_free(props);
+                }
+                gst_object_unref(audio_sink);
             }
         }