Merge lp:~charlesk/indicator-sound/blacklist-leaks into lp:indicator-sound/fifth
Proposed by
Charles Kerr
Status: | Merged |
---|---|
Approved by: | Charles Kerr |
Approved revision: | 312 |
Merged at revision: | 315 |
Proposed branch: | lp:~charlesk/indicator-sound/blacklist-leaks |
Merge into: | lp:indicator-sound/fifth |
Diff against target: |
57 lines (+8/-8) 1 file modified
src/sound-service-dbus.c (+8/-8) |
To merge this branch: | bzr merge lp:~charlesk/indicator-sound/blacklist-leaks |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Conor Curran (community) | Approve | ||
Review via email: mp+98024@code.launchpad.net |
To post a comment you must log in.
g_variant_ get("(s) ") gets a newly-allocated string get("(& s)") peeks the variant's internal pointer to be treated as const
g_variant_
We were doing the former and leaking the string; this patch uses the latter and adds const