linux-6.17/arch/mips/kernel/kgdb.c: 2 * Array sanity check in wrong place ?

From: David Binderman

Date: Tue Sep 30 2025 - 06:02:05 EST


Hello there,

Static analyser cppcheck said:

1.

linux-6.17/arch/mips/kernel/kgdb.c:137:17: style: Array index 'regno' is used before limits check. [arrayIndexThenCheck]

Source code is

if (dbg_reg_def[regno].offset != -1 && regno < 38) {

Maybe better code:

if (regno < 38 && dbg_reg_def[regno].offset != -1) {

2.

linux-6.17/arch/mips/kernel/kgdb.c:170:17: style: Array index 'regno' is used before limits check. [arrayIndexThenCheck]

Duplicate.

Regards
David Binderman