Bug 192400 - gdm does not use xdm pam service
Summary: gdm does not use xdm pam service
Status: RESOLVED WONTFIX
Alias: None
Product: openSUSE 10.2
Classification: openSUSE
Component: GNOME (show other bugs)
Version: unspecified
Hardware: Other Other
: P5 - None : Normal (vote)
Target Milestone: ---
Assignee: Stanislav Brabec
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on: 186189 279426
Blocks:
  Show dependency treegraph
 
Reported: 2006-07-14 09:03 UTC by Ludwig Nussel
Modified: 2008-06-25 09:23 UTC (History)
8 users (show)

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ludwig Nussel 2006-07-14 09:03:15 UTC
gdm contains copies of the xdm and xdm-np pam config files as gdm and gdm-autologin. Since xdm, wdm and kdm all use the xdm service gdm should do so as well.
Comment 1 JP Rosevear 2006-08-01 03:43:24 UTC
Stanislav, is there a historical reason for this?
Comment 2 Stanislav Brabec 2006-08-01 11:18:14 UTC
It was created in past and never revised to the new style.

We can keep gdm* PAM files and include xdm there or change code to use xdm.

The first can have reason, if we will want to do something gdm specific (differences password dialogs, maybe pam-keyring,...).
Comment 3 Stanislav Brabec 2006-09-01 11:49:11 UTC
Would you prefer /etc/pam.d/gdm which includes xdm or code change to use /etc/pam.d/xdm.

Note that there does not exist corresponding xdm-autologin.

Related: bug 186189
Comment 4 Ludwig Nussel 2006-09-01 14:36:10 UTC
I don't mind. The autologin xdm config is called xdm-np and is only used by kdm I think so if you are worried about the name it could easily be changed.
Comment 5 Stanislav Brabec 2006-09-04 13:50:09 UTC
/etc/pam.d/xdm-np is part of kdebase3-kdm. It has to be moved e. g. to xorg-x11 to be usable for gdm.
Comment 6 Stanislav Brabec 2006-09-20 14:42:08 UTC
Reassigning to X maintainer. If /etc/pam.d/xdm-np (or xdm-autologin or any other file of this meaning) will be packaged as part of X or any other package installed by default in the graphical environment, please reassign it back.

gdm cannot depend of kdebase3-kdm and cannot install file, which conflicts with /etc/pam.d/xdm-np.
Comment 7 Stefan Dirsch 2006-09-20 15:12:08 UTC
I'm not aware of any xdm-autologin file. I can take over xdm-np if this is ok with the kdebase3-kdm maintainer.
Comment 8 Stanislav Brabec 2006-09-20 15:32:08 UTC
I guess that xdm-np is an be equivalent of gdm-autologin. If it is true, "-np" looks more ugly than "-autologin", which is self descriptive.

It's your decision, how to name this file.
Comment 9 Stefan Dirsch 2006-09-20 15:42:12 UTC
Ok, I see.
Comment 10 Stefan Dirsch 2006-10-27 15:31:19 UTC
Still no information by kde-maintainers@suse.de. Therefore reassigning.
Comment 11 Stephan Kulow 2006-10-30 14:03:36 UTC
kdm hardcodes %s-np and I don't want to differ from upstream in that regard. 
Comment 12 Stefan Dirsch 2006-10-30 14:21:00 UTC
I've now added xdm-np to xorg-x11 package. Coolo, please remove it from kdebaes3-kdm. Stanislav, please remove xdm-autologin from gdm package and make use of xdm-np. 

Reassigning to Coolo. Please reassign to Stanislav after changing kdebase3-kdm accordingly.
Comment 13 Stanislav Brabec 2006-10-30 17:58:28 UTC
Note about gdm.autologin:

There may be required also SuSEconfig change, because SuSEconfig.gdm edits gdm-autologin to enable/disable passwordless login.

We will probably postpone gdm change to use xdm-np to 10.3.

