Bug 6300 - Memory leak in glib bindings
Summary: Memory leak in glib bindings
Status: RESOLVED FIXED
Alias: None
Product: dbus
Classification: Unclassified
Component: core (show other bugs)
Version: unspecified
Hardware: Other Linux (All)
: high normal
Assignee: Havoc Pennington
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-17 21:29 UTC by Jakub Stachowski
Modified: 2006-08-01 10:26 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
Fix (2.98 KB, patch)
2006-03-17 21:30 UTC, Jakub Stachowski 		Details | Splinter Review
Fix with assert (3.03 KB, patch)
2006-03-18 04:55 UTC, Jakub Stachowski 		Details | Splinter Review
Show Obsolete (1) View All

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.