<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 4/19/13 11:39 PM, Shawn Wells wrote:<br>
    </div>
    <blockquote cite="mid:20130420033955.GA1246@mail.corp.redhat.com"
      type="cite">
      <br>
      <fieldset class="mimeAttachmentHeader"><legend
          class="mimeAttachmentHeaderName">0001-bugfix-Ticket-168-iptables_smtp_enabled-false-positi.patch</legend></fieldset>
      <br>
      <div class="moz-text-plain" wrap="true" graphical-quote="true"
        style="font-family: -moz-fixed; font-size: 12px;"
        lang="x-western">
        <pre wrap="">From ff769fddff246376939d6b2812442aab79206c05 Mon Sep 17 00:00:00 2001
From: Shawn Wells <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:shawn@redhat.com">&lt;shawn@redhat.com&gt;</a>
Date: Fri, 19 Apr 2013 23:37:34 -0400
Subject: [PATCH] [bugfix] Ticket 168 - iptables_smtp_enabled false positive
 The test did not first check if Postfix is installed and running, so this was a false positive on most scans.
 Added OVAL to check for existence of Postfix

NOTE: Logic needs to be added to detect if IPv6 in use. But that's ticket 398 (aka, for another time).
---
 RHEL6/input/checks/iptables_smtp_enabled.xml |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/RHEL6/input/checks/iptables_smtp_enabled.xml b/RHEL6/input/checks/iptables_smtp_enabled.xml
index a84d2b0..430da42 100644
--- a/RHEL6/input/checks/iptables_smtp_enabled.xml
+++ b/RHEL6/input/checks/iptables_smtp_enabled.xml
@@ -10,10 +10,20 @@
       the ldap service.&lt;/description&gt;
     &lt;/metadata&gt;
     &lt;criteria operator="AND"&gt;
+      &lt;criterion comment="Package Postfix is installed" test_ref="test_package_postfix_installed" /&gt;
       &lt;criterion comment="Test ipv4 25 Accept" test_ref="test_iptables_smtp_enabled_ipv4" /&gt;
       &lt;criterion comment="Test ipv6 25 Accept" test_ref="test_iptables_smtp_enabled_ipv6" /&gt;
     &lt;/criteria&gt;
   &lt;/definition&gt;
+
+  &lt;linux:rpminfo_test check="all" check_existence="all_exist" id="test_package_postfix_installed" version="1"
+  comment="Package Postfix is installed"&gt;
+    &lt;linux:object object_ref="obj_package_postfix" /&gt;
+  &lt;/linux:rpminfo_test&gt;
+  &lt;linux:rpminfo_object id="obj_package_postfix" version="1"&gt;
+    &lt;linux:name&gt;postfix&lt;/linux:name&gt;
+  &lt;/linux:rpminfo_object&gt;
+
   &lt;ind:textfilecontent54_test check="all"
   check_existence="all_exist" comment="Test for 25 access over ipv4"
   id="test_iptables_smtp_enabled_ipv4" version="1"&gt;
<div class="moz-txt-sig">-- 
1.7.1
</div></pre>
      </div>
    </blockquote>
    <br>
    pushed as bugfix<br>
    <br>
    [shawn@rhel6 scap-security-guide]$ git push<br>
    Counting objects: 11, done.<br>
    Compressing objects: 100% (6/6), done.<br>
    Writing objects: 100% (6/6), 878 bytes, done.<br>
    Total 6 (delta 5), reused 0 (delta 0)<br>
    To <a class="moz-txt-link-freetext" href="ssh://git.fedorahosted.org/git/scap-security-guide.git">ssh://git.fedorahosted.org/git/scap-security-guide.git</a><br>
       908e826..3d5c7ff  master -&gt; master<br>
  </body>
</html>