[PATCH weston] compositor: Load shell plugin after option modules

[Tiago Vignatti]

On 09/25/2012 06:19 PM, Kristian Høgsberg wrote:
> On Tue, Sep 25, 2012 at 05:54:40PM +0300, Tiago Vignatti wrote:
>>
>> BTW (a bit related with this) to solve part of the Bug 53679, the
>> way we're using socketpair_cloexec to create the client socket and
>> later get its credentials in wl_client_create is wrong; clients end
>> up with the same pid, uid, etc as the compositor instead.
>> Specifically xwayland and weston-desktop-shell are getting the wrong
>> values.
>
> Oh, because we're still holding both ends when we create the client?
> Hehe, that's wonderful, nice find :) Hmm... do the credentials change
> after fork or after we close the socket in the parent...  I'm not sure
> how things are supposed to work in that case, should be interesting to
> find out.

we discussed this on IRC and it doesn't seem related with who's holding 
any end.

"returned credentials are those that were in effect at the time of the 
call to connect(2) or socketpair(2)", that's what man 7 socket says 
about the SO_PEERCRED option. So I assume we cannot use it after fork, 
the way we're doing. Man page says also this options is specific to use 
with sockets created via socketpair, which is funny cause for us it's 
working with accept(). But I guess this is okay.

Maybe we should special-case the forking clients and get the return 
value from the fork() call, i.e the pid now?

                   Tiago