Package: sa-exim
Version: 4.2.1-9
Severity: normal
File: /usr/share/perl5/Mail/SpamAssassin/Plugin/Greylisting.pm

Hi

I'm not sure if greylisting should be called at all for locally
generated messages. But it currently is and complains about not defined
$ipbyte[1-4]. Attached patch changes address of locally generated
messages to 127.0.0.1 to avoid failures in further code.

-- 
        Michal Čihař | http://cihar.com | http://blog.cihar.com


-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.22-4-vserver-amd64 (SMP w/1 CPU core)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages sa-exim depends on:
ii  debconf [debconf-2.0]         1.5.21     Debian configuration management sy
ii  exim4-daemon-heavy [exim4-loc 4.69-5+b1  Exim MTA (v4) daemon with extended
ii  libc6                         2.7-10     GNU C Library: Shared libraries
ii  spamc                         3.2.4-1    Client for SpamAssassin spam filte

Versions of packages sa-exim recommends:
ii  perl                          5.10.0-9   Larry Wall's Practical Extraction 

-- debconf information excluded
--- /tmp/Greylisting.pm 2008-05-07 11:52:49.000000000 +0200
+++ /usr/share/perl5/Mail/SpamAssassin/Plugin/Greylisting.pm    2008-05-07 
11:52:43.000000000 +0200
@@ -104,8 +104,12 @@
     }
     chomp($connectip);
     # Clean up input (for security, if you use files/dirs)
-    $connectip =~ /([\d.:]+)/;
-    $connectip = ($1 or "");
+    if ($connectip eq '<locally generated>') {
+        $connectip = '127.0.0.1';
+    } else {
+        $connectip =~ /([\d.:]+)/;
+        $connectip = ($1 or "");
+    }
 
     # Account for a null envelope from
     if (not defined ($envfrom = $permsgstatus->get($option{'envfromhdr'})))

Reply via email to