On Thu, 6 Oct 2011 13:19:35 +0100, Barry Scott <[email protected]> wrote:
> Is there anything else I need to do to make this patch acceptable
> for inclusion in Xorg?
Two things:
1. This option needs to be restricted to root, much like many other
options in the server (-modulepath, etc). There's a patch in the
queue which regularizes this, but the xf86 DDX uses:
if ( (geteuid() == 0) && (getuid() != 0) ) {
FatalError("The '%s' option can only be used by root.\n", argv[i]);
}
2. I'd love to figure out how to fork at the time of the error; this
would encourage people to actually use this option regularly.
posix threads makes the usual libc fork() function take the malloc
mutex across its operations, but I wonder if syscall(SYS_fork) is
portable enough to be used instead?
--
[email protected]
pgpX1iPIXr5ti.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
