Skip to content

Commit e04918d

Browse files
herbertxbrauner
authored andcommitted
cred: Fix RCU warnings in override/revert_creds
Fix RCU warnings in override_creds and revert_creds by turning the RCU pointer into a normal pointer using rcu_replace_pointer. These warnings were previously private to the cred code, but due to the move into the header file they are now polluting unrelated subsystems. Fixes: 49dffdf ("cred: Add a light version of override/revert_creds()") Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> Link: https://lore.kernel.org/r/Z8QGQGW0IaSklKG7@gondor.apana.org.au Signed-off-by: Christian Brauner <brauner@kernel.org>
1 parent d082ecb commit e04918d

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

include/linux/cred.h

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -172,18 +172,12 @@ static inline bool cap_ambient_invariant_ok(const struct cred *cred)
172172

173173
static inline const struct cred *override_creds(const struct cred *override_cred)
174174
{
175-
const struct cred *old = current->cred;
176-
177-
rcu_assign_pointer(current->cred, override_cred);
178-
return old;
175+
return rcu_replace_pointer(current->cred, override_cred, 1);
179176
}
180177

181178
static inline const struct cred *revert_creds(const struct cred *revert_cred)
182179
{
183-
const struct cred *override_cred = current->cred;
184-
185-
rcu_assign_pointer(current->cred, revert_cred);
186-
return override_cred;
180+
return rcu_replace_pointer(current->cred, revert_cred, 1);
187181
}
188182

189183
/**

0 commit comments

Comments
 (0)