]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
CLEANUP: ebmbtree: Replace always-taken elseif by else
authorTim Düsterhus <tim@bastelstu.be>
Sat, 11 Sep 2021 15:02:33 +0000 (17:02 +0200)
committerWilly Tarreau <w@1wt.eu>
Sat, 11 Sep 2021 18:15:28 +0000 (20:15 +0200)
`diff` is guaranteed to be less than 0, because the `if` handles the `>= 0`
case.

Found using GitHub's CodeQL scan in HAProxy's codebase.

include/import/ebmbtree.h

index f99c16ba9a4b1d4ea58fa751d18f25ac083a7d82..1a454262475cbc6d0b7be8f21195da92c0301677 100644 (file)
@@ -358,7 +358,7 @@ __ebmb_insert(struct eb_root *root, struct ebmb_node *new, unsigned int len)
                new->node.leaf_p = new_rght;
                *up_ptr = new_left;
        }
-       else if (diff < 0) {
+       else {
                new->node.branches.b[EB_LEFT] = new_leaf;
                new->node.branches.b[EB_RGHT] = troot;
                new->node.leaf_p = new_left;