summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPaul Moore <paul.moore@hp.com>2007-01-05 15:08:21 -0500
committerDavid S. Miller <davem@sunset.davemloft.net>2007-01-09 00:30:00 -0800
commit797951200679f1d5ea12a2e58cc7bdbc2848764c (patch)
treeaaf0785e317ad5f4651324669bcafbd163d1833d
parent86112ffdccab3ee75bc9d9dfae6745df73189e37 (diff)
downloadop-kernel-dev-797951200679f1d5ea12a2e58cc7bdbc2848764c.zip
op-kernel-dev-797951200679f1d5ea12a2e58cc7bdbc2848764c.tar.gz
NetLabel: correct locking in selinux_netlbl_socket_setsid()
The spinlock protecting the update of the "sksec->nlbl_state" variable is not currently softirq safe which can lead to problems. This patch fixes this by changing the spin_{un}lock() functions into spin_{un}lock_bh() functions. Signed-off-by: Paul Moore <paul.moore@hp.com> Signed-off-by: James Morris <jmorris@namei.org>
-rw-r--r--security/selinux/ss/services.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/security/selinux/ss/services.c b/security/selinux/ss/services.c
index 372eaad..3eb1fa9 100644
--- a/security/selinux/ss/services.c
+++ b/security/selinux/ss/services.c
@@ -2491,9 +2491,9 @@ static int selinux_netlbl_socket_setsid(struct socket *sock, u32 sid)
rc = netlbl_socket_setattr(sock, &secattr);
if (rc == 0) {
- spin_lock(&sksec->nlbl_lock);
+ spin_lock_bh(&sksec->nlbl_lock);
sksec->nlbl_state = NLBL_LABELED;
- spin_unlock(&sksec->nlbl_lock);
+ spin_unlock_bh(&sksec->nlbl_lock);
}
netlbl_socket_setsid_return:
OpenPOWER on IntegriCloud