diff options
| author | Kyle Moffett <mrmacman_g4@mac.com> | 2005-07-27 17:22:30 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-07-27 17:22:30 -0400 |
| commit | a77be819f94fc55627ee257f496198ad703aaad4 (patch) | |
| tree | 1b6d1a29f39ce0541e1248639adc496986261a13 /net | |
| parent | cbcd2a4cca1a6223e4f98661ef524919d322675e (diff) | |
[NET]: Fix setsockopt locking bug
On Sparc, SO_DONTLINGER support resulted in sock_reset_flag being
called without lock_sock().
Signed-off-by: Kyle Moffett <mrmacman_g4@mac.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/core/sock.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/net/core/sock.c b/net/core/sock.c index 8b35ccdc2b3b..12f6d9a2a522 100644 --- a/net/core/sock.c +++ b/net/core/sock.c | |||
| @@ -206,13 +206,14 @@ int sock_setsockopt(struct socket *sock, int level, int optname, | |||
| 206 | */ | 206 | */ |
| 207 | 207 | ||
| 208 | #ifdef SO_DONTLINGER /* Compatibility item... */ | 208 | #ifdef SO_DONTLINGER /* Compatibility item... */ |
| 209 | switch (optname) { | 209 | if (optname == SO_DONTLINGER) { |
| 210 | case SO_DONTLINGER: | 210 | lock_sock(sk); |
| 211 | sock_reset_flag(sk, SOCK_LINGER); | 211 | sock_reset_flag(sk, SOCK_LINGER); |
| 212 | return 0; | 212 | release_sock(sk); |
| 213 | return 0; | ||
| 213 | } | 214 | } |
| 214 | #endif | 215 | #endif |
| 215 | 216 | ||
| 216 | if(optlen<sizeof(int)) | 217 | if(optlen<sizeof(int)) |
| 217 | return(-EINVAL); | 218 | return(-EINVAL); |
| 218 | 219 | ||
