Commit e40cf353 authored by YOSHIFUJI Hideaki's avatar YOSHIFUJI Hideaki Committed by David S. Miller

[IPV6]: ROUTE: Convert rt6_cow() to rt6_alloc_cow().

Signed-off-by: default avatarYOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent fb9de91e
...@@ -397,10 +397,6 @@ int ip6_ins_rt(struct rt6_info *rt, struct nlmsghdr *nlh, ...@@ -397,10 +397,6 @@ int ip6_ins_rt(struct rt6_info *rt, struct nlmsghdr *nlh,
return err; return err;
} }
/* No rt6_lock! If COW failed, the function returns dead route entry
with dst->error set to errno value.
*/
static struct rt6_info *rt6_alloc_cow(struct rt6_info *ort, struct in6_addr *daddr, static struct rt6_info *rt6_alloc_cow(struct rt6_info *ort, struct in6_addr *daddr,
struct in6_addr *saddr) struct in6_addr *saddr)
{ {
...@@ -439,26 +435,6 @@ static struct rt6_info *rt6_alloc_cow(struct rt6_info *ort, struct in6_addr *dad ...@@ -439,26 +435,6 @@ static struct rt6_info *rt6_alloc_cow(struct rt6_info *ort, struct in6_addr *dad
return rt; return rt;
} }
static struct rt6_info *rt6_cow(struct rt6_info *ort, struct in6_addr *daddr,
struct in6_addr *saddr, struct netlink_skb_parms *req)
{
struct rt6_info *rt = rt6_alloc_cow(ort, daddr, saddr);
int err;
if (!rt) {
dst_hold(&ip6_null_entry.u.dst);
return &ip6_null_entry;
}
dst_hold(&rt->u.dst);
err = ip6_ins_rt(rt, NULL, NULL, req);
if (err)
rt->u.dst.error = err;
return rt;
}
static struct rt6_info *rt6_alloc_clone(struct rt6_info *ort, struct in6_addr *daddr) static struct rt6_info *rt6_alloc_clone(struct rt6_info *ort, struct in6_addr *daddr)
{ {
struct rt6_info *rt = ip6_rt_copy(ort); struct rt6_info *rt = ip6_rt_copy(ort);
...@@ -518,15 +494,23 @@ void ip6_route_input(struct sk_buff *skb) ...@@ -518,15 +494,23 @@ void ip6_route_input(struct sk_buff *skb)
if (!rt->rt6i_nexthop && !(rt->rt6i_flags & RTF_NONEXTHOP)) { if (!rt->rt6i_nexthop && !(rt->rt6i_flags & RTF_NONEXTHOP)) {
struct rt6_info *nrt; struct rt6_info *nrt;
int err;
nrt = rt6_cow(rt, &skb->nh.ipv6h->daddr, nrt = rt6_alloc_cow(rt, &skb->nh.ipv6h->daddr,
&skb->nh.ipv6h->saddr, &skb->nh.ipv6h->saddr);
&NETLINK_CB(skb));
dst_release(&rt->u.dst); dst_release(&rt->u.dst);
rt = nrt; rt = nrt ? : &ip6_null_entry;
dst_hold(&rt->u.dst);
if (nrt) {
err = ip6_ins_rt(nrt, NULL, NULL,
&NETLINK_CB(skb));
if (!err)
goto out2;
}
if (rt->u.dst.error != -EEXIST || --attempts <= 0) if (--attempts <= 0)
goto out2; goto out2;
/* Race condition! In the gap, when rt6_lock was /* Race condition! In the gap, when rt6_lock was
...@@ -582,13 +566,21 @@ struct dst_entry * ip6_route_output(struct sock *sk, struct flowi *fl) ...@@ -582,13 +566,21 @@ struct dst_entry * ip6_route_output(struct sock *sk, struct flowi *fl)
if (!rt->rt6i_nexthop && !(rt->rt6i_flags & RTF_NONEXTHOP)) { if (!rt->rt6i_nexthop && !(rt->rt6i_flags & RTF_NONEXTHOP)) {
struct rt6_info *nrt; struct rt6_info *nrt;
int err;
nrt = rt6_cow(rt, &fl->fl6_dst, &fl->fl6_src, NULL); nrt = rt6_alloc_cow(rt, &fl->fl6_dst, &fl->fl6_src);
dst_release(&rt->u.dst); dst_release(&rt->u.dst);
rt = nrt; rt = nrt ? : &ip6_null_entry;
if (rt->u.dst.error != -EEXIST || --attempts <= 0) dst_hold(&rt->u.dst);
if (nrt) {
err = ip6_ins_rt(nrt, NULL, NULL, NULL);
if (!err)
goto out2;
}
if (--attempts <= 0)
goto out2; goto out2;
/* Race condition! In the gap, when rt6_lock was /* Race condition! In the gap, when rt6_lock was
...@@ -597,6 +589,7 @@ struct dst_entry * ip6_route_output(struct sock *sk, struct flowi *fl) ...@@ -597,6 +589,7 @@ struct dst_entry * ip6_route_output(struct sock *sk, struct flowi *fl)
dst_release(&rt->u.dst); dst_release(&rt->u.dst);
goto relookup; goto relookup;
} }
out2: out2:
rt->u.dst.lastuse = jiffies; rt->u.dst.lastuse = jiffies;
rt->u.dst.__use++; rt->u.dst.__use++;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment