attachment_outer_name and attachment_email_name

Mark Martinec Mark.Martinec+amavis at ijs.si
Fri May 6 19:19:42 CEST 2011


Andreas,

> > I get errors when setting using the placeholder %n:
> > All other placeholders (%P,%b,%m,%i) work fine.

> Thanks. As a quick fix please try:
> -            : $1 eq 'n' ? $msginfo->log_id
> +            : $1 eq 'n' ? untaint($msginfo->log_id)
> I'll try to figure out later how this happens

The above should be alright as a temporary measure, but
perhaps the following patch (against 2.7.0-pre15) is cleaner:

--- amavisd.orig	2011-04-13 02:29:25.000000000 +0200
+++ amavisd	2011-05-06 19:16:02.493477054 +0200
@@ -17562,7 +17562,7 @@
     # RFC 4648 base64:       62 +, 63 /
     # RFC 4648 base64url:    62 -, 63 _
-    $mail_id =~ m{^ [A-Za-z0-9] [A-Za-z0-9/_+-]* ={0,2} \z}xs
+    $mail_id =~ m{^ [A-Za-z0-9] [A-Za-z0-9_+-]* ={0,2} \z}xs
       or die "Invalid mail_id '$mail_id'";
-    $msginfo->mail_id($mail_id);
+    $msginfo->mail_id(untaint($mail_id));
     if (!exists($attr_ref->{'secret_id'}) || $attr_ref->{'secret_id'} eq '') {
       die "Secret_id is required, but missing"  if c('auth_required_release');


Thanks for the problem report.

  Mark


More information about the amavis-users mailing list