batman-adv: Fix NULL pointer dereference in DAT hash collision avoidance
authorPau Koning <paukoning@gmail.com>
Tue, 12 Feb 2013 00:18:45 +0000 (00:18 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 13 Feb 2013 18:35:24 +0000 (13:35 -0500)
commit816cd5b83e4d8f3c8106966e64a025408caee3f6
tree3d784959629752b71d77e4504472189f8972d9cb
parent8770e91aa61127f82eb8a9e641832474dbbcc8f1
batman-adv: Fix NULL pointer dereference in DAT hash collision avoidance

An entry in DAT with the hashed position of 0 can cause a NULL pointer
dereference when the first entry is checked by batadv_choose_next_candidate.
This first candidate automatically has the max value of 0 and the max_orig_node
of NULL. Not checking max_orig_node for NULL in batadv_is_orig_node_eligible
will lead to a NULL pointer dereference when checking for the lowest address.

This problem was added in 785ea1144182c341b8b85b0f8180291839d176a8
("batman-adv: Distributed ARP Table - create DHT helper functions").

Signed-off-by: Pau Koning <paukoning@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/batman-adv/distributed-arp-table.c