app/testpmd: guarantee port array access in range
authorFerruh Yigit <ferruh.yigit@intel.com>
Wed, 19 Feb 2020 12:40:03 +0000 (12:40 +0000)
committerFerruh Yigit <ferruh.yigit@intel.com>
Wed, 19 Feb 2020 14:40:24 +0000 (15:40 +0100)
commit5e4154cec49cb30c82c19c13fb62827e13563019
treeb2779222d4b4772a8e24ccac596b573e905afc8a
parent5693233823b09b527376e94ce24df71918632dfa
app/testpmd: guarantee port array access in range

Coverity complains about out of bound access, which is a false positive.

The return value of the 'parse_port_list()' can't be bigger than
'maxsize' because of the logic in the function. ('value >= (int)maxsize'
check and 'marked[]' usage.)

But this is not explicitly clear, causing coverity warning and same
question can be rise by reviews later.

Adding a redundant check to highlight the access is in range, this is
done by replacing existing redundant check.

This is also good to protect against out out bound access in case
'parse_port_list()' behaviour changes later unexpectedly.

Coverity issue: 354229
Fixes: 2df00d562d20 ("app/testpmd: add --portlist option")

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Reviewed-by: Herakliusz Lipiec <herakliusz.lipiec@intel.com>
Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
app/test-pmd/config.c