ima: Log the same audit cause whenever a file has no signature
authorThiago Jung Bauermann <bauerman@linux.vnet.ibm.com>
Thu, 8 Jun 2017 01:49:12 +0000 (22:49 -0300)
committerMimi Zohar <zohar@linux.vnet.ibm.com>
Wed, 21 Jun 2017 18:37:12 +0000 (14:37 -0400)
If the file doesn't have an xattr, ima_appraise_measurement sets cause to
"missing-hash" while if there's an xattr but it's a digest instead of a
signature it sets cause to "IMA-signature-required".

Fix it by setting cause to "IMA-signature-required" in both cases.

Signed-off-by: Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com>
Signed-off-by: Mimi Zohar <zohar@linux.vnet.ibm.com>
security/integrity/ima/ima_appraise.c

index ea36a4f134f4a3332226240f2e768c1a764a438a..809ba70fbbbfb3137339895db07301b39662d5a1 100644 (file)
@@ -217,7 +217,8 @@ int ima_appraise_measurement(enum ima_hooks func,
                if (rc && rc != -ENODATA)
                        goto out;
 
-               cause = "missing-hash";
+               cause = iint->flags & IMA_DIGSIG_REQUIRED ?
+                               "IMA-signature-required" : "missing-hash";
                status = INTEGRITY_NOLABEL;
                if (opened & FILE_CREATED)
                        iint->flags |= IMA_NEW_FILE;