diff options
author | Jasper St. Pierre <jstpierre@mecheye.net> | 2014-02-13 14:48:00 -0500 |
---|---|---|
committer | Jasper St. Pierre <jstpierre@mecheye.net> | 2014-02-14 20:54:53 -0500 |
commit | 8fda79cda9b8a89ddcfe3a95436c2b92699d1869 (patch) | |
tree | f200fa1bfad593325542b4e24c0ec9eceea05486 /daemon/gdm-session.h | |
parent | 7c34b11869261870d3fba1ccb4d5471e43d3dbba (diff) | |
download | gdm-8fda79cda9b8a89ddcfe3a95436c2b92699d1869.tar.gz |
session: Prevent memory leaks by removing strdup accessors
gdm_session_get_* dup the strings they return, but not a lot of code
frees the resulting strings. It's a lot easier to not dup the return
values and make them const char *, as they won't ever change while
some code holds onto them...
Diffstat (limited to 'daemon/gdm-session.h')
-rw-r--r-- | daemon/gdm-session.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/daemon/gdm-session.h b/daemon/gdm-session.h index 33ff43a7..fb190bf7 100644 --- a/daemon/gdm-session.h +++ b/daemon/gdm-session.h @@ -105,11 +105,11 @@ void gdm_session_start_reauthentication (GdmSession *session, GPid pid_of_caller, uid_t uid_of_caller); -char *gdm_session_get_server_address (GdmSession *session); -char *gdm_session_get_username (GdmSession *session); -char *gdm_session_get_display_device (GdmSession *session); -char *gdm_session_get_display_seat_id (GdmSession *session); -char *gdm_session_get_session_id (GdmSession *session); +const char *gdm_session_get_server_address (GdmSession *session); +const char *gdm_session_get_username (GdmSession *session); +const char *gdm_session_get_display_device (GdmSession *session); +const char *gdm_session_get_display_seat_id (GdmSession *session); +const char *gdm_session_get_session_id (GdmSession *session); gboolean gdm_session_bypasses_xsession (GdmSession *session); void gdm_session_start_conversation (GdmSession *session, |