On Tue, 2010-01-12 at 00:24 -0600, Brian Cameron wrote:
> - gdm-22-scripts.diff to fix doo bug #13811, GNOME bugzilla #602403.
> 
>    Fixes GDM so it sets PATH reasonably when running Init, PreSession,
>    PostSession, and PostLogin scripts, and also ensures that the
>    PostSession script is run with the user set to the user logging in
>    rather than the "gdm" user, so it works like the old GDM did.  See:
> 
>    http://defect.opensolaris.org/bz/show_bug.cgi?id=13811
>    https://bugzilla.gnome.org/show_bug.cgi?id=602403 

I found this patch introduce another issue - gdm-simple-slave
won't quit if gdm-binary died when no user selected.

After investigate, I found gdm_session_direct_get_username() is NULL when no
user is choose or entered. So I rework this patch to check whether user is NULL.
If NULL, run PostLogin script as user gdm, otherwise run as the entered user.

I updated the latest patch on GNOME bug report 
https://bugzilla.gnome.org/show_bug.cgi?id=602403#c6

I'll update the patch on trunk and gnome-2-28 branch also.

Thanks,
Halton.

Reply via email to