decnet: always not take dst->__refcnt when inserting dst into hash table
authorWei Wang <weiwan@google.com>
Fri, 16 Jun 2017 17:46:37 +0000 (10:46 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 16 Jun 2017 19:00:00 +0000 (15:00 -0400)
commit247488c0a490214a0b5227a678ec2d46b7dba4e5
tree6d59ff6f4db686fb2b0902db8e43c2ca2346dee7
parent34a95132e0e44bd2e0235ee90f8c3eafe90f269d
decnet: always not take dst->__refcnt when inserting dst into hash table

In the existing dn_route.c code, dn_route_output_slow() takes
dst->__refcnt before calling dn_insert_route() while dn_route_input_slow()
does not take dst->__refcnt before calling dn_insert_route().
This makes the whole routing code very buggy.
In dn_dst_check_expire(), dnrt_free() is called when rt expires. This
makes the routes inserted by dn_route_output_slow() not able to be
freed as the refcnt is not released.
In dn_dst_gc(), dnrt_drop() is called to release rt which could
potentially cause the dst->__refcnt to be dropped to -1.
In dn_run_flush(), dst_free() is called to release all the dst. Again,
it makes the dst inserted by dn_route_output_slow() not able to be
released and also, it does not wait on the rcu and could potentially
cause crash in the path where other users still refer to this dst.

This patch makes sure both input and output path do not take
dst->__refcnt before calling dn_insert_route() and also makes sure
dnrt_free()/dst_free() is called when removing dst from the hash table.
The only difference between those 2 calls is that dnrt_free() waits on
the rcu while dst_free() does not.

Signed-off-by: Wei Wang <weiwan@google.com>
Acked-by: Martin KaFai Lau <kafai@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/decnet/dn_route.c