Browse Source

fix for security exploit described in bug #812690 reported by Neal Krawetz
(hackerfactor)

stekkel 22 years ago
parent
commit
3891538bb1
1 changed files with 1 additions and 1 deletions
  1. 1 1
      class/deliver/Deliver_SendMail.class.php

+ 1 - 1
class/deliver/Deliver_SendMail.class.php

@@ -23,7 +23,7 @@ class Deliver_SendMail extends Deliver {
     function initStream($message, $sendmail_path) {
         $rfc822_header = $message->rfc822_header;
 	$from = $rfc822_header->from[0];
-	$envelopefrom = $from->mailbox.'@'.$from->host;
+	$envelopefrom = trim($from->mailbox.'@'.$from->host);
 	if (strstr($sendmail_path, "qmail-inject")) {
     	    $stream = popen (escapeshellcmd("$sendmail_path -i -f$envelopefrom"), "w");
 	} else {