Bug 331365: Sendmail complains about a possible attack because there's a newline…

Bug 331365: Sendmail complains about a possible attack because there's a newline in the email address - Patch by Teemu Mannermaa <wicked+bz@etlicon.fi> r=glob a=justdave

Bug 331365: Sendmail complains about a possible attack because there's a newline…
16498860 · lpsolit%gmail.com · ad75173f · 16498860
Commit 16498860 authored May 23, 2006 by lpsolit%gmail.com
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

BugMail.pm Bugzilla/BugMail.pm +2 -1

No files found.
--- a/Bugzilla/BugMail.pm
+++ b/Bugzilla/BugMail.pm
@@ -682,7 +682,8 @@ sub MessageToMTA {
        $headers = new Mail::Header \@header_lines, Modify => 0;
    }

-    my $from = $headers->get('from');
+    # Use trim to remove any whitespace (incl. newlines)
+    my $from = trim($headers->get('from'));

    if (Param("mail_delivery_method") eq "sendmail" && $^O =~ /MSWin32/i) {
        my $cmd = '|' . SENDMAIL_EXE . ' -t -i';