Bug#493524: xdm: SE Linux patch was written for old version

2008-08-11 Thread Julien Cristau
On Sun, Aug 3, 2008 at 01:51:56 +0200, Julien Cristau wrote: On Sun, Aug 3, 2008 at 09:30:37 +1000, Russell Coker wrote: + if (getseuserbyname(login, seuser, level) == 0) + ret=get_default_context_with_level(seuser, level, 0, scontext); + if (ret 0 || scontext == NULL) {

Bug#493524: xdm: SE Linux patch was written for old version

2008-08-11 Thread Russell Coker
On Monday 11 August 2008 21:26, Julien Cristau [EMAIL PROTECTED] wrote: Looks like seuser and level are never freed. Am I missing something? Looking at selinux/selinux.h, we read: /* Get the SELinux username and level to use for a given Linux username.? These values may then be passed

Bug#493524: xdm: SE Linux patch was written for old version

2008-08-02 Thread Russell Coker
Package: xdm Version: 1:1.1.8-3 Severity: normal The SE Linux patch was written for an older release of SE Linux and doesn't work properly with the latest code base (gets the wrong login context). Below is a modified patch to make it work correctly. Could you please build with this patch ASAP

Bug#493524: xdm: SE Linux patch was written for old version

2008-08-02 Thread Julien Cristau
On Sun, Aug 3, 2008 at 09:30:37 +1000, Russell Coker wrote: +#ifdef HAVE_SELINUX +/* This should be run just before we exec the user session. */ +static int +xdm_selinux_setup (const char *login) + { + security_context_t scontext; + int ret = -1; + char *seuser=NULL; +