Patch to fix security/f-prot periodic annoyance.

jhell jhell at DataIX.net
Fri Jan 29 22:49:52 UTC 2010


The following patch "commented (#) in-line and attached" fixes the 
following annoyances.

Example output that it fixes:
Checking for denied zone transfers (AXFR and IXFR):
Updating f-prot definitions: Error: Network - Could not resolve host
failed.

Everything else in periodic reports are spaced and default to off/no for 
ports that are installed.

-: ${fpupdate_enable="YES"}
# Do not assume the user wants this on!.
+: ${fpupdate_enable:="NO"}

  case "$fpupdate_enable" in
         [Yy][Ee][Ss])
 		# Add to make sure we do not look like a failed zone
 		# transfer. This can be confusing.
+               echo
                 echo -n "Updating f-prot definitions: "
-               /usr/local/bin/fpupdate >/dev/null
 		# If we are echoing our own output then we do not need any
 		# feedback from the program whatsoever. This causes an
 		# unseen line break to happen in the output above.
+               /usr/local/bin/fpupdate >>/dev/null 2>&1
                 if [ $? = 0 ]; then
                         echo "done."
                         rc=0


Best regards.

-- 

  jhell
-------------- next part --------------
--- 900.fpupdate.orig	2010-01-29 16:46:11.861356385 -0500
+++ 900.fpupdate	2010-01-29 16:48:08.868534673 -0500
@@ -10,12 +10,13 @@
 	source_periodic_confs
 fi
 
-: ${fpupdate_enable="YES"}
+: ${fpupdate_enable:="NO"}
 
 case "$fpupdate_enable" in
 	[Yy][Ee][Ss])
+		echo
 		echo -n "Updating f-prot definitions: "
-		/usr/local/bin/fpupdate >/dev/null
+		/usr/local/bin/fpupdate >>/dev/null 2>&1
 		if [ $? = 0 ]; then
 			echo "done."
 			rc=0


More information about the freebsd-ports mailing list