Skip to content

Commit c0efc88

Browse files
committed
inet: Pass inetpeer root into inet_getpeer*() interfaces.
Otherwise we reference potentially non-existing members when ipv6 is disabled. Signed-off-by: David S. Miller <[email protected]>
1 parent 8b51b06 commit c0efc88

File tree

5 files changed

+11
-18
lines changed

5 files changed

+11
-18
lines changed

include/net/inetpeer.h

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -83,30 +83,30 @@ static inline bool inet_metrics_new(const struct inet_peer *p)
8383
}
8484

8585
/* can be called with or without local BH being disabled */
86-
struct inet_peer *inet_getpeer(struct net *net,
86+
struct inet_peer *inet_getpeer(struct inet_peer_base *base,
8787
const struct inetpeer_addr *daddr,
8888
int create);
8989

90-
static inline struct inet_peer *inet_getpeer_v4(struct net *net,
90+
static inline struct inet_peer *inet_getpeer_v4(struct inet_peer_base *base,
9191
__be32 v4daddr,
9292
int create)
9393
{
9494
struct inetpeer_addr daddr;
9595

9696
daddr.addr.a4 = v4daddr;
9797
daddr.family = AF_INET;
98-
return inet_getpeer(net, &daddr, create);
98+
return inet_getpeer(base, &daddr, create);
9999
}
100100

101-
static inline struct inet_peer *inet_getpeer_v6(struct net *net,
101+
static inline struct inet_peer *inet_getpeer_v6(struct inet_peer_base *base,
102102
const struct in6_addr *v6daddr,
103103
int create)
104104
{
105105
struct inetpeer_addr daddr;
106106

107107
*(struct in6_addr *)daddr.addr.a6 = *v6daddr;
108108
daddr.family = AF_INET6;
109-
return inet_getpeer(net, &daddr, create);
109+
return inet_getpeer(base, &daddr, create);
110110
}
111111

112112
/* can be called from BH context or outside */

net/ipv4/inetpeer.c

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -391,12 +391,6 @@ static void unlink_from_pool(struct inet_peer *p, struct inet_peer_base *base,
391391
call_rcu(&p->rcu, inetpeer_free_rcu);
392392
}
393393

394-
static struct inet_peer_base *family_to_base(struct net *net,
395-
int family)
396-
{
397-
return family == AF_INET ? net->ipv4.peers : net->ipv6.peers;
398-
}
399-
400394
/* perform garbage collect on all items stacked during a lookup */
401395
static int inet_peer_gc(struct inet_peer_base *base,
402396
struct inet_peer __rcu **stack[PEER_MAXDEPTH],
@@ -434,12 +428,11 @@ static int inet_peer_gc(struct inet_peer_base *base,
434428
return cnt;
435429
}
436430

437-
struct inet_peer *inet_getpeer(struct net *net,
431+
struct inet_peer *inet_getpeer(struct inet_peer_base *base,
438432
const struct inetpeer_addr *daddr,
439433
int create)
440434
{
441435
struct inet_peer __rcu **stack[PEER_MAXDEPTH], ***stackptr;
442-
struct inet_peer_base *base = family_to_base(net, daddr->family);
443436
struct inet_peer *p;
444437
unsigned int sequence;
445438
int invalidated, gccnt = 0;

net/ipv4/ip_fragment.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ static void ip4_frag_init(struct inet_frag_queue *q, void *a)
184184
qp->daddr = arg->iph->daddr;
185185
qp->user = arg->user;
186186
qp->peer = sysctl_ipfrag_max_dist ?
187-
inet_getpeer_v4(net, arg->iph->saddr, 1) : NULL;
187+
inet_getpeer_v4(net->ipv4.peers, arg->iph->saddr, 1) : NULL;
188188
}
189189

190190
static __inline__ void ip4_frag_free(struct inet_frag_queue *q)

net/ipv4/route.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1328,7 +1328,7 @@ void rt_bind_peer(struct rtable *rt, __be32 daddr, int create)
13281328
struct net *net = dev_net(rt->dst.dev);
13291329
struct inet_peer *peer;
13301330

1331-
peer = inet_getpeer_v4(net, daddr, create);
1331+
peer = inet_getpeer_v4(net->ipv4.peers, daddr, create);
13321332

13331333
if (peer && cmpxchg(&rt->peer, NULL, peer) != NULL)
13341334
inet_putpeer(peer);
@@ -1684,7 +1684,7 @@ unsigned short ip_rt_frag_needed(struct net *net, const struct iphdr *iph,
16841684
unsigned short est_mtu = 0;
16851685
struct inet_peer *peer;
16861686

1687-
peer = inet_getpeer_v4(net, iph->daddr, 1);
1687+
peer = inet_getpeer_v4(net->ipv4.peers, iph->daddr, 1);
16881688
if (peer) {
16891689
unsigned short mtu = new_mtu;
16901690

@@ -1929,7 +1929,7 @@ static void rt_init_metrics(struct rtable *rt, const struct flowi4 *fl4,
19291929
if (fl4 && (fl4->flowi4_flags & FLOWI_FLAG_PRECOW_METRICS))
19301930
create = 1;
19311931

1932-
rt->peer = peer = inet_getpeer_v4(net, rt->rt_dst, create);
1932+
rt->peer = peer = inet_getpeer_v4(net->ipv4.peers, rt->rt_dst, create);
19331933
if (peer) {
19341934
rt->rt_peer_genid = rt_peer_genid();
19351935
if (inet_metrics_new(peer))

net/ipv6/route.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -306,7 +306,7 @@ void rt6_bind_peer(struct rt6_info *rt, int create)
306306
struct net *net = dev_net(rt->dst.dev);
307307
struct inet_peer *peer;
308308

309-
peer = inet_getpeer_v6(net, &rt->rt6i_dst.addr, create);
309+
peer = inet_getpeer_v6(net->ipv6.peers, &rt->rt6i_dst.addr, create);
310310
if (peer && cmpxchg(&rt->rt6i_peer, NULL, peer) != NULL)
311311
inet_putpeer(peer);
312312
else

0 commit comments

Comments
 (0)