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 335387 - main() leaks session_name
main() leaks session_name
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
2.14.x
Other Linux
: Normal normal
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-03-21 18:06 UTC by Rodrigo Moya
Modified: 2006-07-26 12:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to fix the leak (1.47 KB, patch)
2006-03-21 18:07 UTC, Rodrigo Moya
committed Details | Review

Description Rodrigo Moya 2006-03-21 18:06:53 UTC
Attaching patch to fix it
Comment 1 Rodrigo Moya 2006-03-21 18:07:25 UTC
Created attachment 61711 [details] [review]
Patch to fix the leak
Comment 2 Claudio Saavedra 2006-06-30 10:34:32 UTC
Good patch. Can we commit it?
Comment 3 Rodrigo Moya 2006-06-30 10:47:25 UTC
Ugh, there is one line that should be removed from the patch, the g_free (session_name) at the end. Freeing it there makes the save_session code use an invalid string for saving the session.

Apart from that, yeah, it should be committed
Comment 4 Vincent Untz 2006-07-26 12:14:31 UTC
Committed (after having moved the g_free).