After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 168262 - display_name leaked
display_name leaked
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: General
unspecified
Other Linux
: Immediate blocker
: ---
Assigned To: Shaun McCance
Yelp maintainers
Depends on:
Blocks:
 
 
Reported: 2005-02-23 14:12 UTC by Paolo Borelli
Modified: 2005-03-06 09:59 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (1.16 KB, patch)
2005-02-23 14:13 UTC, Paolo Borelli 		none Details | Review

Description Paolo Borelli 2005-02-23 14:12:36 UTC 
spotted this memleak in gedit, here is the similar patch for yelp.
Comment 1 Paolo Borelli 2005-02-23 14:13:59 UTC 
Created attachment 37834 [details] [review]
patch
Comment 2 Paolo Borelli 2005-02-23 19:08:27 UTC 
committed.
Comment 3 Paolo Borelli 2005-03-05 18:45:33 UTC 
darn. We need to revert this asap: according to the docs "gdk_display_get_name
returns a string representing the display name. This string is owned by GDK and
should not be modified or freed"

I'm really sorry about it, but valgrind was definately marking the string as
leaked... (gtk bug?)


reopening and bumping prio.
Comment 4 Paolo Borelli 2005-03-06 09:59:29 UTC 
closing again, I made a big mess with this issue... the problem is tracked at
bug #169160