- simple fix for malformed Content-Disposition (#1485965)

alecpl

2009-07-06 b0d56e93f542a348b0695705dcae3b07fa2fea09

 program/include/rcube_smtp.inc

@@ -25,10 +25,6 @@
 * @package Mail
 */

// include required PEAR classes
require_once('Net/SMTP.php');


// define headers delimiter
define('SMTP_MIME_CRLF', "\r\n");

@@ -53,9 +49,20 @@
 *
 * @return bool  Returns TRUE on success, or FALSE on error
 */
function smtp_mail($from, $recipients, &$headers, &$body, &$response)
function smtp_mail($from, $recipients, &$headers, &$body, &$response, &$error)
  {
  global $SMTP_CONN, $CONFIG, $RCMAIL;
  global $SMTP_CONN, $RCMAIL;
  
  // let plugins alter smtp connection config
  $CONFIG = $RCMAIL->plugins->exec_hook('smtp_connect', array(
    'smtp_server' => $RCMAIL->config->get('smtp_server'),
    'smtp_port'   => $RCMAIL->config->get('smtp_port', 25),
    'smtp_user'   => $RCMAIL->config->get('smtp_user'),
    'smtp_pass'   => $RCMAIL->config->get('smtp_pass'),
    'smtp_auth_type' => $RCMAIL->config->get('smtp_auth_type'),
    'smtp_helo_host' => $RCMAIL->config->get('smtp_helo_host'),
  ));

  $smtp_timeout = null;
  $smtp_host = $CONFIG['smtp_server'];
  $smtp_port = is_numeric($CONFIG['smtp_port']) ? $CONFIG['smtp_port'] : 25;
@@ -89,8 +96,9 @@
    $result = $SMTP_CONN->connect($smtp_timeout);
    if (PEAR::isError($result))
      {
      $SMTP_CONN = null;
      $response[] = "Connection failed: ".$result->getMessage();
      $error = array('label' => 'smtpconnerror', 'vars' => array('code' => $SMTP_CONN->_code));
      $SMTP_CONN = null;
      return FALSE;
      }
      
@@ -103,7 +111,7 @@
        $smtp_user = $CONFIG['smtp_user'];

      if (strstr($CONFIG['smtp_pass'], '%p'))
        $smtp_pass = str_replace('%p', $RCMAIL->decrypt_passwd($_SESSION['password']), $CONFIG['smtp_pass']);
        $smtp_pass = str_replace('%p', $RCMAIL->decrypt($_SESSION['password']), $CONFIG['smtp_pass']);
      else
        $smtp_pass = $CONFIG['smtp_pass'];

@@ -112,8 +120,9 @@
    
      if (PEAR::isError($result))
        {
        smtp_reset();
        $error = array('label' => 'smtpautherror', 'vars' => array('code' => $SMTP_CONN->_code));
        $response[] .= 'Authentication failure: ' . $result->getMessage() . ' (Code: ' . $result->getCode() . ')';
        smtp_reset();
        return FALSE;
        }
      }
@@ -153,8 +162,9 @@
  // set From: address
  if (PEAR::isError($SMTP_CONN->mailFrom($from)))
    {
    smtp_reset();
    $error = array('label' => 'smtpfromerror', 'vars' => array('from' => $from, 'code' => $SMTP_CONN->_code));
    $response[] .= "Failed to set sender '$from'";
    smtp_reset();
    return FALSE;
    }

@@ -163,6 +173,7 @@
  $recipients = smtp_parse_rfc822($recipients);
  if (PEAR::isError($recipients))
    {
    $error = array('label' => 'smtprecipientserror');
    smtp_reset();
    return FALSE;
    }
@@ -173,8 +184,9 @@
    {
    if (PEAR::isError($SMTP_CONN->rcptTo($recipient)))
      {
      smtp_reset();
      $error = array('label' => 'smtptoerror', 'vars' => array('to' => $recipient, 'code' => $SMTP_CONN->_code));
      $response[] .= "Failed to add recipient '$recipient'";
      smtp_reset();
      return FALSE;
      }
    }
@@ -190,10 +202,11 @@
  unset($text_headers, $body);
   
  // Send the message's headers and the body as SMTP data.
  if (PEAR::isError($SMTP_CONN->data($data)))
  if (PEAR::isError($result = $SMTP_CONN->data($data)))
    {
    smtp_reset();
    $error = array('label' => 'smtperror', 'vars' => array('msg' => $result->getMessage()));
    $response[] .= "Failed to send data";
    smtp_reset();
    return FALSE;
    }

@@ -211,13 +224,12 @@
  {
  global $SMTP_CONN;

  if (is_object($SMTP_CONN))
  if (is_object($SMTP_CONN) && is_resource($SMTP_CONN->_socket->fp))
    {
    $SMTP_CONN->rset();
    smtp_disconnect();
    }
  }



/**