Re: [PATCH 1/2] compositor: drop privileges if not using the drm backend

2012-01-31 Thread Ander Conselvan de Oliveira
On 01/31/2012 11:04 AM, Daniel Stone wrote: Hi, On 30 January 2012 13:02, Ander Conselvan de Oliveira wrote: + /* Only drm-backend needs root privileges. */ + if (backend) + seteuid(getuid()); You should be checking seteuid's return code here. My understanding of

Re: [PATCH 1/2] compositor: drop privileges if not using the drm backend

2012-01-31 Thread Daniel Stone
Hi, On 30 January 2012 13:02, Ander Conselvan de Oliveira wrote: > +       /* Only drm-backend needs root privileges. */ > +       if (backend) > +               seteuid(getuid()); You should be checking seteuid's return code here. Cheers, Daniel ___

[PATCH 1/2] compositor: drop privileges if not using the drm backend

2012-01-30 Thread Ander Conselvan de Oliveira
Signed-off-by: Ander Conselvan de Oliveira --- src/compositor.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index ab184ac..0d63e2a 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -,10 +,14 @@ int main(int ar