+1

With this patch, I verified it works as described as the comment #2 of
the bug report.

gdm-binary has one contract #, and all its child share another contract
#.

Cheers,
Halton
On Wed, 2010-01-20 at 14:41 -0600, Brian Cameron wrote:
> Add the attached patch to fix doo bug #14007.  This improves GDM's SMF
> integration.  Refer here:
> 
>    http://defect.opensolaris.org/bz/show_bug.cgi?id=14007
> 
> In build 130, GDM was updated to the new GDM 2.28 version.  This
> version calls the user gnome-session with ctrun which starts the user
> session in a separate contract.  This ensures that a segfault in a user
> application does not cause the gdm service to restart.
> 
> However, this is not good enough.  For background, you can refer to
> bugster CR #6720967, when a similar bug was fixed in the old GDM.
> 
> The problem is that GDM really needs to ensure that the following
> processes are also in a separate contract:
> 
> - The login GUI
> - The Xorg Xserver
> - gdm-session-worker (which calls PAM)
> 
> To solve this problem, the attached patch ensures that the slave daemon
> which starts all of the above processes is in the separate SMF
> contract.  This ensures that any segfaults when the login GUI is
> displayed, from the Xserver, PAM, or other processes started by the
> slave are in a separate contract.
> 
> Note this is makes the new GDM work more similarly like the old GDM
> worked - the old GDM also ran the slave daemon in a separate contract.
> 
> Brian


Reply via email to