But it should not affect current package change of xdm-np file.
Comment 14 Stefan Dirsch 2006-10-30 18:06:43 UTC
Ok with me. There's no SuSEconfig.xdm any more.
Comment 15 Stephan Kulow 2006-10-31 08:47:32 UTC
What use does autologin with required password have? anyway, in the light of "no more suseconfig" it sounds more useful to have two pam services and switch gdm depending on the value in sysconfig
Comment 16 Stanislav Brabec 2006-10-31 16:31:03 UTC
It saves typing of username and brings some level of security, e. g. for notebooks. Additionally, paswordless autologin cannot work with crypted or protected remote filesystems and also cannot unlock gnome-keyring.

Even in autologin mode, gdm displays any PAM requests, if there are any.
Comment 17 Stephan Kulow 2006-11-02 13:16:51 UTC
done for kdm
Comment 18 Stefan Dirsch 2007-04-02 20:16:58 UTC
Stanislav, gdm still includes xdm-autologin (see comment #12).
Comment 19 Stanislav Brabec 2007-04-03 12:32:24 UTC
No. It uses gdm-autologin.

To support password protected autologin, I need to edit it.

So I see following two chances:

1. Keep gdm resp. gdm-autologin and include xdm resp. xdm-np from there.

2. Remove gdm and gdm-autologin and change SuSEconfig.gdm to modify xdm-np.

If we will use 2., it will require to test, whether it will not break kdm. If we will use 1., I have to check, whether it is technically possible to include xdm-np and require password in gdm, if it is configured so.
Comment 20 Stefan Dirsch 2007-04-03 13:54:39 UTC
Ok. I would prefer 1. - if technically possible.
Comment 21 Stefan Dirsch 2007-05-12 10:28:11 UTC
Egbert, JFYI. Since Matthias or me is in Cc of this bugreport or the reported itself, it might be interesting for you as well.
Comment 22 Stefan Dirsch 2007-05-12 10:42:26 UTC
JFYI, Matthias. This is a bugreport, which is assigned to Egbert/me or with Egbert/me in CC or reported by Egbert/me.
Comment 23 Stanislav Brabec 2007-09-05 11:07:27 UTC
Still not done for 10.3.

gdm needs to play with PAM configuration for password less login. Is DISPLAYMANAGER_PASSWORD_LESS_LOGIN implemented in the same way in kdm?

If yes, we can do it just now (and do a change in SuSEconfig.gdm).

If no, we can't do it because gdm could require different configuration.
Comment 24 Stephan Kulow 2007-09-05 11:40:40 UTC
NO, it does not. See also #307566
Comment 25 Stanislav Brabec 2007-09-05 14:39:10 UTC
Then we cannot fix it yet, unless these two issues will get a solution:

- Password less login needs pam_permit in gdm, but not in xdm.

If pam_permit (and no pam login dialog) causes no problems for other display managers, then we probably can do this change depending on sysconfig values.

- Wee need a solution for pam-keyring (bug 186189).
Comment 26 Ludwig Nussel 2007-09-06 07:12:10 UTC
huh? why do you edit the pam config file? gdm switches between 'gdm' and 'gdm-autologin' all by itself based on the config setting "daemon/AutomaticLoginEnable". So if AutomaticLoginEnable=false then gdm-autologin isn't used at all, no need to edit it.
Comment 27 Stanislav Brabec 2007-09-06 13:21:31 UTC
I am switching to provide password-less login, not autologin. In gdm terms, these are different things:

autologin provides automatic login of selected users: implemented by switching betweek gdm/gdm-autologin

password less login provides standard user prompt, but not asks for password: implemented by modification of /etc/pam.d/gdm
Comment 28 Stephan Kulow 2008-06-25 09:19:01 UTC
mass reopening all 10.2 LATER+REMIND bugs.
Comment 29 Stephan Kulow 2008-06-25 09:23:09 UTC
close all 10.2 LATER/REMIND bugs as WONTFIX. Reopen yourself if you still plan to work on it.