From aaf2ae561366d42fc3e0707a648d1b07c944444b Mon Sep 17 00:00:00 2001 From: Maria Matejka Date: Sat, 17 Aug 2019 10:20:46 +0200 Subject: [PATCH] Nest: Several assumptions to tame the static analyzer --- nest/rt-attr.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/nest/rt-attr.c b/nest/rt-attr.c index 8620d321..3f83edce 100644 --- a/nest/rt-attr.c +++ b/nest/rt-attr.c @@ -202,7 +202,7 @@ nexthop__same(struct nexthop *x, struct nexthop *y) } static int -nexthop_compare_node(const struct nexthop *x, const struct nexthop *y) +nexthop_compare_node(const struct nexthop *x, const struct nexthop *y) { int r; @@ -278,18 +278,22 @@ nexthop_merge(struct nexthop *x, struct nexthop *y, int rx, int ry, int max, lin while ((x || y) && max--) { int cmp = nexthop_compare_node(x, y); + if (cmp < 0) { + ASSUME(x); *n = rx ? x : nexthop_copy_node(x, lp); x = x->next; } else if (cmp > 0) { + ASSUME(y); *n = ry ? y : nexthop_copy_node(y, lp); y = y->next; } else { + ASSUME(x && y); *n = rx ? x : (ry ? y : nexthop_copy_node(x, lp)); x = x->next; y = y->next;