[PATCH] weston-launch: Only drop privileges if running as root

Kristian Høgsberg hoegsberg at gmail.com
Tue Oct 1 10:42:44 PDT 2013


On Tue, Oct 01, 2013 at 12:20:29PM +0200, Tomeu Vizoso wrote:
> ---
>  src/weston-launch.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

That looks good, applied.

Kristian

> diff --git a/src/weston-launch.c b/src/weston-launch.c
> index 525c61c..1b560af 100644
> --- a/src/weston-launch.c
> +++ b/src/weston-launch.c
> @@ -577,7 +577,8 @@ launch_compositor(struct weston_launch *wl, int argc, char *argv[])
>  	if (wl->new_user)
>  		setup_session(wl);
>  
> -	drop_privileges(wl);
> +	if (geteuid() == 0)
> +		drop_privileges(wl);
>  
>  	setenv_fd("WESTON_TTY_FD", wl->tty);
>  	setenv_fd("WESTON_LAUNCHER_SOCK", wl->sock[1]);
> -- 
> 1.8.3.1
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list