TigerVNC / tigervnc

High performance, multi-platform VNC client and server
https://tigervnc.org
GNU General Public License v2.0
5.24k stars 952 forks source link

vncsession: use /bin/sh if the user shell is not set #1787

Closed casantos closed 3 months ago

casantos commented 3 months ago

An empty shell field in the password file is valid, although not common. Use /bin/sh in this case, as documented in the passwd(5) man page, since the vncserver script requires a non-empty SHELL environment variable.

Fixes issue #1786.

CendioOssman commented 3 months ago

Ooops. Surprised this wasn't noticed by your enterprise users earlier. :)