From 12337cf2061e4ea544cc2cd7cd5ad4efb6ddf50a Mon Sep 17 00:00:00 2001 From: Kevin Traynor Date: Mon, 20 Jul 2020 11:19:08 +0100 Subject: [PATCH] test/eal: check invalid CPU value When using --lcores option, there is a limit of CPU_SETSIZE. Currently that allows 0..1023 on Linux. Check it is caught when this limit is exceeded. Signed-off-by: Kevin Traynor Acked-by: Lukasz Wojciechowski --- app/test/test_eal_flags.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 4ee809e3db..b019656b29 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -527,6 +527,9 @@ test_missing_c_flag(void) const char * const argv29[] = { prgname, prefix, mp_flag, "--lcores", "0-1,2@(5-7),(3-5)@(0,2),(0,6),7"}; + /* check an invalid cpu value >= CPU_SETSIZE */ + const char * const argv30[] = { prgname, prefix, mp_flag, + "--lcores", "3@" RTE_STR(CPU_SETSIZE) }; if (launch_proc(argv2) != 0) { printf("Error - " @@ -576,7 +579,7 @@ test_missing_c_flag(void) launch_proc(argv22) == 0 || launch_proc(argv23) == 0 || launch_proc(argv24) == 0 || launch_proc(argv25) == 0 || launch_proc(argv26) == 0 || launch_proc(argv27) == 0 || - launch_proc(argv28) == 0) { + launch_proc(argv28) == 0 || launch_proc(argv30) == 0) { printf("Error - " "process ran without error with invalid --lcore flag\n"); return -1; -- 2.20.1