batman-adv: Drop immediate batadv_orig_ifinfo free function
authorSven Eckelmann <sven@narfation.org>
Tue, 5 Jan 2016 11:06:21 +0000 (12:06 +0100)
committerAntonio Quartulli <a@unstable.cc>
Sat, 16 Jan 2016 14:49:05 +0000 (22:49 +0800)
commitdeed96605f5695cb945e0b3d79429581857a2b9d
tree24e746e060c835f9bd65e3b2b0b542c3b1a19d3d
parent44e8e7e91d6c7c7ab19688750f7257292640d1a0
batman-adv: Drop immediate batadv_orig_ifinfo free function

It is not allowed to free the memory of an object which is part of a list
which is protected by rcu-read-side-critical sections without making sure
that no other context is accessing the object anymore. This usually happens
by removing the references to this object and then waiting until the rcu
grace period is over and no one (allowedly) accesses it anymore.

But the _now functions ignore this completely. They free the object
directly even when a different context still tries to access it. This has
to be avoided and thus these functions must be removed and all functions
have to use batadv_orig_ifinfo_free_ref.

Fixes: 7351a4822d42 ("batman-adv: split out router from orig_node")
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
Signed-off-by: Antonio Quartulli <a@unstable.cc>
net/batman-adv/originator.c