]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commit
KEYS: Fix garbage collector
authorDavid Howells <dhowells@redhat.com>
Mon, 14 Sep 2009 16:26:13 +0000 (17:26 +0100)
committerJames Morris <jmorris@namei.org>
Mon, 14 Sep 2009 23:11:02 +0000 (09:11 +1000)
commitc08ef808ef24df32e25fbd949fe5310172f3c408
tree12bae6fd48e1cdcc1b792c221376c727d9472cc6
parent5c84342a3e147a23752276650340801c237d0e56
KEYS: Fix garbage collector

Fix a number of problems with the new key garbage collector:

 (1) A rogue semicolon in keyring_gc() was causing the initial count of dead
     keys to be miscalculated.

 (2) A missing return in keyring_gc() meant that under certain circumstances,
     the keyring semaphore would be unlocked twice.

 (3) The key serial tree iterator (key_garbage_collector()) part of the garbage
     collector has been modified to:

     (a) Complete each scan of the keyrings before setting the new timer.

     (b) Only set the new timer for keys that have yet to expire.  This means
         that the new timer is now calculated correctly, and the gc doesn't
         get into a loop continually scanning for keys that have expired, and
         preventing other things from happening, like RCU cleaning up the old
         keyring contents.

     (c) Perform an extra scan if any keys were garbage collected in this one
       as a key might become garbage during a scan, and (b) could mean we
       don't set the timer again.

 (4) Made key_schedule_gc() take the time at which to do a collection run,
     rather than the time at which the key expires.  This means the collection
     of dead keys (key type unregistered) can happen immediately.

Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
security/keys/gc.c
security/keys/key.c
security/keys/keyctl.c
security/keys/keyring.c