lkml.org 
[lkml]   [2017]   [Sep]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/6] KEYS: fix race between updating and finding negative key
On Tue, Sep 26, 2017 at 01:11:00PM -0700, Eric Biggers wrote:
> +static void mark_key_instantiated(struct key *key, unsigned int reject_error)
> +{
> + unsigned long old, new;
> +
> + do {
> + old = READ_ONCE(key->flags);
> + new = (old & ~(KEY_FLAG_NEGATIVE |
> + KEY_FLAGS_REJECT_ERROR_MASK)) |
> + KEY_FLAG_INSTANTIATED |
> + (reject_error ? KEY_FLAG_NEGATIVE : 0) |
> + (reject_error << KEY_FLAGS_REJECT_ERROR_SHIFT);
> + } while (cmpxchg_release(&key->flags, old, new) != old);
> +}

Sorry, I realized I screwed this up --- the flags like KEY_FLAG_NEGATIVE need to
be (1 << KEY_FLAG_NEGATIVE). I'll send another version which will be better
tested...

Eric

\
 
 \ /
  Last update: 2017-09-26 22:40    [W:0.103 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site