Merge lp:~vanvugt/compiz/fix-1102829 into lp:compiz/0.9.9

Proposed by Daniel van Vugt on 2013-01-22
Status: Merged
Approved by: Sam Spilsbury on 2013-01-22
Approved revision: 3571
Merged at revision: 3576
Proposed branch: lp:~vanvugt/compiz/fix-1102829
Merge into: lp:compiz/0.9.9
Diff against target: 22 lines (+11/-1)
1 file modified
tests/compiz.supp (+11/-1)
To merge this branch: bzr merge lp:~vanvugt/compiz/fix-1102829
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve on 2013-01-22
Sam Spilsbury 2013-01-22 Approve on 2013-01-22
Review via email: mp+144253@code.launchpad.net

Commit message

Suppress leaks reported from Google Protobuf. They're out of our control,
I think, and likely one-time.
(LP: #1102829)

To post a comment you must log in.
Sam Spilsbury (smspillaz) wrote :

Thanks. Good catch, again :)

review: Approve
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Sam Spilsbury (smspillaz) wrote :

BTW, I don't know if the function name mangling is necessary. I can't check right now but it is worth a follow up.

PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/compiz.supp'
2--- tests/compiz.supp 2013-01-17 16:26:28 +0000
3+++ tests/compiz.supp 2013-01-22 09:42:24 +0000
4@@ -19,7 +19,17 @@
5 # Foundation, Inc., 51 Franklin Street, Fifth Floor,
6 # Boston, MA 02110-1301, USA.
7 #
8-
9+
10+{
11+ Google protobuf leaks in operator new(unsigned long)
12+ Memcheck:Leak
13+ fun:_Znwm
14+ ...
15+ fun:_ZN6google8protobuf25EncodedDescriptorDatabase3AddEPKvi
16+ fun:_ZN6google8protobuf14DescriptorPool24InternalAddGeneratedFileEPKvi
17+ fun:*protobuf_AddDesc_*
18+}
19+
20 {
21 g_spawn_check_exit_status
22 Memcheck:Cond

Subscribers

People subscribed via source and target branches