[Pkg-shadow-devel] amavisd-new: Please quote the commands given to su

[Nicolas François]

Package: amavisd-new
Severity: wishlist
Tags: patch

Hello,

As announced in [0], su behavior will change.
In the amavisd-new's cron.daily file, the command provided to su must be
quoted, and specified with su's -c option
With the attached patch, the cron job should be compatible with the old
and future su.

You can find a login package in experimental if you want to test the
attached patch.

When the new login package will reach unstable, the severity of this bug
will be raised.

[0] http://lists.debian.org/debian-devel-announce/2006/03/msg00003.html

Kind Regards,
-- 
Nekral
-------------- next part --------------
diff -rauN ../orig/amavisd-new-2.3.3/debian/amavisd-new.cron.daily ./amavisd-new-2.3.3/debian/amavisd-new.cron.daily
--- ../orig/amavisd-new-2.3.3/debian/amavisd-new.cron.daily	2006-03-25 19:07:35.000000000 +0100
+++ ./amavisd-new-2.3.3/debian/amavisd-new.cron.daily	2006-03-25 20:21:47.000000000 +0100
@@ -6,6 +6,6 @@
 # Needed only if spamassassin is in use
 umask 022
 test -e /usr/bin/sa-learn && test -e /usr/sbin/amavisd-new && {
-	su - amavis -- /usr/bin/sa-learn --sync --force-expire >/dev/null
+	su - amavis -c "/usr/bin/sa-learn --sync --force-expire >/dev/null"
 }
 exit 0