[v5,3/3] app/test: fix return value of test_kni_link_change
Checks
Commit Message
If the child process returns -1, WEXITSTATUS converts to 255 because it
takes only 8 least significant bits. As a result, the test is not
considered failed when it should.
Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
---
app/test/test_kni.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
@@ -214,7 +214,7 @@ test_kni_link_change(void)
p_ret = waitpid(pid, &status, WNOHANG);
if (p_ret != 0) {
if (WIFEXITED(status))
- return WEXITSTATUS(status);
+ return WEXITSTATUS(status) ? -1 : 0;
return -1;
}
rte_delay_ms(10);