From: Liam R. Howlett Date: Mon, 17 Apr 2023 15:15:33 +0000 (-0400) Subject: maple_tree: Fix static analyser cppcheck issue X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=3480bfb95c3db6fb5c486b85c9984b740a51b08a;p=users%2Fjedix%2Flinux-maple.git maple_tree: Fix static analyser cppcheck issue Static analyser of the maple tree code noticed that the split variable is being used to dereference into an array prior to checking the variable itself. Fix this issue by changing the order of the statement to check the variable first. Reported-by: David Binderman Signed-off-by: Liam R. Howlett --- diff --git a/lib/maple_tree.c b/lib/maple_tree.c index 110a36479dce..9cf4fca42310 100644 --- a/lib/maple_tree.c +++ b/lib/maple_tree.c @@ -1943,8 +1943,9 @@ static inline int mab_calc_split(struct ma_state *mas, * causes one node to be deficient. * NOTE: mt_min_slots is 1 based, b_end and split are zero. */ - while (((bn->pivot[split] - min) < slot_count - 1) && - (split < slot_count - 1) && (b_end - split > slot_min)) + while ((split < slot_count - 1) && + ((bn->pivot[split] - min) < slot_count - 1) && + (b_end - split > slot_min)) split++; }