Re: [PATCH 3/4] fs, afs: convert afs_server.usage from atomic_t to refcount_t

From: David Howells
Date: Wed Feb 22 2017 - 11:32:27 EST


Elena Reshetova <elena.reshetova@xxxxxxxxx> wrote:

> refcount_t type and corresponding API should be
> used instead of atomic_t when the variable is used as
> a reference counter. This allows to avoid accidental
> refcounter overflows that might lead to use-after-free
> situations.

Although I don't see an assertion for this (the window is too small), it is
possible for a dead server record to get resurrected. Take a look at
afs_put_server() and note there's a check around the move to the graveyard.

So, please hold this patch also.

David