warning: variable 'status' is used uninitialized whenever 'if' condition is false

[yurivict]

clang-14 prints many warnings that look damaging:

util.c:239:7: warning: variable 'status' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
                if (ret < 0) {
                    ^~~~~~~
util.c:253:20: note: uninitialized use occurs here
        ret = WEXITSTATUS(status);
                          ^~~~~~
/usr/include/sys/wait.h:61:32: note: expanded from macro 'WEXITSTATUS'
#define WEXITSTATUS(x)  (_W_INT(x) >> 8)
                                ^
/usr/include/sys/wait.h:52:20: note: expanded from macro '_W_INT'
#define _W_INT(i)       (i)
                         ^
util.c:239:3: note: remove the 'if' if its condition is always true
                if (ret < 0) {
                ^~~~~~~~~~~~~
util.c:195:12: note: initialize the variable 'status' to silence this warning
        int status;
                  ^
                   = 0

[michaelolbrich]

It's a false positive because ret is always -1 here. But clang doesn't know that. We can simplify the code to avoid this warning.