tipc: fix wrong return value from function tipc_node_try_addr()
authorJon Maloy <jon.maloy@ericsson.com>
Fri, 6 Jul 2018 18:10:03 +0000 (20:10 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sat, 7 Jul 2018 10:49:01 +0000 (19:49 +0900)
commit2a57f182420174c7fd4b19db979a2d135231a963
tree2995b49645166b4f744bfc775079b00b6f6fdf78
parent0f62aeec0f05e579ea9fd5c09edda48bd8bed73a
tipc: fix wrong return value from function tipc_node_try_addr()

The function for checking if there is an node address conflict is
supposed to return a suggestion for a new address if it finds a
conflict, and zero otherwise. But in case the peer being checked
is previously unknown it does instead return a "suggestion" for
the checked address itself. This results in a DSC_TRIAL_FAIL_MSG
being sent unecessarily to the peer, and sometimes makes the trial
period starting over again.

Fixes: 25b0b9c4e835 ("tipc: handle collisions of 32-bit node address hash values")
Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/node.c