lib: Remove unnecessary comparison, for linked list

In the comparison function for a linked list code was
always checking against passed in NULL's.  The comparison
function will never receive a NULL value for data from
the linklist.c code.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

diff --git a/lib/routemap.c b/lib/routemap.c
index dcaecd8085ba70a835cb3598debd5e5ba715032a..44d7185567a6d549f8c68ec07c5afba5a7bda296 100644 (file)
--- a/lib/routemap.c
+++ b/lib/routemap.c
@@ -1907,12 +1907,7 @@ route_map_get_index(struct route_map *map, const struct prefix *prefix,
 static int route_map_candidate_list_cmp(struct route_map_index *idx1,
                                        struct route_map_index *idx2)
 {
-       if (!idx1)
-               return -1;
-       if (!idx2)
-               return 1;
-
-       return (idx1->pref - idx2->pref);
+       return idx1->pref - idx2->pref;
 }
 
 /*