tree 842806138758343d41598fded61943de51c4c363
parent 2d362d1099f346202cb628665ff4ae563b2594f5
author Stephen Hemminger <shemminger@vyatta.com> 1261410643 +0300
committer Denis Ovsienko <infrastation@yandex.ru> 1261410643 +0300

zebra: change router-id selection algo

The router-id table looks like is supposed to be sorted in current
quagga code, but the nodes are not added with the sorting
function.

The sorting function is host byte order dependent.
The values need to converted before comparison.

Fixing this causes Zebra to choose the largest IP address
as router-id, rather than the last address. This probably will
surprise some users. The other option would be to just remove the
comparison function and keep the existing LIFO behavior.

Lastly, simple subtraction works well for comparing.

* zebra/router-id.c
  * router_id_add_address(): employ listnode_add_sort()
  * router_id_cmp(): employ ntohl(), then compare integers