https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6340

Kevin A. McGrail <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
                 CC|                            |[email protected]
         Resolution|                            |INVALID

--- Comment #18 from Kevin A. McGrail <[email protected]> 2011-05-24 16:30:31 
UTC ---
(In reply to comment #17)
> One of patch is wrong. Please correct:
> 
> -      if ($vpopusername_tainted =~ /->  &?(.+)$/) {
> +      if ($vpopusername_tainted =~ /-> &?(.+)$/) {
> 
> Here's one more space. It's possible fix this in 3.3.2?
> Thanks.

Best I can tell, this is in 3.3.2rc1 in spamd.raw and there was no reason to
reopen this unless you meant to have two spaces and the patch above is
reversed.

      if ($vpopusername_tainted =~ /-> &?(.+)$/) {
        $vpopusername = untaint_var($1);
        if ($vpopusername =~ s{^(/.+)/Maildir/$}{$1}) {
          # this is the path to a real mailbox
          $userdir = $vpopusername;

Did you mean you WANT it to be 

+      if ($vpopusername_tainted =~ /-> &?(.+)$/) {
-      if ($vpopusername_tainted =~ /->  &?(.+)$/) {

regards,
KAM

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to