fix(iam_policy_no_administrative_privileges): check only *:* permissions (#1802)

This commit is contained in:
Sergio Garcia authored and sergargar committed 2023-02-03 14:36:14 +01:00
1 parent 45196d1e97
commit 99445818f9
2 files changed
+4 -4

No files matched your search

@@ -18,15 +18,15 @@ class iam_policy_no_administrative_privileges(Check):
else:
policy_statements = policy["PolicyDocument"]["Statement"]
for statement in policy_statements:
# Check policies with "Effect": "Allow" with "Action": "*" over "Resource": "*".
if (
statement["Effect"] == "Allow"
and "Action" in statement
and "*" in statement["Action"]
and "*" in statement["Resource"]
and (statement["Action"] == "*" or statement["Action"] == ["*"])
and (statement["Resource"] == "*" or statement["Resource"] == ["*"])
):
report.status = "FAIL"
report.status_extended = f"Policy {policy['PolicyName']} allows '*:*' administrative privileges"
break
findings.append(report)
return findings
@@ -83,7 +83,7 @@ class Test_iam_policy_no_administrative_privileges_test:
policy_document_non_administrative = {
"Version": "2012-10-17",
"Statement": [
{"Effect": "Allow", "Action": "logs:CreateLogGroup", "Resource": "*"},
{"Effect": "Allow", "Action": "logs:*", "Resource": "*"},
],
}
policy_name_administrative = "policy2"