Bug 467286 - Lack of default shell should default to /bin/sh, not send nothing at all and cause gdm logins to fail
Summary: Lack of default shell should default to /bin/sh, not send nothing at all and ...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: xorg-x11-xinit
Version: 5.3
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: Søren Sandmann Pedersen
QA Contact: desktop-bugs@redhat.com
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-10-16 17:31 UTC by Suzanne Hillman
Modified: 2014-06-18 09:10 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 484963 (view as bug list)
Environment:
Last Closed: 2009-02-10 21:19:07 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Suzanne Hillman 2008-10-16 17:31:07 UTC
Description of problem:
Was trying to login with ldap users who had local home directories, but nothing else was local.

Login on virtual terminals worked, but gdm logins fails. After much investigation by rstrode, it was determined that there was no shell being set, and therefore sent, in Xsession.

Lack of default shell should default to /bin/sh, not send nothing at all and cause gdm logins to fail, at least based on passwd(5):

              shell     the program to run at login (if empty,  use  /bin/sh).
                        If  set to a non-existing executable, the user will be
                        unable to login through login(1).

I'm not yet sure if I should also open a gdm bug to suggest that it should correctly handle a lack of shell information being sent.

Version-Release number of selected component (if applicable):
xorg-x11-xinit-1.0.2-15.el5

How reproducible:
Always, if you have a user with no shell (I suspect this would happen if local, as well)

Steps to Reproduce:
1. Set up a user with no shell specified.
2. Try to log into gdm
  
Actual results:
It'll fail with _weird_ errors.

Expected results:
Should work.

Additional info:


Note You need to log in before you can comment on or make changes to this bug.