diff options
author | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-27 11:47:29 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-27 11:47:29 -0500 |
commit | fdccffc6b711d1bfcd1e1b569537686dcad423e2 (patch) | |
tree | 51a4cdfad5ed6ea0fcec697f0f8e53a1d33ef326 | |
parent | 4fa639123d9e6e8dfaa3d116368f4b2478da31af (diff) | |
parent | 3eb4801d7bde42b82f05137392a1ee0ece090bad (diff) |
Merge master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6
* master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6:
[NET]: drop duplicate assignment in request_sock
[IPSEC]: Fix tunnel error handling in ipcomp6
-rw-r--r-- | include/net/request_sock.h | 2 | ||||
-rw-r--r-- | net/core/request_sock.c | 2 | ||||
-rw-r--r-- | net/ipv6/ipcomp6.c | 5 |
3 files changed, 7 insertions, 2 deletions
diff --git a/include/net/request_sock.h b/include/net/request_sock.h index 11641c9384f7..c5d7f920c352 100644 --- a/include/net/request_sock.h +++ b/include/net/request_sock.h | |||
@@ -145,7 +145,7 @@ static inline struct request_sock * | |||
145 | { | 145 | { |
146 | struct request_sock *req = queue->rskq_accept_head; | 146 | struct request_sock *req = queue->rskq_accept_head; |
147 | 147 | ||
148 | queue->rskq_accept_head = queue->rskq_accept_head = NULL; | 148 | queue->rskq_accept_head = NULL; |
149 | return req; | 149 | return req; |
150 | } | 150 | } |
151 | 151 | ||
diff --git a/net/core/request_sock.c b/net/core/request_sock.c index 98f0fc923f91..1e44eda1fda9 100644 --- a/net/core/request_sock.c +++ b/net/core/request_sock.c | |||
@@ -51,7 +51,7 @@ int reqsk_queue_alloc(struct request_sock_queue *queue, | |||
51 | 51 | ||
52 | get_random_bytes(&lopt->hash_rnd, sizeof(lopt->hash_rnd)); | 52 | get_random_bytes(&lopt->hash_rnd, sizeof(lopt->hash_rnd)); |
53 | rwlock_init(&queue->syn_wait_lock); | 53 | rwlock_init(&queue->syn_wait_lock); |
54 | queue->rskq_accept_head = queue->rskq_accept_head = NULL; | 54 | queue->rskq_accept_head = NULL; |
55 | lopt->nr_table_entries = nr_table_entries; | 55 | lopt->nr_table_entries = nr_table_entries; |
56 | 56 | ||
57 | write_lock_bh(&queue->syn_wait_lock); | 57 | write_lock_bh(&queue->syn_wait_lock); |
diff --git a/net/ipv6/ipcomp6.c b/net/ipv6/ipcomp6.c index 028b636687ec..d4cfec3f414e 100644 --- a/net/ipv6/ipcomp6.c +++ b/net/ipv6/ipcomp6.c | |||
@@ -228,6 +228,9 @@ static struct xfrm_state *ipcomp6_tunnel_create(struct xfrm_state *x) | |||
228 | 228 | ||
229 | t->id.proto = IPPROTO_IPV6; | 229 | t->id.proto = IPPROTO_IPV6; |
230 | t->id.spi = xfrm6_tunnel_alloc_spi((xfrm_address_t *)&x->props.saddr); | 230 | t->id.spi = xfrm6_tunnel_alloc_spi((xfrm_address_t *)&x->props.saddr); |
231 | if (!t->id.spi) | ||
232 | goto error; | ||
233 | |||
231 | memcpy(t->id.daddr.a6, x->id.daddr.a6, sizeof(struct in6_addr)); | 234 | memcpy(t->id.daddr.a6, x->id.daddr.a6, sizeof(struct in6_addr)); |
232 | memcpy(&t->sel, &x->sel, sizeof(t->sel)); | 235 | memcpy(&t->sel, &x->sel, sizeof(t->sel)); |
233 | t->props.family = AF_INET6; | 236 | t->props.family = AF_INET6; |
@@ -243,7 +246,9 @@ out: | |||
243 | return t; | 246 | return t; |
244 | 247 | ||
245 | error: | 248 | error: |
249 | t->km.state = XFRM_STATE_DEAD; | ||
246 | xfrm_state_put(t); | 250 | xfrm_state_put(t); |
251 | t = NULL; | ||
247 | goto out; | 252 | goto out; |
248 | } | 253 | } |
249 | 254 | ||