https://bugs.freedesktop.org/show_bug.cgi?id=55607

          Priority: medium
            Bug ID: 55607
                CC: [email protected]
          Assignee: [email protected]
           Summary: crash because of null string on solaris
        QA Contact: [email protected]
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: [email protected]
          Hardware: Other
            Status: NEW
           Version: unspecified
         Component: core
           Product: PulseAudio

Solaris' printf doesn't support NULL arguments for %s, which pulseaudio
unfortunately gives it now and then. Making a safe printf wrapper is just a
major pain, so I guess we'll have to find the offenders that provide null
pointers. The most glaring one is pa_thread_get_name():

Index: src/pulsecore/thread-posix.c
===================================================================
--- src/pulsecore/thread-posix.c    (revision 25954)
+++ src/pulsecore/thread-posix.c    (working copy)
@@ -211,6 +211,9 @@
     }
 #endif

+    if (t->name == NULL)
+        return "(null)";
+
     return t->name;
 }

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
_______________________________________________
pulseaudio-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-bugs

Reply via email to