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

--- Comment #1 from Arun Raghavan <[email protected]> ---
Hey Sachin, you're right. Looking at the code, you'll also notice that fltr is
being leaked. Would you like to submit a fix for this? It might make sense to
initialise fltr and module_name as NULL, and then just have something like:

    if (something) {
       pa_log(...);
       goto done;
    }

    if (something else) {
       pa_log(...);
       goto done;
    }

done:
    pa_xfree(module_name);
    pa_xfree(fltr);

    return ...

This makes it more likely that we won't miss freeing if we add additional
conditions.

-- 
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]
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-bugs

Reply via email to