From bfb7361980977c586980b9de9ff0e0ef5ddb51f0 Mon Sep 17 00:00:00 2001
From: Craig Small <csmall@dropbear.xyz>
Date: Sat, 14 May 2016 22:55:16 +1000
Subject: [PATCH] library: oldproc_open has always yes check

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>
---
 proc/pids.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proc/pids.c b/proc/pids.c
index cc3a4ac9..b7a88ebf 100644
--- a/proc/pids.c
+++ b/proc/pids.c
@@ -977,7 +977,7 @@ static inline int oldproc_open (
     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;