Fix line separator in exported messages (#1488603)

Aleksander Machniak

2012-08-15 fb2f825cd95677a2ede38e0a2ea99b6823841d74

Fix line separator in exported messages (#1488603)

 CHANGELOG

@@ -1,6 +1,7 @@
CHANGELOG Roundcube Webmail
===========================

- Fix line separator in exported messages (#1488603)
- Fix XSS issue where plain signatures wasn't secured in HTML mode (#1488613)
- Fix XSS issue where href="javascript:" wasn't secured (#1488613)
- Fix impossible to create message with empty plain text part (#1488610)

 program/include/rcube_imap.php

@@ -2138,14 +2138,17 @@

    /**
     * Sends the whole message source to stdout
     *
     * @param int  $uid       Message UID
     * @param bool $formatted Enables line-ending formatting
     */
    public function print_raw_body($uid)
    public function print_raw_body($uid, $formatted = true)
    {
        if (!$this->check_connection()) {
            return;
        }

        $this->conn->handlePartBody($this->folder, $uid, true, NULL, NULL, true);
        $this->conn->handlePartBody($this->folder, $uid, true, null, null, true, null, $formatted);
    }



 program/include/rcube_imap_generic.php

@@ -2377,7 +2377,7 @@
        return $this->handlePartBody($mailbox, $id, $is_uid, $part);
    }

    function handlePartBody($mailbox, $id, $is_uid=false, $part='', $encoding=NULL, $print=NULL, $file=NULL)
    function handlePartBody($mailbox, $id, $is_uid=false, $part='', $encoding=NULL, $print=NULL, $file=NULL, $formatted=true)
    {
        if (!$this->select($mailbox)) {
            return false;
@@ -2494,7 +2494,7 @@
                        continue;
                    $line = convert_uudecode($line);
                // default
                } else {
                } else if ($formatted) {
                    $line = rtrim($line, "\t\r\n\0\x0B") . "\n";
                }


 program/include/rcube_storage.php

@@ -502,8 +502,11 @@

    /**
     * Sends the whole message source to stdout
     *
     * @param int  $uid       Message UID
     * @param bool $formatted Enables line-ending formatting
     */
    abstract function print_raw_body($uid);
    abstract function print_raw_body($uid, $formatted = true);


    /**

 program/steps/mail/viewsource.inc

@@ -44,7 +44,7 @@
    header("Content-Disposition: attachment; filename=\"$filename\"");
  }

  $RCMAIL->storage->print_raw_body($uid);
  $RCMAIL->storage->print_raw_body($uid, empty($_GET['_save']));
}
else
{