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 598494 - keyring doesn't go away when session
keyring doesn't go away when session
Status: RESOLVED FIXED
Product: gnome-keyring
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME keyring maintainer(s)
GNOME keyring maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-10-14 21:29 UTC by Ray Strode [halfline]
Modified: 2011-03-09 16:29 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Listen for the session bus to go away and make an exit in that case (714 bytes, application/octet-stream)
2009-10-14 21:29 UTC, Ray Strode [halfline] 		  Details
Don't start keyring at all if password is empty (791 bytes, patch)
2009-10-19 15:32 UTC, Ray Strode [halfline] 		none Details | Review

Description Ray Strode [halfline] 2009-10-14 21:29:38 UTC 
Created attachment 145455 [details]
Listen for the session bus to go away and make an exit in that case

The keyring only seems to die when the session ends in an orderly fashion.

It should make sure it goes away when the session crashes as well.
Comment 1 Ray Strode [halfline] 2009-10-19 15:30:28 UTC 
Also, see https://bugzilla.redhat.com/show_bug.cgi?id=529709
Comment 2 Ray Strode [halfline] 2009-10-19 15:32:49 UTC 
Created attachment 145791 [details] [review]
Don't start keyring at all if password is empty

This is a patch from Dan Walsh to prevent the keyring from getting started at all if there is no password.

Previously it was getting started in a way where it wouldn't scope itself to the session bus which would prevent it from getting cleaned up on logout.
Comment 3 Stef Walter 2010-02-07 19:22:24 UTC 
Committed the first patch. The gnome-keyring startup has been significantly rewritten, and there's some issues like bug #607085. So the second patch needs some more thought. Will leave this open and commit something later.
Comment 4 Stef Walter 2011-03-09 16:29:10 UTC 
Ray, I believe these issues have been fixed. So I'm closing this bug. If the startup needs more tweaking, let me know.