When running "rv mon" on a monitor that is already enabled, the tool fails to start but incorrectly exits with a success status (0).
Fix the exit condition to ensure it returns a failure code on any execution error. Signed-off-by: Gabriele Monaco <[email protected]> --- tools/verification/rv/src/rv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/verification/rv/src/rv.c b/tools/verification/rv/src/rv.c index b8fe24a87d97..6d65f4037581 100644 --- a/tools/verification/rv/src/rv.c +++ b/tools/verification/rv/src/rv.c @@ -127,7 +127,7 @@ static void rv_mon(int argc, char **argv) if (!run) err_msg("rv: monitor %s does not exist\n", monitor_name); - exit(!run); + exit(run <= 0); } static void usage(int exit_val, const char *fmt, ...) -- 2.54.0
