[PATCH] dix: Prevent access to freed memory if a client kills itself.

Rami Ylimäki rami.ylimaki at vincit.fi
Fri Sep 23 00:28:12 PDT 2011


On 09/22/2011 07:34 PM, Jamey Sharp wrote:
> This patch makes sense to me, but I have a couple of requests:
>
> I'm not sure why you extracted out a separate function; I'm not
> convinced that makes the code more clear, in this case.

Originally, there was no good reason for extracting the code to a 
separate function. However, I'm currently suspecting that a certain XACE 
module 
(https://meego.gitorious.org/maemo-multimedia/xserver-policy-enforcement) might 
be suffering from the same problem. In this case I'd want to have a 
separate function and export it as well so that I wouldn't have to 
copy-paste its contents into the XACE module. If it turns out that XPE 
works fine, I can remove that function in the next patch revision.

> More importantly, I'd like to see justification in the commit message
> for deleting LBX support here. I seem to recall that support was
> deleted from the rest of the server already, which would be excellent
> justification, but please say so if that's true.

To be honest, I didn't know what LBX was when I wrote the patch and 
didn't take into account the effect of extensions on closing down 
clients. But it seems that LBX has been gone for 5 years now, so I can 
clarify this in the commit message of the next patch revision.

-- Rami



More information about the xorg-devel mailing list