Bug 41129

Summary:	Fix memory leaks in the regression tests
Product:	dbus	Reporter:	Simon McVittie <smcv>
Component:	GLib	Assignee:	Simon McVittie <smcv>
Status:	RESOLVED FIXED	QA Contact:	John (J5) Palmieri <johnp>
Severity:	normal
Priority:	medium	CC:	rob.taylor
Version:	unspecified	Keywords:	patch
Hardware:	Other
OS:	All
URL:	http://cgit.freedesktop.org/~smcv/dbus-glib/log/?h=test-leaks-41129
Whiteboard:
i915 platform:		i915 features:
Bug Depends on:	41125, 41126
Bug Blocks:

Description Simon McVittie 2011-09-22 11:21:43 UTC

My 'test-leaks' branch fixes Bug #41125 (leaks) and Bug #41126 (crashes), then gets rid of various test-specific memory leaks and adds infrastructure to run the tests under valgrind, so we don't get into this situation again. See "URL" for the branch in cgit.

Comment 1 Dan Williams 2012-12-04 17:55:19 UTC

Test branch could use a rebase...

Comment 2 Simon McVittie 2012-12-04 18:04:22 UTC

Branch updated.

Comment 3 Simon McVittie 2012-12-04 18:15:38 UTC

18:14 < smcv> dcbw: ... and updated again. one new patch "Fix yet another leak, 
              this one caused by poor documentation (#29195)", and reword'd the 
              other commits to have a bug reference and signed-off-by

Comment 4 Simon McVittie 2013-10-21 11:39:53 UTC

10 months later, I'm going for "easier to ask forgiveness than permission". Branch merged, please revert anything you don't like. Fixed in git for 0.102.

Comment 5 Simon McVittie 2014-01-14 11:17:50 UTC

(In reply to comment #4)
> 10 months later, I'm going for "easier to ask forgiveness than permission".
> Branch merged, please revert anything you don't like.

I didn't do this when I said I had, but now I have. Fixed in git for 0.102.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.