Additionally, this commit breaks 'make check' donald
On Wed, Nov 25, 2015 at 7:06 PM, Donald Sharp <sha...@cumulusnetworks.com> wrote: > One comment: > > On Wed, Nov 25, 2015 at 12:14 PM, Paul Jakma <paul.ja...@hpe.com> wrote: > >> >> @@ -552,32 +543,32 @@ bgp_info_cmp (struct bgp *bgp, struct bgp_info >> *new, struct bgp_info *exist, >> exist_cluster = existattre->cluster->length; >> >> if (new_cluster < exist_cluster) >> - return 1; >> + return -1; >> if (new_cluster > exist_cluster) >> - return 0; >> + return 1; >> >> /* 13. Neighbor address comparision. */ >> /* Do this only if neither path is "stale" as stale paths do not have >> * valid peer information (as the connection may or may not be up). >> */ >> if (CHECK_FLAG (exist->flags, BGP_INFO_STALE)) >> - return 1; >> + return -1; >> if (CHECK_FLAG (new->flags, BGP_INFO_STALE)) >> - return 0; >> + return 1; >> /* locally configured routes to advertise do not have su_remote */ >> if (new->peer->su_remote == NULL) >> - return 0; >> - if (exist->peer->su_remote == NULL) >> return 1; >> + if (exist->peer->su_remote == NULL) >> + return -1; >> >> ret = sockunion_cmp (new->peer->su_remote, exist->peer->su_remote); >> - >> + >> if (ret == 1) >> - return 0; >> - if (ret == -1) >> return 1; >> + if (ret == -1) >> + return -1; >> >> - return 1; >> + return -1; >> } >> >> > > Shouldn't this return -1 be a return 0? > > We are at the end of the comparisons and there was no resolution therefore > they are equal? > > donald >
_______________________________________________ Quagga-dev mailing list Quagga-dev@lists.quagga.net https://lists.quagga.net/mailman/listinfo/quagga-dev