After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 352260 - Esd blocks pam_mount / pam_umount
Esd blocks pam_mount / pam_umount
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
2.14.x
Other Linux
: Normal normal
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-08-21 13:47 UTC by Loïc Minier
Modified: 2007-01-22 19:07 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
Kills esd properly (found in Debian package) (1.26 KB, patch)
2006-08-21 13:48 UTC, Loïc Minier
none Details | Review

Description Loïc Minier 2006-08-21 13:47:28 UTC
Hi,

Daniel Kahn Gillmor reported in Debian bug http://bugs.debian.org/335944:
"""
in a system using pam_mount, it can be important for security that the
user's home directory be unmounted (via pam_close_session()) when they
log out.

However, debian's gnome-session spawns esd at login with a current
working directory (cwd) inherited from gnome-session (which appears to
default to $HOME when using gdm).  Since esd lives on in $HOME a
little while after the user exits the session, pam_mount can't unmount
the home directory.
"""

Back then, I myself heard other reports from person having trouble with esd still running after the session.  (This problem is probably not Debian specific.)

I'll attach the patch that we ship in Debian's gnome-session package since April.

Bye,
Comment 1 Loïc Minier 2006-08-21 13:48:45 UTC
Created attachment 71306 [details] [review]
Kills esd properly (found in Debian package)
Comment 2 Kjartan Maraas 2006-09-26 18:52:49 UTC
I think there were some changes in that area in esound recently, but we still haven't had a release of it. Any chance you could test CVS to see if it has been fixed "upstream"?
Comment 3 Josselin Mouette 2006-09-26 20:01:47 UTC
The fix to #342729 that has been committed would allow for a simpler patch, that would just shutdown esd instantaneously. It would then exit after playing the logout sound.

I also have to add that esd still doesn't chdir() to /, so this part of the patch has to be retained for pam_umount to work.
Comment 4 Vincent Untz 2007-01-06 13:06:32 UTC
I committed the code, without the sleep hack. You'll want to push the esd maintainer to make a new release.
Comment 5 Vincent Untz 2007-01-21 11:58:40 UTC
Josselin: btw, is there a bug against esd about it not doing a chdir()? I couldn't find one.
Comment 6 Josselin Mouette 2007-01-22 19:07:49 UTC
Indeed. Here it is: http://bugzilla.gnome.org/show_bug.cgi?id=399500