gdm r6401 - in branches/gnome-2-20: . daemon



Author: bcameron
Date: Thu Aug 21 03:08:18 2008
New Revision: 6401
URL: http://svn.gnome.org/viewvc/gdm?rev=6401&view=rev

Log:
2008-08-20 Brian Cameron <brian cameron sun com>

        * daemon/slave.c: Fix memory leak in the SELinux code.  Fixes bug
          #548670.  Patch by Josselin Mouette <joss malsain org> and
          Julien Cristau.


Modified:
   branches/gnome-2-20/ChangeLog
   branches/gnome-2-20/daemon/slave.c

Modified: branches/gnome-2-20/daemon/slave.c
==============================================================================
--- branches/gnome-2-20/daemon/slave.c	(original)
+++ branches/gnome-2-20/daemon/slave.c	Thu Aug 21 03:08:18 2008
@@ -3518,8 +3518,11 @@
 	if (is_selinux_enabled () <= 0)
 		return TRUE;
 
-	if (getseuserbyname(login, &seuser, &level) == 0)
+	if (getseuserbyname(login, &seuser, &level) == 0) {
 		ret=get_default_context_with_level(seuser, level, 0, &scontext);
+		free(seuser);
+		free(level);
+	}
 
 	if (ret < 0) {
 		gdm_error ("SELinux gdm login: unable to obtain default security context for %s.", login);



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]