[PATCH-V4] xserver: Optional backtrace handler
Keith Packard
keithp at keithp.com
Wed Oct 12 10:50:32 PDT 2011
On Thu, 6 Oct 2011 13:19:35 +0100, Barry Scott <barry.scott at onelan.co.uk> 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?
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20111012/e81f4eb1/attachment.pgp>
More information about the xorg-devel
mailing list