summaryrefslogtreecommitdiff
path: root/security/safesetid
diff options
context:
space:
mode:
authorPaul E. McKenney <paulmck@kernel.org>2019-10-05 01:07:09 +0300
committerPaul E. McKenney <paulmck@kernel.org>2019-10-30 18:45:57 +0300
commita60a5746004d7dbb68cbccd4c16d0529e2b2d1d9 (patch)
treea98e2179e735ebc2f18e9ad7a728833c2ef46eba /security/safesetid
parent445d3749315f34229dcfc3efd82796f97fc72e92 (diff)
downloadlinux-a60a5746004d7dbb68cbccd4c16d0529e2b2d1d9.tar.xz
security/safesetid: Replace rcu_swap_protected() with rcu_replace_pointer()
This commit replaces the use of rcu_swap_protected() with the more intuitively appealing rcu_replace_pointer() as a step towards removing rcu_swap_protected(). Link: https://lore.kernel.org/lkml/CAHk-=wiAsJLw1egFEE=Z7-GGtM6wcvtyytXZA1+BHqta4gg6Hw@mail.gmail.com/ Reported-by: Linus Torvalds <torvalds@linux-foundation.org> Reported-by: Reported-by: kbuild test robot <lkp@intel.com> [ paulmck: From rcu_replace() to rcu_replace_pointer() per Ingo Molnar. ] Signed-off-by: Paul E. McKenney <paulmck@kernel.org> Cc: Micah Morton <mortonm@chromium.org> Cc: James Morris <jmorris@namei.org> Cc: "Serge E. Hallyn" <serge@hallyn.com> Cc: <linux-security-module@vger.kernel.org>
Diffstat (limited to 'security/safesetid')
-rw-r--r--security/safesetid/securityfs.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/security/safesetid/securityfs.c b/security/safesetid/securityfs.c
index 74a13d432ed8..f8bc574cea9c 100644
--- a/security/safesetid/securityfs.c
+++ b/security/safesetid/securityfs.c
@@ -179,8 +179,8 @@ out_free_rule:
* doesn't currently exist, just use a spinlock for now.
*/
mutex_lock(&policy_update_lock);
- rcu_swap_protected(safesetid_setuid_rules, pol,
- lockdep_is_held(&policy_update_lock));
+ pol = rcu_replace_pointer(safesetid_setuid_rules, pol,
+ lockdep_is_held(&policy_update_lock));
mutex_unlock(&policy_update_lock);
err = len;