policydb.c: In function ‘user_read’:
policydb.c:1443:26: error: ‘buf[2]’ may be used uninitialized in this function
[-Werror=maybe-uninitialized]
usrdatum->bounds = le32_to_cpu(buf[2]);
^
cc1: all warnings being treated as errors
Which makes no sense, as :
1). A couple of lines above it is being filled (buf[2]).
2). There are other instances of this which are not triggering this
failure.
But GCC insists, and this patch stops the failures.
Signed-off-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
---
v2: Wrong placement of the buf[2]=0;
---
xen/xsm/flask/ss/policydb.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/xen/xsm/flask/ss/policydb.c b/xen/xsm/flask/ss/policydb.c
index a1060b1..30ef9c5 100644
--- a/xen/xsm/flask/ss/policydb.c
+++ b/xen/xsm/flask/ss/policydb.c
@@ -1266,6 +1266,7 @@ static int role_read(struct policydb *p, struct hashtab
*h, void *fp)
goto out;
}
+ buf[2] = 0; /* To shut up compiler warnings. */
if ( p->policyvers >= POLICYDB_VERSION_BOUNDARY )
rc = next_entry(buf, fp, sizeof(buf[0]) * 3);
else
@@ -1429,6 +1430,7 @@ static int user_read(struct policydb *p, struct hashtab
*h, void *fp)
goto out;
}
+ buf[2] = 0; /* To shut up compiler warnings. */
if ( p->policyvers >= POLICYDB_VERSION_BOUNDARY )
rc = next_entry(buf, fp, sizeof(buf[0]) * 3);
else
--
2.4.3
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
