diff options
| author | David S. Miller <davem@davemloft.net> | 2012-06-12 02:52:29 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-06-12 02:52:29 +0400 |
| commit | 55afabaa0df0dd139c8796a71beb43d1216fbe43 (patch) | |
| tree | 5f3ad3d76d9101c11d15197b40fb4f86402db3a9 /include/net/route.h | |
| parent | 352e04b9111d608bd89ba7bd8070846d4f97d104 (diff) | |
| download | linux-55afabaa0df0dd139c8796a71beb43d1216fbe43.tar.xz | |
inet: Fix BUG triggered by __rt{,6}_get_peer().
If no peer actually gets attached (either because create is zero or
the peer allocation fails) we'll trigger a BUG because we
unconditionally do an rt{,6}_peer_ptr() afterwards.
Fix this by guarding it with the proper check.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/route.h')
| -rw-r--r-- | include/net/route.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/net/route.h b/include/net/route.h index cc693a5bb20d..2bfbc9329ea9 100644 --- a/include/net/route.h +++ b/include/net/route.h @@ -334,7 +334,7 @@ static inline struct inet_peer *__rt_get_peer(struct rtable *rt, __be32 daddr, i return rt_peer_ptr(rt); rt_bind_peer(rt, daddr, create); - return rt_peer_ptr(rt); + return (rt_has_peer(rt) ? rt_peer_ptr(rt) : NULL); } static inline struct inet_peer *rt_get_peer(struct rtable *rt, __be32 daddr) |