Bug 6300

Summary:	Memory leak in glib bindings
Product:	dbus	Reporter:	Jakub Stachowski <qbast>
Component:	core	Assignee:	Havoc Pennington <hp>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	high
Version:	unspecified
Hardware:	Other
OS:	Linux (All)
Whiteboard:
i915 platform:		i915 features:
Attachments:	Fix Fix with assert

Description Jakub Stachowski 2006-03-17 21:29:38 UTC

Sending or receiving array of complex types (like structs) causes memory leak.  
This is because array is represented as GPtrArray and its elements are never  
freed (only array itself is freed). The same applies to GSList.

Comment 1 Jakub Stachowski 2006-03-17 21:30:55 UTC

Created attachment 4978 [details] [review]
Fix

Comment 2 John (J5) Palmieri 2006-03-18 03:53:49 UTC

Jakub,

As talked about on the mailing list can you update your patch to assert on
pointers?  Theoreticly the situation should not happen but adding the assert
will allow us to catch it if it does.  Thanks.

Comment 3 Jakub Stachowski 2006-03-18 04:55:09 UTC

Created attachment 4981 [details] [review]
Fix with assert

Added g_assert_not_reached() for pointer case

Comment 4 Robert McQueen 2006-04-29 07:09:11 UTC

Applied in CVS.

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.