[pulseaudio-tickets] [Bug 63923] New: Module index can be read before a real index has been assigned

[bugzilla-daemon at freedesktop.org]

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

          Priority: medium
            Bug ID: 63923
                CC: lennart at poettering.net
          Assignee: pulseaudio-bugs at lists.freedesktop.org
           Summary: Module index can be read before a real index has been
                    assigned
        QA Contact: pulseaudio-bugs at lists.freedesktop.org
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: tanuk at iki.fi
          Hardware: Other
            Status: NEW
           Version: unspecified
         Component: core
           Product: PulseAudio

pa_module.index is set after calling the init() callback. Prior to that it's
PA_IDXSET_INVALID. The init() callback can do things that cause hooks to be
fired, and that in turn may make the module object visible to code that reads
the module index.

I'm not sure what would be the best way to fix this. The simplest way would be
to add the new module to pa_core.modules before calling init(), but then module
indexes would be consumed also by modules that fail to load. Perhaps that's not
a significant problem, though.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/pulseaudio-bugs/attachments/20130425/37c20433/attachment.html>