diff options
| -rw-r--r-- | net/ipv4/route.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/net/ipv4/route.c b/net/ipv4/route.c index cd76b3cb7092..65b3a8b11a6c 100644 --- a/net/ipv4/route.c +++ b/net/ipv4/route.c | |||
| @@ -1085,8 +1085,16 @@ restart: | |||
| 1085 | now = jiffies; | 1085 | now = jiffies; |
| 1086 | 1086 | ||
| 1087 | if (!rt_caching(dev_net(rt->u.dst.dev))) { | 1087 | if (!rt_caching(dev_net(rt->u.dst.dev))) { |
| 1088 | rt_drop(rt); | 1088 | /* |
| 1089 | return 0; | 1089 | * If we're not caching, just tell the caller we |
| 1090 | * were successful and don't touch the route. The | ||
| 1091 | * caller hold the sole reference to the cache entry, and | ||
| 1092 | * it will be released when the caller is done with it. | ||
| 1093 | * If we drop it here, the callers have no way to resolve routes | ||
| 1094 | * when we're not caching. Instead, just point *rp at rt, so | ||
| 1095 | * the caller gets a single use out of the route | ||
| 1096 | */ | ||
| 1097 | goto report_and_exit; | ||
| 1090 | } | 1098 | } |
| 1091 | 1099 | ||
| 1092 | rthp = &rt_hash_table[hash].chain; | 1100 | rthp = &rt_hash_table[hash].chain; |
| @@ -1217,6 +1225,8 @@ restart: | |||
| 1217 | rcu_assign_pointer(rt_hash_table[hash].chain, rt); | 1225 | rcu_assign_pointer(rt_hash_table[hash].chain, rt); |
| 1218 | 1226 | ||
| 1219 | spin_unlock_bh(rt_hash_lock_addr(hash)); | 1227 | spin_unlock_bh(rt_hash_lock_addr(hash)); |
| 1228 | |||
| 1229 | report_and_exit: | ||
| 1220 | if (rp) | 1230 | if (rp) |
| 1221 | *rp = rt; | 1231 | *rp = rt; |
| 1222 | else | 1232 | else |
