ports/169660: security/amavisd-new: let amavisd_ram respect tmpfs

[Helmut Schneider]

>Number:         169660
>Category:       ports
>Synopsis:       security/amavisd-new: let amavisd_ram respect tmpfs
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Jul 05 12:00:25 UTC 2012
>Closed-Date:
>Last-Modified:
>Originator:     Helmut Schneider
>Release:        9.0-RELEASE
>Organization:
>Environment:
>Description:
amavisd_ram checks for an existing ramdisk at /var/amavisd/tmp and if it does not exists it creates a new one using mdmfs. While one can argue if mdm of tmpfs is the way of choice, amavisd at least should respect existing tmpfs filesystems.
>How-To-Repeat:

>Fix:
--- /usr/local/etc/rc.d/amavisd 2012-07-05 13:35:50.000000000 +0200
+++ amavisd     2012-07-05 13:50:18.000000000 +0200
@@ -35,8 +35,8 @@
   echo "WARNING: using ramdisk is reported to be unstable and"
   echo "thus it is highly recommended to be turned off."
   echo "========================================================"
-  df /var/amavis/tmp |  grep '^/dev/md' > /dev/null
-  if [ $? -eq 1 ];then
+  df /var/amavis/tmp | grep -E '^(/dev/md|tmpfs)' > /dev/null
+  if [ $? -eq 1 ]; then
     mdmfs -s ${amavisd_ram} -p 750 -w vscan:vscan md /var/amavis/tmp || true
   fi
 fi

>Release-Note:
>Audit-Trail:
>Unformatted: