Since commit 58962eb3d847 ("libsepol,checkpolicy: add binary module
support for xperms") function display_avrule() has been using its "p"
argument even though it was previously marked unused. This makes clang
report:
policydb.c:3276:7: error: 'p' was marked unused but was used
[-Werror,-Wused-but-marked-unused].
if (p->policyvers < MOD_POLICYDB_VERSION_XPERMS_IOCTL) {
^
Remove the attribute to make the code consistent again.
Signed-off-by: Nicolas Iooss <nicolas.iooss@xxxxxxx>
---
libsepol/src/policydb.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/libsepol/src/policydb.c b/libsepol/src/policydb.c
index 069eb7e66540..b1530955aa39 100644
--- a/libsepol/src/policydb.c
+++ b/libsepol/src/policydb.c
@@ -3212,8 +3212,7 @@ common_read, class_read, role_read, type_read, user_read,
/************** module reading functions below **************/
-static avrule_t *avrule_read(policydb_t * p
- __attribute__ ((unused)), struct policy_file *fp)
+static avrule_t *avrule_read(policydb_t * p, struct policy_file *fp)
{
unsigned int i;
uint32_t buf[2], len;
--
2.13.0
