[PATCH xserver] check for elevated privileges not uid=0

Tormod Volden lists.tormod at gmail.com
Tue Oct 11 00:17:00 PDT 2011


On Mon, Oct 10, 2011 at 9:04 PM, Antoine Martin wrote:
> And another blooper, sorry for the spam. Correct patch attached.
> Need sleep, will pick this up tomorrow.

>        Subject: [PATCH xserver] check for elevated privileges not uid=0 (V3)
>
> Couldn't find any reference to it, so I just removed it.

To avoid confusion, you should version your patches. What I said was
to put the version inside the bracketed part. And if your code changes
are limited to the hw/xfree86 tree, you might add xfree86 as a prefix.
Just as a suggestion, feel free to rephrase:

Subject: [PATCH v3] xfree86: Check for elevated privileges instead of
just uid==0

+          /* XXX ouch, coudn't get back to original uid
+            what can we do ??? */

You do not need a comment if you have an explaining error message.
Otherwise, comments should also be properly spelled and reflect the
quality of the code.

Cheers,
Tormod


More information about the xorg-devel mailing list