projects
/
haproxy-2.5.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
d5fc8fc
)
CLEANUP: ebmbtree: Replace always-taken elseif by else
author
Tim Düsterhus
<tim@bastelstu.be>
Sat, 11 Sep 2021 15:02:33 +0000
(17:02 +0200)
committer
Willy 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
patch
|
blob
|
history
diff --git
a/include/import/ebmbtree.h
b/include/import/ebmbtree.h
index
f99c16b
..
1a45426
100644
(file)
--- a/
include/import/ebmbtree.h
+++ b/
include/import/ebmbtree.h
@@
-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;