[PATCH 1/4] Use explicitly /bin/sh for running of Xwilling script via su.

Alan Coopersmith alan.coopersmith at oracle.com
Mon Aug 8 13:02:44 PDT 2011


On 08/08/11 12:26, Matěj Cepl wrote:
> su nobody -c <command> uses a default shell of the user, which in case
> of nobody is (at least on Linux distros) /sbin/nologin.
> 
> Signed-off-by: Matěj Cepl <mcepl at redhat.com>
> ---
>  config/xdm-config.cpp |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/config/xdm-config.cpp b/config/xdm-config.cpp
> index 59d548b..a685129 100644
> --- a/config/xdm-config.cpp
> +++ b/config/xdm-config.cpp
> @@ -16,7 +16,7 @@ DisplayManager.keyFile:		XDMCONFIGDIR/xdm-keys
>  DisplayManager.servers:		XDMCONFIGDIR/Xservers
>  DisplayManager.accessFile:	XDMCONFIGDIR/Xaccess
>  DisplayManager*resources:	XDMCONFIGDIR/Xresources
> -DisplayManager.willing:		SU nobody -c XDMSCRIPTDIR/Xwilling
> +DisplayManager.willing:		SU nobody -s /bin/sh -c XDMSCRIPTDIR/Xwilling
>  ! All displays should use authorization, but we cannot be sure
>  ! X terminals may not be configured that way, so they will require
>  ! individual resource settings.

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list