From 2276972999ecb8c54ddea8ad40bdc15a7ea86a3a Mon Sep 17 00:00:00 2001

From: Jan Lieskovsky <jlieskov@redhat.com>

Date: Fri, 1 Jul 2016 15:02:12 +0200

Subject: [PATCH] [BugFix] Enhance the OVAL checks for: *

 accounts_passwords_pam_faillock_deny_root *

 accounts_passwords_pam_faillock_deny

rules to work properly also in case sssd package is installed

and sssd daemon is running

Fixes downstream: https://bugzilla.redhat.com/show_bug.cgi?id=1344581

---

 RHEL/7/input/oval/accounts_passwords_pam_faillock_deny_root.xml | 8 ++++----

 shared/oval/accounts_passwords_pam_faillock_deny.xml            | 8 ++++----

 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/RHEL/7/input/oval/accounts_passwords_pam_faillock_deny_root.xml b/RHEL/7/input/oval/accounts_passwords_pam_faillock_deny_root.xml

index 50f2e5a..7b60d22 100644

--- a/RHEL/7/input/oval/accounts_passwords_pam_faillock_deny_root.xml

+++ b/RHEL/7/input/oval/accounts_passwords_pam_faillock_deny_root.xml

@@ -34,7 +34,7 @@

     <ind:filepath>/etc/pam.d/system-auth</ind:filepath>

          pam_unix.so module in auth section -->

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*even_deny_root[\s]*[^\n]*[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*even_deny_root[\s]*(?s).*[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>

@@ -51,7 +51,7 @@

     <ind:behaviors singleline="true" />

     <ind:filepath>/etc/pam.d/system-auth</ind:filepath>

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]+[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*even_deny_root[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]+(?s).*[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*even_deny_root[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>

@@ -69,7 +69,7 @@

     <ind:filepath>/etc/pam.d/password-auth</ind:filepath>

          pam_unix.so module in auth section -->

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*even_deny_root[\s]*[^\n]*[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*even_deny_root[\s]*(?s).*[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>

@@ -86,7 +86,7 @@

     <ind:behaviors singleline="true" />

     <ind:filepath>/etc/pam.d/password-auth</ind:filepath>

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]+[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*even_deny_root[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]+(?s).*[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*even_deny_root[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>

diff --git a/shared/oval/accounts_passwords_pam_faillock_deny.xml b/shared/oval/accounts_passwords_pam_faillock_deny.xml

index 96b5043..0923dc9 100644

--- a/shared/oval/accounts_passwords_pam_faillock_deny.xml

+++ b/shared/oval/accounts_passwords_pam_faillock_deny.xml

@@ -51,7 +51,7 @@

     <ind:filepath>/etc/pam.d/system-auth</ind:filepath>

          pam_unix.so module in auth section -->

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*deny=([0-9]+)[\s]*[^\n]*[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*deny=([0-9]+)[\s]*(?s).*[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>

@@ -69,7 +69,7 @@

     <ind:behaviors singleline="true" />

     <ind:filepath>/etc/pam.d/system-auth</ind:filepath>

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]+[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*deny=([0-9]+)[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]+(?s).*[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*deny=([0-9]+)[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>

@@ -106,7 +106,7 @@

     <ind:filepath>/etc/pam.d/password-auth</ind:filepath>

          pam_unix.so module in auth section -->

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*deny=([0-9]+)[\s]*[^\n]*[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+required[\s]+pam_faillock\.so[\s]+preauth[\s]+silent[\s]+[^\n]*deny=([0-9]+)[\s]*(?s).*[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>

@@ -124,7 +124,7 @@

     <ind:behaviors singleline="true" />

     <ind:filepath>/etc/pam.d/password-auth</ind:filepath>

-    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+sufficient[\s]+pam_unix\.so[^\n]+[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*deny=([0-9]+)[^\n]*[\n]</ind:pattern>

+    <ind:pattern operation="pattern match">[\n][\s]*auth[\s]+(?:(?:sufficient)|(?:\[.*default=die.*\]))[\s]+pam_unix\.so[^\n]+(?s).*[\n][\s]*auth[\s]+\[default=die\][\s]+pam_faillock\.so[\s]+authfail[\s]+[^\n]*deny=([0-9]+)[^\n]*[\n]</ind:pattern>

     <ind:instance datatype="int" operation="equals">1</ind:instance>

   </ind:textfilecontent54_object>