diff options
| author | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-01-21 22:45:49 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-01-21 22:45:49 -0500 |
| commit | 8b85eaadd5b8d3786996bd74c73aff54a92ec456 (patch) | |
| tree | 7ef6ed3e5955a45d54b1d223e3ebf7749aa3b918 /security/selinux/netlabel.c | |
| parent | f290fc3669d659a915e29b6bdb82d454b437cf93 (diff) | |
| parent | 45c950e0f839fded922ebc0bfd59b1081cc71b70 (diff) | |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/selinux-2.6
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/selinux-2.6:
selinux: fix memory leak in netlabel code
Diffstat (limited to 'security/selinux/netlabel.c')
| -rw-r--r-- | security/selinux/netlabel.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/security/selinux/netlabel.c b/security/selinux/netlabel.c index d243ddc723a5..66e013d6f6f6 100644 --- a/security/selinux/netlabel.c +++ b/security/selinux/netlabel.c | |||
| @@ -53,10 +53,11 @@ static int selinux_netlbl_sock_setsid(struct sock *sk, u32 sid) | |||
| 53 | struct sk_security_struct *sksec = sk->sk_security; | 53 | struct sk_security_struct *sksec = sk->sk_security; |
| 54 | struct netlbl_lsm_secattr secattr; | 54 | struct netlbl_lsm_secattr secattr; |
| 55 | 55 | ||
| 56 | netlbl_secattr_init(&secattr); | ||
| 57 | |||
| 56 | rc = security_netlbl_sid_to_secattr(sid, &secattr); | 58 | rc = security_netlbl_sid_to_secattr(sid, &secattr); |
| 57 | if (rc != 0) | 59 | if (rc != 0) |
| 58 | return rc; | 60 | goto sock_setsid_return; |
| 59 | |||
| 60 | rc = netlbl_sock_setattr(sk, &secattr); | 61 | rc = netlbl_sock_setattr(sk, &secattr); |
| 61 | if (rc == 0) { | 62 | if (rc == 0) { |
| 62 | spin_lock_bh(&sksec->nlbl_lock); | 63 | spin_lock_bh(&sksec->nlbl_lock); |
| @@ -64,6 +65,8 @@ static int selinux_netlbl_sock_setsid(struct sock *sk, u32 sid) | |||
| 64 | spin_unlock_bh(&sksec->nlbl_lock); | 65 | spin_unlock_bh(&sksec->nlbl_lock); |
| 65 | } | 66 | } |
| 66 | 67 | ||
| 68 | sock_setsid_return: | ||
| 69 | netlbl_secattr_destroy(&secattr); | ||
| 67 | return rc; | 70 | return rc; |
| 68 | } | 71 | } |
| 69 | 72 | ||
