It should be >= ARRAY_SIZE() instead of > ARRAY_SIZE(). Fixes: 6231da675578 ('rcu: Print symbolic name for ->gp_state') Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c index 6a5e1c4..6559b3c 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c @@ -1203,7 +1203,7 @@ static void record_gp_stall_check_time(struct rcu_state *rsp) */ static char *gp_state_getname(short gs) { - if (gs < 0 || gs > ARRAY_SIZE(gp_state_names)) + if (gs < 0 || gs >= ARRAY_SIZE(gp_state_names)) return "???"; return gp_state_names[gs]; } -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html