Working with Lokanath Das, the patch has been reworked to no longer set
the Workstation Owner.  Alan, this avoids the issue you highlight below.

Now, the patch sets PRIV_WIN_DAC_READ and PRIV_WIN_DAC_WRITE when
launching gnome-session only when the GDM login session is started (not
the user session).

Also, the patch now changes the property UID of the "RESOURCE MANAGER"
XAtom to the user logging in when the user session starts, avoiding
problems in the user session when using Trusted.

The name of the patch changed from gdm-19-workstation-owner.diff to
gdm-19-trusted.diff since it doesn't change the workstation owner
anymore.

Brian


On 09-12-21 12:51 PM, Alan Coopersmith wrote:
> +#ifdef HAVE_SUNTSOL
> +        passwd_entry = getpwnam ("gdm");
> +        XTSOLsetWorkstationOwner (slave->priv->server_display,
> &(passwd_entry->pw_uid));
> +#endif
>
> Presumably if the gdm user doesn't exist you've errored out before
> this and don't need to worry about introducing a segfault here?
>

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: gdm-19-trusted.diff
URL: 
<http://mail.opensolaris.org/pipermail/jds-review/attachments/20091222/081c6a35/attachment.ksh>

Reply via email to