<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - PulseAudio pre-v9.0 induce Totem failure"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=95135#c2">Comment # 2</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - PulseAudio pre-v9.0 induce Totem failure"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=95135">bug 95135</a>
              from <span class="vcard"><a class="email" href="mailto:arun@accosted.net" title="Arun Raghavan <arun@accosted.net>"> <span class="fn">Arun Raghavan</span></a>
</span></b>
        <pre>We have the root cause of this bug. The options are to revert the offending
patch for 9.0 (leaving the underlying problem unaddressed, but this is not a
regression). The alternative is to look at merging my patches to drop json-c as
a dep soon. Root cause from email discussion follows:

----

Okay, so this is a fun one. The offending commit is:

  commit 12a202c510dcacbd2b85fcc1170484eb16fef491
  Author: Arun Raghavan <<a href="mailto:git@arunraghavan.net">git@arunraghavan.net</a>>
  Date:   Thu Dec 31 09:27:56 2015 +0530

      format: Make pa_format_info_valid() stricter for PCM

      We should do stricter validation when we can.

The check fails at:

  if (json_object_get_type(o) != json_type_string) {
      pa_log_debug("Format info property '%s' type is not string.", key);
      json_object_put(o);
      return -PA_ERR_INVALID;
  }

Rhythmbox and Totem are linked to the json-glib library which *also*
has a json_object_get_type(). That is the function which gets called,
and returns a junk type which makes this code fail.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>