if (info->flags | PROC_UID)
Something OR a non-zero constant is always true.
Looks like it should be and'ed for the standard flag masking
pattern.
References:
Coverity #99118
Signed-off-by: Craig Small <csmall@dropbear.xyz>
if (info->PT == NULL) {
va_start(vl, supp_flgs);
ids = va_arg(vl, int*);
- if (info->flags | PROC_UID) num = va_arg(vl, int);
+ if (info->flags & PROC_UID) num = va_arg(vl, int);
va_end(vl);
if (NULL == (info->PT = openproc(info->flags | supp_flgs, ids, num)))
return 0;