documenting two functions
[squirrelmail.git] / functions / mime.php
index 33b829c40218473d2f66e336cd95090333031e7a..989018ae08263043e02a707c5df5b6826d105269 100644 (file)
 <?php
 
 /**
- * mime.php
- *
- * Copyright (c) 1999-2002 The SquirrelMail Project Team
- * Licensed under the GNU GPL. For full terms see the file COPYING.
- *
- * This contains the functions necessary to detect and decode MIME
- * messages.
- *
- * $Id$
- */
-
-require_once('../functions/imap.php');
-require_once('../functions/attachment_common.php');
-
-/** Setting up the objects that have the structure for the message **/
-class msg_header {
-    /** msg_header contains generic variables for values that **/
-    /** could be in a header.                                 **/
-
-    var $type0 = '', $type1 = '', $boundary = '', $charset = '',
-        $encoding = '', $size = 0, $to = array(), $from = '', $date = '',
-        $cc = array(), $bcc = array(), $reply_to = '', $subject = '',
-        $id = 0, $mailbox = '', $description = '', $filename = '',
-        $entity_id = 0, $message_id = 0, $name = '', $priority = 3, $type = '';
-}
-
-class message {
-    /** message is the object that contains messages.  It is a recursive
-      object in that through the $entities variable, it can contain
-      more objects of type message.  See documentation in mime.txt for
-      a better description of how this works.
-    **/
-    var $header = '', $entities = array();
-
-    function addEntity ($msg) {
-        $this->entities[] = $msg;
-    }
-}
-
-/* --------------------------------------------------------------------------------- */
-/* MIME DECODING                                                                     */
-/* --------------------------------------------------------------------------------- */
+* mime.php
+*
+* Copyright (c) 1999-2005 The SquirrelMail Project Team
+* Licensed under the GNU GPL. For full terms see the file COPYING.
+*
+* This contains the functions necessary to detect and decode MIME
+* messages.
+*
+* @version $Id$
+* @package squirrelmail
+*/
 
-/* This function gets the structure of a message and stores it in the "message" class.
- * It will return this object for use with all relevant header information and
- * fully parsed into the standard "message" object format.
- */
-function mime_structure ($imap_stream, $header) {
+/** The typical includes... */
+require_once(SM_PATH . 'functions/imap.php');
+require_once(SM_PATH . 'functions/attachment_common.php');
 
-    $ssid = sqimap_session_id();
-    $lsid = strlen( $ssid );
-    $id = $header->id;
-    fputs ($imap_stream, "$ssid FETCH $id BODYSTRUCTURE\r\n");
-    //
-    // This should use sqimap_read_data instead of reading it itself
-    //
-    $read = fgets ($imap_stream, 9216);
-    $bodystructure = '';
-    while ( substr($read, 0, $lsid) <> $ssid &&
-         !feof( $imap_stream ) ) {
-        $bodystructure .= $read;
-        $read = fgets ($imap_stream, 9216);
-    }
-    $read = $bodystructure;
+/* -------------------------------------------------------------------------- */
+/* MIME DECODING                                                              */
+/* -------------------------------------------------------------------------- */
 
-    // isolate the body structure and remove beginning and end parenthesis
-    $read = trim(substr ($read, strpos(strtolower($read), 'bodystructure') + 13));
+/**
+* Get the MIME structure
+*
+* This function gets the structure of a message and stores it in the "message" class.
+* It will return this object for use with all relevant header information and
+* fully parsed into the standard "message" object format.
+*/
+function mime_structure ($bodystructure, $flags=array()) {
 
+    /* Isolate the body structure and remove beginning and end parenthesis. */
+    $read = trim(substr ($bodystructure, strpos(strtolower($bodystructure), 'bodystructure') + 13));
     $read = trim(substr ($read, 0, -1));
-    $end = mime_match_parenthesis(0, $read);
-    while ($end == strlen($read)-1) {
-        $read = trim(substr ($read, 0, -1));
-        $read = trim(substr ($read, 1));
-        $end = mime_match_parenthesis(0, $read);
-    }
-
-    $msg = mime_parse_structure ($read, 0);
-    $msg->header = $header;
-
-    return( $msg );
-}
-
-/* this starts the parsing of a particular structure.  It is called recursively,
- * so it can be passed different structures.  It returns an object of type
- * $message.
- * First, it checks to see if it is a multipart message.  If it is, then it
- * handles that as it sees is necessary.  If it is just a regular entity,
- * then it parses it and adds the necessary header information (by calling out
- * to mime_get_elements()
- */
-function mime_parse_structure ($structure, $ent_id) {
-  global $mailbox;
-  $properties = array();
-  $msg = new message();
-  if ($structure{0} == '(') {
-     $old_ent_id = $ent_id;
-     $ent_id = mime_new_element_level($ent_id);
-     $start = $end = -1;
-     do {
-        $start = $end+1;
-        $end = mime_match_parenthesis ($start, $structure);
-
-       /* check if we are dealing with a new entity-level */
-       $i = strrpos($ent_id,'.');
-       if ($i>0) {
-           $ent = substr($ent_id, $i+1);
-       } else {
-           $ent = '';
-       }
-        /* add "forgotten"  parent entities (alternative and relative) */
-       if ($ent == '0') {
-           /* new entity levels have information about the type (type1) and 
-           *  the properties. This information is situated at the end of the 
-           *  structure string like for example (example between the brackets) 
-           *  [ "RELATED" ("BOUNDARY" "myboundary" "TYPE" "plain/html") ]
-           */
-           
-           /* get the involved properties for parsing to mime_get_properties */
-           $startprop = strrpos($structure,'(');
-           $properties_str = substr($structure,$startprop);
-           $endprop = mime_match_parenthesis ($startprop, $structure);
-           $propstr = substr($structure, $startprop + 1, ($endprop - $startprop)-1);
-           /* cut off the used properties */
-           if ($startprop) { 
-               $structure_end = substr($structure, $endprop+2);
-               $structure = trim(substr($structure,0,$startprop));
-           }
-           /* get type1 */
-           $pos = strrpos($structure,' ');
-           if ($structure{$pos+1} =='(') $pos++;
-           
-           $type1 = strtolower(substr($structure, $pos+2, (count($structure)-2)));
-           /* cut off  type1 */
-           if ($pos && $startprop) {
-               $structure = trim(substr($structure, 0, $pos));
-           }
-
-           /* process the found information */
-            $properties = mime_get_props($properties, $properties_str);
-           if (count($properties)>0) {
-               $msg->header->entity_id = $old_ent_id;
-               $msg->header->type0 = 'multipart';
-               $msg->header->type1 = $type1;
-               for ($i=0; $i < count($properties); $i++) {
-                   $msg->header->{$properties[$i]['name']} = $properties[$i]['value'];
-               }
-           }
-           $structure = $structure . ' ' . $structure_end;
-       } 
-       $element = substr($structure, $start+1, ($end - $start)-1);
-       $ent_id = mime_increment_id ($ent_id);
-       $newmsg = mime_parse_structure ($element, $ent_id);
-       /* set mailbox in case of message/rfc822 entities */
-       if (isset($newmsg->header->type0) && isset($newmsg->header->type1)) {
-           if ($newmsg->header->type0 == 'message' && $newmsg->header->type1 == 'rfc822') {
-               $newmsg->header->mailbox=$mailbox;
-           }
-       }
-       $msg->addEntity ($newmsg);
-
-     } while ($structure{$end+1} == '(');
-  } else {
-     // parse the elements
-    $msg = mime_get_element ($structure, $msg, $ent_id);
-  }
-  return $msg;
-}
-
-
-/* Increments the element ID.  An element id can look like any of
- * the following:  1, 1.2, 4.3.2.4.1, etc.  This function increments
- * the last number of the element id, changing 1.2 to 1.3.
- */
-function mime_increment_id ($id) {
-
-    if (strpos($id, '.')) {
-        $first = substr($id, 0, strrpos($id, '.'));
-        $last = substr($id, strrpos($id, '.')+1);
-        $last++;
-        $new = $first . '.' .$last;
-    } else {
-        $new = $id + 1;
-    }
-
-    return $new;
-}
-
-/*
- * See comment for mime_increment_id().
- * This adds another level on to the entity_id changing 1.3 to 1.3.0
- * NOTE:  1.3.0 is not a valid element ID.  It MUST be incremented
- *        before it can be used.  I left it this way so as not to have
- *        to make a special case if it is the first entity_id.  It
- *        always increments it, and that works fine.
- */
-function mime_new_element_level ($id) {
-
-    if (!$id) {
-        $id = 0;
-    } else {
-        $id = $id . '.0';
+    $i = 0;
+    $msg = Message::parseStructure($read,$i);
+    if (!is_object($msg)) {
+        include_once(SM_PATH . 'functions/display_messages.php');
+        global $color, $mailbox;
+        /* removed urldecode because $_GET is auto urldecoded ??? */
+        displayPageHeader( $color, $mailbox );
+        $errormessage  = _("SquirrelMail could not decode the bodystructure of the message");
+        $errormessage .= '<br />'._("the provided bodystructure by your imap-server").':<br /><br />';
+        $errormessage .= '<pre>' . htmlspecialchars($read) . '</pre>';
+        plain_error_message( $errormessage, $color );
+        echo '</body></html>';
+        exit;
     }
-
-    return( $id );
-}
-
-function mime_get_element (&$structure, $msg, $ent_id) {
-
-  $elem_num = 1;
-  $msg->header = new msg_header();
-  $msg->header->entity_id = $ent_id;
-  $properties = array();
-  while (strlen($structure) > 0) {
-     $structure = trim($structure);
-     $char = $structure{0};
-
-     if (strtolower(substr($structure, 0, 3)) == 'nil') {
-        $text = '';
-        $structure = substr($structure, 3);
-     } else if ($char == '"') {
-        // loop through until we find the matching quote, and return that as a string
-        $pos = 1;
-        $text = '';
-        while ( ($char = $structure{$pos} ) <> '"' && $pos < strlen($structure)) {
-           $text .= $char;
-           $pos++;
-        }
-        $structure = substr($structure, strlen($text) + 2);
-     } else if ($char == '{') {
-         /**
-          * loop through until we find the matching quote, 
-          * and return that as a string
-          */
-         $pos = 1;
-         $len = '';
-         while (($char = $structure{$pos}) != '}' 
-                && $pos < strlen($structure)) {
-             $len .= $char;
-             $pos++;
-         }
-         $structure = substr($structure, strlen($len) + 4);
-         $text = substr($structure, 0, $len);
-         $structure = substr($structure, $len + 1);
-     } else if ($char == '(') {
-        // comment me
-        $end = mime_match_parenthesis (0, $structure);
-        $sub = substr($structure, 1, $end-1);
-        $properties = mime_get_props($properties, $sub);
-        $structure = substr($structure, strlen($sub) + 2);
-     } else {
-        // loop through until we find a space or an end parenthesis
-        $pos = 0;
-        $char = $structure{$pos};
-        $text = '';
-        while ($char != ' ' && $char != ')' && $pos < strlen($structure)) {
-           $text .= $char;
-           $pos++;
-           $char = $structure{$pos};
-        }
-        $structure = substr($structure, strlen($text));
-     }
-
-     // This is where all the text parts get put into the header
-     switch ($elem_num) {
-        case 1:
-           $msg->header->type0 = strtolower($text);
-           break;
-        case 2:
-           $msg->header->type1 = strtolower($text);
-           break;
-        case 4: // Id
-           // Invisimail enclose images with <>
-           $msg->header->id = str_replace( '<', '', str_replace( '>', '', $text ) );
-           break;
-        case 5:
-           $msg->header->description = $text;
-           break;
-        case 6:
-           $msg->header->encoding = strtolower($text);
-           break;
-        case 7:
-           $msg->header->size = $text;
-           break;
-        default:
-           if ($msg->header->type0 == 'text' && $elem_num == 8) {
-              // This is a plain text message, so lets get the number of lines
-              // that it contains.
-              $msg->header->num_lines = $text;
-
-           } else if ($msg->header->type0 == 'message' && $msg->header->type1 == 'rfc822' && $elem_num == 8) {
-              // This is an encapsulated message, so lets start all over again and
-              // parse this message adding it on to the existing one.
-              $structure = trim($structure);
-              if ( $structure{0} == '(' ) {
-                 $e = mime_match_parenthesis (0, $structure);
-                 $structure = substr($structure, 0, $e);
-                 $structure = substr($structure, 1);
-                 $m = mime_parse_structure($structure, $msg->header->entity_id);
-
-                 // the following conditional is there to correct a bug that wasn't
-                 // incrementing the entity IDs correctly because of the special case
-                 // that message/rfc822 is.  This fixes it fine.
-                 if (substr($structure, 1, 1) != '(')
-                    $m->header->entity_id = mime_increment_id(mime_new_element_level($ent_id));
-
-                 // Now we'll go through and reformat the results.
-                 if ($m->entities) {
-                    for ($i=0; $i < count($m->entities); $i++) {
-                       $msg->addEntity($m->entities[$i]);
+    if (count($flags)) {
+        foreach ($flags as $flag) {
+            $char = strtoupper($flag{1});
+            switch ($char) {
+                case 'S':
+                    if (strtolower($flag) == '\\seen') {
+                        $msg->is_seen = true;
                     }
-                 } else {
-                    $msg->addEntity($m);
-                 }
-                 $structure = "";
-              }
-           }
-           break;
-     }
-     $elem_num++;
-     $text = "";
-  }
-  // loop through the additional properties and put those in the various headers
-  for ($i=0; $i < count($properties); $i++) {
-     $msg->header->{$properties[$i]['name']} = $properties[$i]['value'];
-  }
-
-  return $msg;
-}
-
-/*
- * I did most of the MIME stuff yesterday (June 20, 2000), but I couldn't
- * figure out how to do this part, so I decided to go to bed.  I woke up
- * in the morning and had a flash of insight.  I went to the white-board
- * and scribbled it out, then spent a bit programming it, and this is the
- * result.  Nothing complicated, but I think my brain was fried yesterday.
- * Funny how that happens some times.
- *
- * This gets properties in a nested parenthesisized list.  For example,
- * this would get passed something like:  ("attachment" ("filename" "luke.tar.gz"))
- * This returns an array called $props with all paired up properties.
- * It ignores the "attachment" for now, maybe that should change later
- * down the road.  In this case, what is returned is:
- *    $props[0]["name"] = "filename";
- *    $props[0]["value"] = "luke.tar.gz";
- */
-function mime_get_props ($props, $structure) {
-
-  while (strlen($structure) > 0) {
-     $structure = trim($structure);
-     $char = $structure{0};
-     if ($char == '"') {
-        $pos = 1;
-        $tmp = '';
-        while ( ( $char = $structure{$pos} ) != '"' &&
-                $pos < strlen($structure)) {
-           $tmp .= $char;
-           $pos++;
-        }
-        $structure = trim(substr($structure, strlen($tmp) + 2));
-        $char = $structure{0};
-
-        if ($char == '"') {
-           $pos = 1;
-           $value = '';
-           while ( ( $char = $structure{$pos} ) != '"' &&
-                   $pos < strlen($structure) ) {
-              $value .= $char;
-              $pos++;
-           }
-           $structure = trim(substr($structure, strlen($value) + 2));
-           $k = count($props);
-           $props[$k]['name'] = strtolower($tmp);
-           $props[$k]['value'] = $value;
-          if ($structure != '') {
-               mime_get_props($props, $structure);
-          } else {
-            return $props;
-          }            
-        } else if ($char == '(') {
-           $end = mime_match_parenthesis (0, $structure);
-           $sub = substr($structure, 1, $end-1);
-          if (! isset($props))
-              $props = array();
-              $props = mime_get_props($props, $sub);
-              $structure = substr($structure, strlen($sub) + 2);
-          return $props;      
-        }
-     } else if ($char == '(') {
-        $end = mime_match_parenthesis (0, $structure);
-        $sub = substr($structure, 1, $end-1);
-        $props = mime_get_props($props, $sub);
-        $structure = substr($structure, strlen($sub) + 2);
-        return $props;
-     } else {
-        return $props;
-     }
-  }
-}
-
-/*
- *  Matches parenthesis.  It will return the position of the matching
- *  parenthesis in $structure.  For instance, if $structure was:
- *     ("text" "plain" ("val1name", "1") nil ... )
- *     x                                         x
- *  then this would return 42 to match up those two.
- */
-function mime_match_parenthesis ($pos, $structure) {
-
-    $j = strlen( $structure );
-    
-    // ignore all extra characters
-    // If inside of a string, skip string -- Boundary IDs and other
-    // things can have ) in them.
-    if ( $structure{$pos} != '(' ) {
-        return( $j );
-    }
-    
-    while ( $pos < $j ) {
-        $pos++;
-        if ($structure{$pos} == ')') {
-            return $pos;
-        } elseif ($structure{$pos} == '"') {
-            $pos++;
-            while ( $structure{$pos} != '"' &&
-                    $pos < $j ) {
-               if (substr($structure, $pos, 2) == '\\"') {
-                  $pos++;
-               } elseif (substr($structure, $pos, 2) == '\\\\') {
-                  $pos++;
-               }
-               $pos++;
+                    break;
+                case 'A':
+                    if (strtolower($flag) == '\\answered') {
+                        $msg->is_answered = true;
+                    }
+                    break;
+                case 'D':
+                    if (strtolower($flag) == '\\deleted') {
+                        $msg->is_deleted = true;
+                    }
+                    break;
+                case 'F':
+                    if (strtolower($flag) == '\\flagged') {
+                        $msg->is_flagged = true;
+                    }
+                    break;
+                case 'M':
+                    if (strtolower($flag) == '$mdnsent') {
+                        $msg->is_mdnsent = true;
+                    }
+                    break;
+                default:
+                    break;
             }
-        } elseif ( $structure{$pos} == '(' ) {
-            $pos = mime_match_parenthesis ($pos, $structure);
         }
     }
-    echo _("Error decoding mime structure.  Report this as a bug!") . '<br>';
-    return( $pos );
+    //    listEntities($msg);
+    return $msg;
 }
 
-function mime_fetch_body($imap_stream, $id, $ent_id) {
 
-    /*
-     * do a bit of error correction.  If we couldn't find the entity id, just guess
-     * that it is the first one.  That is usually the case anyway.
-     */
+
+/* This starts the parsing of a particular structure.  It is called recursively,
+* so it can be passed different structures.  It returns an object of type
+* $message.
+* First, it checks to see if it is a multipart message.  If it is, then it
+* handles that as it sees is necessary.  If it is just a regular entity,
+* then it parses it and adds the necessary header information (by calling out
+* to mime_get_elements()
+*/
+
+function mime_fetch_body($imap_stream, $id, $ent_id=1, $fetch_size=0) {
+    /* Do a bit of error correction.  If we couldn't find the entity id, just guess
+    * that it is the first one.  That is usually the case anyway.
+    */
+
     if (!$ent_id) {
-        $ent_id = 1;
+        $cmd = "FETCH $id BODY[]";
+    } else {
+        $cmd = "FETCH $id BODY[$ent_id]";
     }
-    $cmd = "FETCH $id BODY[$ent_id]";
 
-    $data = sqimap_run_command ($imap_stream, $cmd, true, $response, $message);
+    if ($fetch_size!=0) $cmd .= "<0.$fetch_size>";
+
+    $data = sqimap_run_command ($imap_stream, $cmd, true, $response, $message, TRUE);
     do {
-        $topline = trim(array_shift( $data ));
-    } while( $topline && $topline[0] == '*' && !preg_match( '/\* [0-9]+ FETCH.*/i', $topline )) ;
+        $topline = trim(array_shift($data));
+    } while($topline && ($topline[0] == '*') && !preg_match('/\* [0-9]+ FETCH.*/i', $topline)) ;
+
     $wholemessage = implode('', $data);
     if (ereg('\\{([^\\}]*)\\}', $topline, $regs)) {
-
-        $ret = substr( $wholemessage, 0, $regs[1] );
-        /*
-            There is some information in the content info header that could be important
-            in order to parse html messages. Let's get them here.
+        $ret = substr($wholemessage, 0, $regs[1]);
+        /* There is some information in the content info header that could be important
+        * in order to parse html messages. Let's get them here.
         */
-        if ( $ret{0} == '<' ) {
-            $data = sqimap_run_command ($imap_stream, "FETCH $id BODY[$ent_id.MIME]", true, $response, $message);
-            /* BASE within HTML documents is illegal (see w3 spec)
-*            $base = '';
-*            $k = 10;
-*            foreach( $data as $d ) {
-*                if ( substr( $d, 0, 13 ) == 'Content-Base:' ) {
-*                    $j = strlen( $d );
-*                    $i = 13;
-*                    $base = '';
-*                    while ( $i < $j &&
-*                           ( !isNoSep( $d{$i} ) || $d{$i} == '"' )  )
-*                        $i++;
-*                    while ( $i < $j ) {
-*                        if ( isNoSep( $d{$i} ) )
-*                            $base .= $d{$i};
-*                        $i++;
-*                    }
-*                    $k = 0;
-*                } elseif ( $k == 1 && !isnosep( $d{0} ) ) {
-*                    $base .= substr( $d, 1 );
-*                }
-*                $k++;
-*            }
-*            if ( $base <> '' ) {
-*                $ret = "<base href=\"$base\">" . $ret;
-*            }
-*           */
-        }
+//        if ($ret{0} == '<') {
+//            $data = sqimap_run_command ($imap_stream, "FETCH $id BODY[$ent_id.MIME]", true, $response, $message, TRUE);
+//        }
     } else if (ereg('"([^"]*)"', $topline, $regs)) {
         $ret = $regs[1];
     } else {
         global $where, $what, $mailbox, $passed_id, $startMessage;
-        $par = 'mailbox=' . urlencode($mailbox) . "&amp;passed_id=$passed_id";
+        $par = 'mailbox=' . urlencode($mailbox) . '&amp;passed_id=' . $passed_id;
         if (isset($where) && isset($what)) {
-            $par .= '&amp;where='. urlencode($where) . "&amp;what=" . urlencode($what);
+            $par .= '&amp;where=' . urlencode($where) . '&amp;what=' . urlencode($what);
         } else {
-            $par .= "&amp;startMessage=$startMessage&amp;show_more=0";
+            $par .= '&amp;startMessage=' . $startMessage . '&amp;show_more=0';
         }
         $par .= '&amp;response=' . urlencode($response) .
-                '&amp;message=' . urlencode($message).
-                '&amp;topline=' . urlencode($topline);
-
-        echo   '<tt><br>' .
-               '<table width="80%"><tr>' .
-               '<tr><td colspan=2>' .
-               _("Body retrieval error. The reason for this is most probably that the message is malformed. Please help us making future versions better by submitting this message to the developers knowledgebase!") .
-               " <A HREF=\"../src/retrievalerror.php?$par\"><br>" .
-               _("Submit message") . '</A><BR>&nbsp;' .
-               '</td></tr>' .
-               '<td><b>' . _("Command:") . "</td><td>$cmd</td></tr>" .
-               '<td><b>' . _("Response:") . "</td><td>$response</td></tr>" .
-               '<td><b>' . _("Message:") . "</td><td>$message</td></tr>" .
-               '<td><b>' . _("FETCH line:") . "</td><td>$topline</td></tr>" .
-               "</table><BR></tt></font><hr>";
-
-        $data = sqimap_run_command ($imap_stream, "FETCH $passed_id BODY[]", true, $response, $message);
+                '&amp;message='  . urlencode($message)  .
+                '&amp;topline='  . urlencode($topline);
+
+        echo   '<tt><br />' .
+            '<table width="80%"><tr>' .
+            '<tr><td colspan="2">' .
+            _("Body retrieval error. The reason for this is most probably that the message is malformed.") .
+            '</td></tr>' .
+            '<tr><td><b>' . _("Command:") . "</td><td>$cmd</td></tr>" .
+            '<tr><td><b>' . _("Response:") . "</td><td>$response</td></tr>" .
+            '<tr><td><b>' . _("Message:") . "</td><td>$message</td></tr>" .
+            '<tr><td><b>' . _("FETCH line:") . "</td><td>$topline</td></tr>" .
+            "</table><br /></tt></font><hr />";
+
+        $data = sqimap_run_command ($imap_stream, "FETCH $passed_id BODY[]", true, $response, $message, TRUE);
         array_shift($data);
         $wholemessage = implode('', $data);
 
         $ret = $wholemessage;
     }
-    return( $ret );
+    return $ret;
 }
 
-function mime_print_body_lines ($imap_stream, $id, $ent_id, $encoding) {
-    // do a bit of error correction.  If we couldn't find the entity id, just guess
-    // that it is the first one.  That is usually the case anyway.
-    if (!$ent_id) {
-        $ent_id = 1;
-    }
-    $sid = sqimap_session_id();
-    // Don't kill the connection if the browser is over a dialup
-    // and it would take over 30 seconds to download it.
+function mime_print_body_lines ($imap_stream, $id, $ent_id=1, $encoding) {
 
-    // don´t call set_time_limit in safe mode.
-    if (!ini_get("safe_mode")) {
+    /* Don't kill the connection if the browser is over a dialup
+    * and it would take over 30 seconds to download it.
+    * Don't call set_time_limit in safe mode.
+    */
+
+    if (!ini_get('safe_mode')) {
         set_time_limit(0);
     }
+    /* in case of base64 encoded attachments, do not buffer them.
+    Instead, echo the decoded attachment directly to screen */
+    if (strtolower($encoding) == 'base64') {
+        if (!$ent_id) {
+        $query = "FETCH $id BODY[]";
+        } else {
+        $query = "FETCH $id BODY[$ent_id]";
+        }
+        sqimap_run_command($imap_stream,$query,true,$response,$message,TRUE,'sqimap_base64_decode','php://stdout',true);
+    } else {
+    $body = mime_fetch_body ($imap_stream, $id, $ent_id);
+    echo decodeBody($body, $encoding);
+    }
+
+    /*
+    TODO, use the same method for quoted printable.
+    However, I assume that quoted printable attachments aren't that large
+    so the performancegain / memory usage drop will be minimal.
+    If we decide to add that then we need to adapt sqimap_fread because
+    we need to split te result on \n and fread doesn't stop at \n. That
+    means we also should provide $results from sqimap_fread (by ref) to
+    te function and set $no_return to false. The $filter function for
+    quoted printable should handle unsetting of $results.
+    */
+    /*
+    TODO 2: find out how we write to the output stream php://stdout. fwrite
+    doesn't work because 'php://stdout isn't a stream.
+    */
 
+    return;
+/*
     fputs ($imap_stream, "$sid FETCH $id BODY[$ent_id]\r\n");
     $cnt = 0;
     $continue = true;
-    $read = fgets ($imap_stream,4096);
+    $read = fgets ($imap_stream,8192);
+
+
     // This could be bad -- if the section has sqimap_session_id() . ' OK'
     // or similar, it will kill the download.
-    while (!ereg("^".$sid." (OK|BAD|NO)(.*)$", $read, $regs)) {
-      if (trim($read) == ')==') {
-          $read1 = $read;
-          $read = fgets ($imap_stream,4096);
-          if (ereg("^".$sid." (OK|BAD|NO)(.*)$", $read, $regs)) {
-              return;
-          } else {
-              echo decodeBody($read1, $encoding) .
-                   decodeBody($read, $encoding);
-          }
-      } else if ($cnt) {
-          echo decodeBody($read, $encoding);
-      }
-      $read = fgets ($imap_stream,4096);
-      $cnt++;
+    while (!ereg("^".$sid_s." (OK|BAD|NO)(.*)$", $read, $regs)) {
+        if (trim($read) == ')==') {
+            $read1 = $read;
+            $read = fgets ($imap_stream,4096);
+            if (ereg("^".$sid." (OK|BAD|NO)(.*)$", $read, $regs)) {
+                return;
+            } else {
+                echo decodeBody($read1, $encoding) .
+                    decodeBody($read, $encoding);
+            }
+        } else if ($cnt) {
+            echo decodeBody($read, $encoding);
+        }
+        $read = fgets ($imap_stream,4096);
+        $cnt++;
+//      break;
     }
+*/
 }
 
 /* -[ END MIME DECODING ]----------------------------------------------------------- */
 
-
-
-/* This is the first function called.  It decides if this is a multipart
-   message or if it should be handled as a single entity
- */
-function decodeMime ($imap_stream, &$header) {
-    global $username, $key, $imapServerAddress, $imapPort;
-    return mime_structure ($imap_stream, $header);
-}
-
-// This is here for debugging purposese.  It will print out a list
-// of all the entity IDs that are in the $message object.
-
+/* This is here for debugging purposes.  It will print out a list
+* of all the entity IDs that are in the $message object.
+*/
 function listEntities ($message) {
-if ($message) {
- if ($message->header->entity_id)
- echo "<tt>" . $message->header->entity_id . ' : ' . $message->header->type0 . '/' . $message->header->type1 . '<br>';
- for ($i = 0; $message->entities[$i]; $i++) {
-    $msg = listEntities($message->entities[$i], $ent_id);
-    if ($msg)
-       return $msg;
- }
-}
-}
-
-
-/* returns a $message object for a particular entity id */
-function getEntity ($message, $ent_id) {
     if ($message) {
-        if ($message->header->entity_id == $ent_id && strlen($ent_id) == strlen($message->header->entity_id))
-       {
-            return $message;
-        } else {
-            for ($i = 0; isset($message->entities[$i]); $i++) {
-                $msg = getEntity ($message->entities[$i], $ent_id);
-                if ($msg) {
-                    return $msg;
-               }
+        echo "<tt>" . $message->entity_id . ' : ' . $message->type0 . '/' . $message->type1 . ' parent = '. $message->parent->entity_id. '<br />';
+        for ($i = 0; isset($message->entities[$i]); $i++) {
+            echo "$i : ";
+            $msg = listEntities($message->entities[$i]);
+
+            if ($msg) {
+                echo "return: ";
+                return $msg;
             }
         }
     }
 }
 
-/*
- * figures out what entity to display and returns the $message object
- * for that entity.
- */
-function findDisplayEntity ($msg, $textOnly = true, $entity = array() )   {
-    global $show_html_default;
-    
-    $found = false;    
-    if ($msg) {
-        $type = $msg->header->type0.'/'.$msg->header->type1;
-        if ( $type == 'multipart/alternative') {
-           $msg = findAlternativeEntity($msg, $textOnly);
-           if (count($msg->entities) == 0) {
-               $entity[] = $msg->header->entity_id;
-           } else {
-               $found = true;
-                $entity =findDisplayEntity($msg,$textOnly, $entity);
-           }
-       } else  if ( $type == 'multipart/related') {
-            $msgs = findRelatedEntity($msg);
-           for ($i = 0; $i < count($msgs); $i++) {
-               $msg = $msgs[$i];
-               if (count($msg->entities) == 0) {
-                   $entity[] = $msg->header->entity_id;
-               } else {
-                   $found = true;
-                    $entity =findDisplayEntity($msg,$textOnly, $entity);
-               }
-           }
-       } else if ( count($entity) == 0 &&
-             $msg->header->type0 == 'text' &&
-             ( $msg->header->type1 == 'plain' ||
-               $msg->header->type1 == 'html' ) &&
-             isset($msg->header->entity_id) ) {
-            if (count($msg->entities) == 0) {
-               $entity[] = $msg->header->entity_id;
-            } 
-        } 
-       $i = 0;
-       while ( isset($msg->entities[$i]) && count($entity) == 0 && !$found )  {
-           $entity = findDisplayEntity($msg->entities[$i], $textOnly, $entity);
-           $i++;
-       }
-    }
-    if ( !isset($entity[0]) ) {
-        $entity[]="";
-    }
-    return( $entity );
-}
+function getPriorityStr($priority) {
+    $priority_level = substr($priority,0,1);
 
-/* Shows the HTML version */
-function findDisplayEntityHTML ($message) {
+    switch($priority_level) {
+        /* Check for a higher then normal priority. */
+        case '1':
+        case '2':
+            $priority_string = _("High");
+            break;
 
-    if ( $message->header->type0 == 'text' &&
-         $message->header->type1 == 'html' &&
-         isset($message->header->entity_id)) {
-        return $message->header->entity_id;
-    }
-    for ($i = 0; isset($message->entities[$i]); $i ++) {
-       if ( $message->header->type0 == 'message' &&
-           $message->header->type1 == 'rfc822' &&
-            isset($message->header->entity_id)) {
-           return 0;
-       }
-       
-        $entity = findDisplayEntityHTML($message->entities[$i]);
-        if ($entity != 0) {
-            return $entity;
-        }
-    }
+        /* Check for a lower then normal priority. */
+        case '4':
+        case '5':
+            $priority_string = _("Low");
+            break;
 
-    return 0;
-}
+        /* Check for a normal priority. */
+        case '3':
+        default:
+            $priority_level = '3';
+            $priority_string = _("Normal");
+            break;
 
-function findAlternativeEntity ($message, $textOnly) {
-    global $show_html_default;
-    /* if we are dealing with alternative parts then we choose the best 
-     * viewable message supported by SM.
-     */
-    if ($show_html_default && !$textOnly) {     
-       $alt_order = array ('text/plain','text/html');
-    } else {
-       $alt_order = array ('text/plain');
     }
-    $best_view = 0;
-    $ent_id = 0;
-    $k = 0; 
-    for ($i = 0; $i < count($message->entities); $i ++) {
-        $type = $message->entities[$i]->header->type0.'/'.$message->entities[$i]->header->type1;
-       if ($type == 'multipart/related') {
-          $type = $message->entities[$i]->header->type;
-       }
-       for ($j = $k; $j < count($alt_order); $j++) {
-           if ($alt_order[$j] == $type && $j > $best_view) {
-               $best_view = $j;
-               $ent_id = $i;
-               $k = $j;
-           }
-       }
-    }
-    return $message->entities[$ent_id];
+    return $priority_string;
 }
 
-function findRelatedEntity ($message) {
-    $msgs = array(); 
-    for ($i = 0; $i < count($message->entities); $i ++) {
-        $type = $message->entities[$i]->header->type0.'/'.$message->entities[$i]->header->type1;
-        if ($message->header->type == $type) {
-           $msgs[] = $message->entities[$i];
-       }
-    }
-    return $msgs;
-}    
+/* returns a $message object for a particular entity id */
+function getEntity ($message, $ent_id) {
+    return $message->getEntity($ent_id);
+}
 
-/*
- * translateText
- * Extracted from strings.php 23/03/2002
- */
+/* translateText
+* Extracted from strings.php 23/03/2002
+*/
 
 function translateText(&$body, $wrap_at, $charset) {
-    global $where, $what; /* from searching */
-    global $color; /* color theme */
+    global $where, $what;   /* from searching */
+    global $color;          /* color theme */
 
-    require_once('../functions/url_parser.php');
+    require_once(SM_PATH . 'functions/url_parser.php');
 
     $body_ary = explode("\n", $body);
-    $PriorQuotes = 0;
     for ($i=0; $i < count($body_ary); $i++) {
         $line = $body_ary[$i];
         if (strlen($line) - 2 >= $wrap_at) {
-            sqWordWrap($line, $wrap_at);
+            sqWordWrap($line, $wrap_at, $charset);
         }
         $line = charset_decode($charset, $line);
         $line = str_replace("\t", '        ', $line);
 
         parseUrl ($line);
 
-        $Quotes = 0;
+        $quotes = 0;
         $pos = 0;
-        $j = strlen( $line );
+        $j = strlen($line);
 
-        while ( $pos < $j ) {
+        while ($pos < $j) {
             if ($line[$pos] == ' ') {
-                $pos ++;
+                $pos++;
             } else if (strpos($line, '&gt;', $pos) === $pos) {
                 $pos += 4;
-                $Quotes ++;
+                $quotes++;
             } else {
                 break;
             }
         }
-        
-        if ($Quotes > 1) {
-            if (! isset($color[14])) {
-                $color[14] = '#FF0000';
-            }
-            $line = '<FONT COLOR="' . $color[14] . '">' . $line . '</FONT>';
-        } elseif ($Quotes) {
-            if (! isset($color[13])) {
+
+        if ($quotes % 2) {
+            if (!isset($color[13])) {
                 $color[13] = '#800000';
             }
-            $line = '<FONT COLOR="' . $color[13] . '">' . $line . '</FONT>';
+            $line = '<font color="' . $color[13] . '">' . $line . '</font>';
+        } elseif ($quotes) {
+            if (!isset($color[14])) {
+                $color[14] = '#FF0000';
+            }
+            $line = '<font color="' . $color[14] . '">' . $line . '</font>';
         }
-        
+
         $body_ary[$i] = $line;
     }
     $body = '<pre>' . implode("\n", $body_ary) . '</pre>';
 }
 
-/* debugfunction for looping through entities and displaying correct entities */
-function listMyEntities ($message) {
-
-if ($message) {
-    if ($message->header->entity_id) {
-       echo "<tt>" . $message->header->entity_id . ' : ' . $message->header->type0 . '/' . $message->header->type1 . '<br>';
-    } 
-    if (!($message->header->type0 == 'message' &&  $message->header->type1 == 'rfc822')) {
-       if (isset($message->header->boundary) ) {
-           $ent_id = $message->header->entity_id;
-           $var = $message->header->boundary;
-           if ($var !='')
-           echo "<b>$ent_id boundary = $var</b><br>";
-       } 
-       if (isset($message->header->type) ) {
-           $var = $message->header->type;
-           if ($var !='')
-           echo "<b>$ent_id type = $var</b><br>";
-       } 
-       for ($i = 0; $message->entities[$i]; $i++) {
-           $msg = listMyEntities($message->entities[$i]);
-       }
-
-       if ($msg )  return $msg;
+/**
+ * This returns a parsed string called $body. That string can then
+ * be displayed as the actual message in the HTML. It contains
+ * everything needed, including HTML Tags, Attachments at the
+ * bottom, etc.
+ * 
+ * Since 1.2.0 function uses message_body hook.
+ * Till 1.3.0 function included output of formatAttachments().
+ *
+ * @param resource $imap_stream imap connection resource
+ * @param object $message squirrelmail message object
+ * @param array $color squirrelmail color theme array
+ * @param integer $wrap_at number of characters per line
+ * @param string $ent_num (since 1.3.0) message part id
+ * @param integer $id (since 1.3.0) message id
+ * @param string $mailbox (since 1.3.0) imap folder name
+ * @param boolean $clean (since 1.5.1) Do not output stuff that's irrelevant for the printable version.
+ * @return string html formated message text
+ */
+function formatBody($imap_stream, $message, $color, $wrap_at, $ent_num, $id, $mailbox='INBOX', $clean=FALSE) {
+    /* This if statement checks for the entity to show as the
+    * primary message. To add more of them, just put them in the
+    * order that is their priority.
+    */
+    global $startMessage, $languages, $squirrelmail_language,
+        $show_html_default, $sort, $has_unsafe_images, $passed_ent_id;
+
+    if( !sqgetGlobalVar('view_unsafe_images', $view_unsafe_images, SQ_GET) ) {
+        $view_unsafe_images = false;
     }
-}
-
-}
-
-
-
-/* This returns a parsed string called $body. That string can then
-be displayed as the actual message in the HTML. It contains
-everything needed, including HTML Tags, Attachments at the
-bottom, etc.
-*/
-function formatBody($imap_stream, $message, $color, $wrap_at, $ent_num) {
-    // this if statement checks for the entity to show as the
-    // primary message. To add more of them, just put them in the
-    // order that is their priority.
-    global $startMessage, $username, $key, $imapServerAddress, $imapPort,
-           $show_html_default, $has_unsafe_images, $view_unsafe_images, $sort;
-
-    $has_unsafe_images = 0;
-
-    $id = $message->header->id;
-
-    $urlmailbox = urlencode($message->header->mailbox);
 
+    $body = '';
+    $urlmailbox = urlencode($mailbox);
     $body_message = getEntity($message, $ent_num);
     if (($body_message->header->type0 == 'text') ||
         ($body_message->header->type0 == 'rfc822')) {
-       $body = mime_fetch_body ($imap_stream, $id, $ent_num);
-       
+        $body = mime_fetch_body ($imap_stream, $id, $ent_num);
         $body = decodeBody($body, $body_message->header->encoding);
+
+        if (isset($languages[$squirrelmail_language]['XTRA_CODE']) &&
+            function_exists($languages[$squirrelmail_language]['XTRA_CODE'] . '_decode')) {
+            if (mb_detect_encoding($body) != 'ASCII') {
+                $body = call_user_func($languages[$squirrelmail_language]['XTRA_CODE'] . '_decode',$body);
+            }
+        }
         $hookResults = do_hook("message_body", $body);
         $body = $hookResults[1];
-        // If there are other types that shouldn't be formatted, add
-        // them here
+
+        /* If there are other types that shouldn't be formatted, add
+        * them here.
+        */
+
         if ($body_message->header->type1 == 'html') {
-            if ( $show_html_default <> 1 ) {
-                $body = strip_tags( $body );
-                translateText($body, $wrap_at, $body_message->header->charset);
+            if ($show_html_default <> 1) {
+                $entity_conv = array('&nbsp;' => ' ',
+                                    '<p>'    => "\n",
+                                    '<P>'    => "\n",
+                                    '<br>'   => "\n",
+                                    '<BR>'   => "\n",
+                                    '<br />' => "\n",
+                                    '<BR />' => "\n",
+                                    '&gt;'   => '>',
+                                    '&lt;'   => '<');
+                $body = strtr($body, $entity_conv);
+                $body = strip_tags($body);
+                $body = trim($body);
+                translateText($body, $wrap_at,
+                            $body_message->header->getParameter('charset'));
             } else {
-                $body = magicHTML( $body, $id, $message );
+                $body = magicHTML($body, $id, $message, $mailbox);
             }
         } else {
-            translateText($body, $wrap_at, $body_message->header->charset);
+            translateText($body, $wrap_at,
+                        $body_message->header->getParameter('charset'));
+        }
+
+        // if this is the clean display (i.e. printer friendly), stop here.
+        if ( $clean ) {
+            return $body;
+        }
+
+        $link = 'passed_id=' . $id . '&amp;ent_id='.$ent_num.
+                '&amp;mailbox=' . $urlmailbox .'&amp;sort=' . $sort .
+                '&amp;startMessage=' . $startMessage . '&amp;show_more=0';
+        if (isset($passed_ent_id)) {
+            $link .= '&amp;passed_ent_id='.$passed_ent_id;
         }
-        $body .= "<CENTER><SMALL><A HREF=\"../src/download.php?absolute_dl=true&amp;passed_id=$id&amp;passed_ent_id=$ent_num&amp;mailbox=$urlmailbox&amp;showHeaders=1\">". _("Download this as a file") ."</A></SMALL></CENTER><BR>";
-        if ($has_unsafe_images) {
-            if ($view_unsafe_images) {
-                $body .= "<CENTER><SMALL><A HREF=\"read_body.php?passed_id=$id&amp;mailbox=$urlmailbox&amp;sort=$sort&amp;startMessage=$startMessage&amp;show_more=0\">". _("Hide Unsafe Images") ."</A></SMALL></CENTER><BR>\n";
+        $body .= '<center><small><a href="download.php?absolute_dl=true&amp;' .
+                $link . '">' . _("Download this as a file") .  '</a>';
+        if ($view_unsafe_images) {
+            $text = _("Hide Unsafe Images");
+        } else {
+            if (isset($has_unsafe_images) && $has_unsafe_images) {
+                $link .= '&amp;view_unsafe_images=1';
+                $text = _("View Unsafe Images");
             } else {
-                $body .= "<CENTER><SMALL><A HREF=\"read_body.php?passed_id=$id&amp;mailbox=$urlmailbox&amp;sort=$sort&amp;startMessage=$startMessage&amp;show_more=0&amp;view_unsafe_images=1\">". _("View Unsafe Images") ."</A></SMALL></CENTER><BR>\n";
+                $text = '';
             }
         }
-
-        /** Display the ATTACHMENTS: message if there's more than one part **/
-        if (isset($message->entities[1])) {
-           /* Header-type alternative means we choose the best one to display 
-              so don't show the alternatives as attachment. Header-type related
-              means that the attachments are already part of the related message.
-           */   
-           if ($message->header->type1 !='related' && $message->header->type1 !='alternative') {
-               $body .= formatAttachments ($message, $ent_num, $message->header->mailbox, $id);
-           }
+        if($text != '') {
+            $body .= '&nbsp;|&nbsp;<a href="read_body.php?' . $link . '">' . $text . '</a>';
         }
-    } else {
-        $body = formatAttachments ($message, -1, $message->header->mailbox, $id);
+        $body .= '</small></center><br />' . "\n";
     }
-    return ($body);
+    return $body;
 }
 
-/*
- * A recursive function that returns a list of attachments with links
- * to where to download these attachments
+/**
+ * Displays attachment links and information
+ * FIXME: SM_PATH is used in URLs
+ *
+ * Since 1.3.0 function is not included in formatBody() call.
+ *
+ * Since 1.0.2 uses attachment $type0/$type1 hook.
+ * Since 1.2.5 uses attachment $type0/* hook.
+ * Since 1.5.0 uses attachments_bottom hook.
+ * 
+ * @param object $message SquirrelMail message object
+ * @param array $exclude_id message parts that are not attachments.
+ * @param string $mailbox mailbox name
+ * @param integer $id message id
+ * @return string html formated attachment information.
  */
-function formatAttachments($message, $ent_id, $mailbox, $id) {
-    global $where, $what;
-    global $startMessage, $color;
-    static $ShownHTML = 0;
+function formatAttachments($message, $exclude_id, $mailbox, $id) {
+    global $where, $what, $startMessage, $color, $passed_ent_id;
 
-    $body = '';
-    if ($ShownHTML == 0) {
-
-        $ShownHTML = 1;
-        $body .= "<TABLE WIDTH=\"100%\" CELLSPACING=0 CELLPADDING=2 BORDER=0 BGCOLOR=\"$color[0]\"><TR>\n" .
-                "<TH ALIGN=\"left\" BGCOLOR=\"$color[9]\"><B>\n" .
-                _("Attachments") . ':' .
-                "</B></TH></TR><TR><TD>\n" .
-                "<TABLE CELLSPACING=0 CELLPADDING=1 BORDER=0>\n" .
-                formatAttachments($message, $ent_id, $mailbox, $id) .
-                "</TABLE></TD></TR></TABLE>";
-
-    } else if ($message) {
-       $header = $message->header;
-        $type0 = strtolower($header->type0);
-        $type1 = strtolower($header->type1);
-       $name = '';
-       if (isset($header->name)) {
-           $name = decodeHeader($header->name);
-       }
-       if ($type0 =='message' && $type1 == 'rfc822') {
-        
-            $filename = decodeHeader($message->header->filename);
-            if (trim($filename) == '') {
-                if (trim($name) == '') {
-                    $display_filename = 'untitled-[' . $message->header->entity_id . ']' ;
-                } else {
-                    $display_filename = $name;
-                    $filename = $name;
-                }
-            } else {
-                $display_filename = $filename;
-            }
+    $att_ar = $message->getAttachments($exclude_id);
 
-            $urlMailbox = urlencode($mailbox);
-            $ent = urlencode($message->header->entity_id);
+    if (!count($att_ar)) return '';
 
-            $DefaultLink =
-                "../src/download.php?startMessage=$startMessage&amp;passed_id=$id&amp;mailbox=$urlMailbox&amp;passed_ent_id=$ent";
-            if ($where && $what) {
-                $DefaultLink .= '&amp;where=' . urlencode($where) . '&amp;what=' . urlencode($what);
-            }
-            $Links['download link']['text'] = _("download");
-            $Links['download link']['href'] =
-                "../src/download.php?absolute_dl=true&amp;passed_id=$id&amp;mailbox=$urlMailbox&amp;passed_ent_id=$ent";
-            $ImageURL = '';
-
-            /* this executes the attachment hook with a specific MIME-type.
-                * if that doens't have results, it tries if there's a rule
-                * for a more generic type. */
-            $HookResults = do_hook("attachment $type0/$type1", $Links,
-                $startMessage, $id, $urlMailbox, $ent, $DefaultLink, $display_filename, $where, $what);
-            if(count($HookResults[1]) <= 1) {
-                $HookResults = do_hook("attachment $type0/*", $Links,
-                $startMessage, $id, $urlMailbox, $ent, $DefaultLink,
-                $display_filename, $where, $what);
-            }
-
-            $Links = $HookResults[1];
-            $DefaultLink = $HookResults[6];
-
-            $body .= '<TR><TD>&nbsp;&nbsp;</TD><TD>' .
-                        "<A HREF=\"$DefaultLink\">$display_filename</A>&nbsp;</TD>" .
-                        '<TD><SMALL><b>' . show_readable_size($message->header->size) .
-                        '</b>&nbsp;&nbsp;</small></TD>' .
-                        "<TD><SMALL>[ $type0/$type1 ]&nbsp;</SMALL></TD>" .
-                        '<TD><SMALL>';
-            if ($message->header->description) {
-                $body .= '<b>' . htmlspecialchars(_($message->header->description)) . '</b>';
-            }
-            $body .= '</SMALL></TD><TD><SMALL>&nbsp;';
+    $attachments = '';
 
+    $urlMailbox = urlencode($mailbox);
 
-            $SkipSpaces = 1;
-            foreach ($Links as $Val) {
-                if ($SkipSpaces) {
-                    $SkipSpaces = 0;
-                } else {
-                    $body .= '&nbsp;&nbsp;|&nbsp;&nbsp;';
-                }
-                $body .= '<a href="' . $Val['href'] . '">' .  $Val['text'] . '</a>';
+    foreach ($att_ar as $att) {
+        $ent = $att->entity_id;
+        $header = $att->header;
+        $type0 = strtolower($header->type0);
+        $type1 = strtolower($header->type1);
+        $name = '';
+        $links['download link']['text'] = _("Download");
+        $links['download link']['href'] = SM_PATH .
+                "src/download.php?absolute_dl=true&amp;passed_id=$id&amp;mailbox=$urlMailbox&amp;ent_id=$ent";
+        if ($type0 =='message' && $type1 == 'rfc822') {
+            $default_page = SM_PATH . 'src/read_body.php';
+            $rfc822_header = $att->rfc822_header;
+            $filename = $rfc822_header->subject;
+            if (trim( $filename ) == '') {
+                $filename = 'untitled-[' . $ent . ']' ;
             }
-
-            unset($Links);
-
-            $body .= "</SMALL></TD></TR>\n";
-            
-           return( $body );    
-       
-        } elseif (!$message->entities) {
-
-            $type0 = strtolower($message->header->type0);
-            $type1 = strtolower($message->header->type1);
-            $name = decodeHeader($message->header->name);
-
-            if ($message->header->entity_id != $ent_id) {
-            $filename = decodeHeader($message->header->filename);
-            if (trim($filename) == '') {
-                if (trim($name) == '') {
-                    if ( trim( $message->header->id ) == '' )
-                        $display_filename = 'untitled-[' . $message->header->entity_id . ']' ;
-                    else
-                        $display_filename = 'cid: ' . $message->header->id;
-                    // $display_filename = 'untitled-[' . $message->header->entity_id . ']' ;
-                } else {
-                    $display_filename = $name;
-                    $filename = $name;
-                }
+            $from_o = $rfc822_header->from;
+            if (is_object($from_o)) {
+                $from_name = decodeHeader($from_o->getAddress(false));
+            } else {
+                $from_name = _("Unknown sender");
+            }
+            $description = $from_name;
+        } else {
+            $default_page = SM_PATH . 'src/download.php';
+            $filename = $att->getFilename();
+            if ($header->description) {
+                $description = decodeHeader($header->description);
             } else {
-                $display_filename = $filename;
+                $description = '';
             }
+        }
 
-            $urlMailbox = urlencode($mailbox);
-            $ent = urlencode($message->header->entity_id);
+        $display_filename = $filename;
+        if (isset($passed_ent_id)) {
+            $passed_ent_id_link = '&amp;passed_ent_id='.$passed_ent_id;
+        } else {
+            $passed_ent_id_link = '';
+        }
+        $defaultlink = $default_page . "?startMessage=$startMessage"
+                    . "&amp;passed_id=$id&amp;mailbox=$urlMailbox"
+                    . '&amp;ent_id='.$ent.$passed_ent_id_link;
+        if ($where && $what) {
+        $defaultlink .= '&amp;where='. urlencode($where).'&amp;what='.urlencode($what);
+        }
 
-            $DefaultLink =
-                "../src/download.php?startMessage=$startMessage&amp;passed_id=$id&amp;mailbox=$urlMailbox&amp;passed_ent_id=$ent";
-            if ($where && $what) {
-              $DefaultLink = '&amp;where='. urlencode($where).'&amp;what='.urlencode($what);
-            }
-            $Links['download link']['text'] = _("download");
-            $Links['download link']['href'] =
-                "../src/download.php?absolute_dl=true&amp;passed_id=$id&amp;mailbox=$urlMailbox&amp;passed_ent_id=$ent";
-            $ImageURL = '';
-
-            /* this executes the attachment hook with a specific MIME-type.
-                * if that doens't have results, it tries if there's a rule
-                * for a more generic type. */
-            $HookResults = do_hook("attachment $type0/$type1", $Links,
-                $startMessage, $id, $urlMailbox, $ent, $DefaultLink,
-                $display_filename, $where, $what);
-            if(count($HookResults[1]) <= 1) {
-                $HookResults = do_hook("attachment $type0/*", $Links,
-                $startMessage, $id, $urlMailbox, $ent, $DefaultLink,
-                $display_filename, $where, $what);
-            }
+        /* This executes the attachment hook with a specific MIME-type.
+        * If that doesn't have results, it tries if there's a rule
+        * for a more generic type.
+        */
+        $hookresults = do_hook("attachment $type0/$type1", $links,
+                            $startMessage, $id, $urlMailbox, $ent, $defaultlink,
+                            $display_filename, $where, $what);
+        if(count($hookresults[1]) <= 1) {
+            $hookresults = do_hook("attachment $type0/*", $links,
+                                $startMessage, $id, $urlMailbox, $ent, $defaultlink,
+                                $display_filename, $where, $what);
+        }
 
-            $Links = $HookResults[1];
-            $DefaultLink = $HookResults[6];
-
-            $body .= '<TR><TD>&nbsp;&nbsp;</TD><TD>' .
-                        "<A HREF=\"$DefaultLink\">$display_filename</A>&nbsp;</TD>" .
-                        '<TD><SMALL><b>' . show_readable_size($message->header->size) .
-                        '</b>&nbsp;&nbsp;</small></TD>' .
-                        "<TD><SMALL>[ $type0/$type1 ]&nbsp;</SMALL></TD>" .
-                        '<TD><SMALL>';
-            if ($message->header->description) {
-                $body .= '<b>' . htmlspecialchars(_($message->header->description)) . '</b>';
+        $links = $hookresults[1];
+        $defaultlink = $hookresults[6];
+
+        $attachments .= '<tr><td>' .
+                        '<a href="'.$defaultlink.'">'.decodeHeader($display_filename).'</a>&nbsp;</td>' .
+                        '<td><small><b>' . show_readable_size($header->size) .
+                        '</b>&nbsp;&nbsp;</small></td>' .
+                        '<td><small>[ '.htmlspecialchars($type0).'/'.htmlspecialchars($type1).' ]&nbsp;</small></td>' .
+                        '<td><small>';
+        $attachments .= '<b>' . $description . '</b>';
+        $attachments .= '</small></td><td><small>&nbsp;';
+
+        $skipspaces = 1;
+        foreach ($links as $val) {
+            if ($skipspaces) {
+                $skipspaces = 0;
+            } else {
+                $attachments .= '&nbsp;&nbsp;|&nbsp;&nbsp;';
             }
-            $body .= '</SMALL></TD><TD><SMALL>&nbsp;';
+            $attachments .= '<a href="' . $val['href'] . '">' . (isset($val['text']) && !empty($val['text']) ? $val['text'] : '') . (isset($val['extra']) && !empty($val['extra']) ? $val['extra'] : '') . '</a>';
+        }
+        unset($links);
+        $attachments .= "</td></tr>\n";
+    }
+    $attachmentadd = do_hook_function('attachments_bottom',$attachments);
+    if ($attachmentadd != '')
+        $attachments = $attachmentadd;
+    return $attachments;
+}
 
+function sqimap_base64_decode(&$string) {
 
-            $SkipSpaces = 1;
-            foreach ($Links as $Val) {
-                if ($SkipSpaces) {
-                    $SkipSpaces = 0;
-                } else {
-                    $body .= '&nbsp;&nbsp;|&nbsp;&nbsp;';
-                }
-                $body .= '<a href="' . $Val['href'] . '">' .  $Val['text'] . '</a>';
-            }
+    // Base64 encoded data goes in pairs of 4 bytes. To achieve on the
+    // fly decoding (to reduce memory usage) you have to check if the
+    // data has incomplete pairs
 
-            unset($Links);
+    // Remove the noise in order to check if the 4 bytes pairs are complete
+    $string = str_replace(array("\r\n","\n", "\r", " "),array('','','',''),$string);
 
-            $body .= "</SMALL></TD></TR>\n";
-            }
+    $sStringRem = '';
+    $iMod = strlen($string) % 4;
+    if ($iMod) {
+        $sStringRem = substr($string,-$iMod);
+        // Check if $sStringRem contains padding characters
+        if (substr($sStringRem,-1) != '=') {
+            $string = substr($string,0,-$iMod);
         } else {
-            for ($i = 0; $i < count($message->entities); $i++) {
-                $body .= formatAttachments($message->entities[$i], $ent_id, $mailbox, $id);
-            }
+            $sStringRem = '';
         }
     }
-    return( $body );
+    $string = base64_decode($string);
+    return $sStringRem;
 }
 
 
-/** this function decodes the body depending on the encoding type. **/
+/* This function decodes the body depending on the encoding type. */
 function decodeBody($body, $encoding) {
-  $body = str_replace("\r\n", "\n", $body);
-  $encoding = strtolower($encoding);
+    global $show_html_default;
 
-  global $show_html_default;
+    $body = str_replace("\r\n", "\n", $body);
+    $encoding = strtolower($encoding);
 
-  if ($encoding == 'quoted-printable' ||
-      $encoding == 'quoted_printable') {
-     $body = quoted_printable_decode($body);
+    $encoding_handler = do_hook_function('decode_body', $encoding);
 
 
-     while (ereg("=\n", $body))
-        $body = ereg_replace ("=\n", "", $body);
+    // plugins get first shot at decoding the body
+    //
+    if (!empty($encoding_handler) && function_exists($encoding_handler)) {
+        $body = $encoding_handler('decode', $body);
 
-  } else if ($encoding == 'base64') {
-     $body = base64_decode($body);
-  }
+    } else if ($encoding == 'quoted-printable' ||
+        $encoding == 'quoted_printable') {
+        $body = quoted_printable_decode($body);
 
-  // All other encodings are returned raw.
-  return $body;
+        while (ereg("=\n", $body)) {
+            $body = ereg_replace ("=\n", '', $body);
+        }
+
+    } else if ($encoding == 'base64') {
+        $body = base64_decode($body);
+    }
+
+    // All other encodings are returned raw.
+    return $body;
 }
 
-/*
- * This functions decode strings that is encoded according to
- * RFC1522 (MIME Part Two: Message Header Extensions for Non-ASCII Text).
- * Patched by Christian Schmidt <christian@ostenfeld.dk>  23/03/2002
- */
-function decodeHeader ($string, $utfencode=true) {
+/**
+* Decodes headers
+*
+* This functions decode strings that is encoded according to
+* RFC1522 (MIME Part Two: Message Header Extensions for Non-ASCII Text).
+* Patched by Christian Schmidt <christian@ostenfeld.dk>  23/03/2002
+*
+* @param string $string header string that has to be made readable
+* @param boolean $utfencode change message in order to be readable on user's charset. defaults to true
+* @param boolean $htmlsave preserve spaces and sanitize html special characters. defaults to true
+* @param boolean $decide decide if string can be utfencoded. defaults to false
+* @return string decoded header string
+*/
+function decodeHeader ($string, $utfencode=true,$htmlsave=true,$decide=false) {
+    global $languages, $squirrelmail_language,$default_charset;
     if (is_array($string)) {
         $string = implode("\n", $string);
     }
+
+    if (isset($languages[$squirrelmail_language]['XTRA_CODE']) &&
+        function_exists($languages[$squirrelmail_language]['XTRA_CODE'] . '_decodeheader')) {
+        $string = call_user_func($languages[$squirrelmail_language]['XTRA_CODE'] . '_decodeheader', $string);
+        // Do we need to return at this point?
+        // return $string;
+    }
     $i = 0;
-    while (preg_match('/^(.{' . $i . '})(.*)=\?([^?]*)\?(Q|B)\?([^?]*)\?=/Ui', 
-                      $string, $res)) {
-        $prefix = $res[1];
-        // Ignore white-space between consecutive encoded-words
-        if (strspn($res[2], " \t") != strlen($res[2])) {
-            $prefix .= $res[2];
+    $iLastMatch = -2;
+    $encoded = true;
+
+    $aString = explode(' ',$string);
+    $ret = '';
+    foreach ($aString as $chunk) {
+        if ($encoded && $chunk === '') {
+            continue;
+        } elseif ($chunk === '') {
+            $ret .= ' ';
+            continue;
         }
+        $encoded = false;
+        /* if encoded words are not separated by a linear-space-white we still catch them */
+        $j = $i-1;
+
+        while ($match = preg_match('/^(.*)=\?([^?]*)\?(Q|B)\?([^?]*)\?=(.*)$/Ui',$chunk,$res)) {
+            /* if the last chunk isn't an encoded string then put back the space, otherwise don't */
+            if ($iLastMatch !== $j) {
+                if ($htmlsave) {
+                    $ret .= '&#32;';
+                } else {
+                    $ret .= ' ';
+                }
+            }
+            $iLastMatch = $i;
+            $j = $i;
+            if ($htmlsave) {
+                $ret .= htmlspecialchars($res[1]);
+            } else {
+                $ret .= $res[1];
+            }
+            $encoding = ucfirst($res[3]);
 
-        if (ucfirst($res[4]) == 'B') {
-            $replace = base64_decode($res[5]);
-        } else {
-            $replace = str_replace('_', ' ', $res[5]);
-            $replace = preg_replace('/=([0-9a-f]{2})/ie', 'chr(hexdec("\1"))', 
+            /* decide about valid decoding */
+            if ($decide && is_conversion_safe($res[2])) {
+            $utfencode=true;
+            $can_be_encoded=true;
+            } else {
+            $can_be_encoded=false;
+            }
+            switch ($encoding)
+            {
+            case 'B':
+                $replace = base64_decode($res[4]);
+                if ($utfencode) {
+                    if ($can_be_encoded) {
+                        /* convert string to different charset,
+                        * if functions asks for it (usually in compose)
+                        */
+                        $ret .= charset_convert($res[2],$replace,$default_charset);
+                    } else {
+                        // convert string to html codes in order to display it
+                        $ret .= charset_decode($res[2],$replace);
+                    }
+                } else {
+                    if ($htmlsave) {
+                        $replace = htmlspecialchars($replace);
+                    }
+                    $ret.= $replace;
+                }
+                break;
+            case 'Q':
+                $replace = str_replace('_', ' ', $res[4]);
+                $replace = preg_replace('/=([0-9a-f]{2})/ie', 'chr(hexdec("\1"))',
                                     $replace);
-            /* Only encode into entities by default. Some places
-               don't need the encoding, like the compose form. */
-            if ($utfencode) {
-                $replace = charset_decode($res[3], $replace);
+                if ($utfencode) {
+                if ($can_be_encoded) {
+                    /* convert string to different charset,
+                    * if functions asks for it (usually in compose)
+                    */
+                    $replace = charset_convert($res[2], $replace,$default_charset);
+                } else {
+                    // convert string to html codes in order to display it
+                    $replace = charset_decode($res[2], $replace);
+                }
+                } else {
+                    if ($htmlsave) {
+                        $replace = htmlspecialchars($replace);
+                    }
+                }
+                $ret .= $replace;
+                break;
+            default:
+                break;
+            }
+            $chunk = $res[5];
+            $encoded = true;
+        }
+        if (!$encoded) {
+            if ($htmlsave) {
+                $ret .= '&#32;';
+            } else {
+                $ret .= ' ';
             }
         }
-        $string = $prefix . $replace . substr($string, strlen($res[0]));
-        $i = strlen($prefix) + strlen($replace);
+
+        if (!$encoded && $htmlsave) {
+            $ret .= htmlspecialchars($chunk);
+        } else {
+            $ret .= $chunk;
+        }
+        ++$i;
     }
-    return( $string );
+    /* remove the first added space */
+    if ($ret) {
+        if ($htmlsave) {
+            $ret = substr($ret,5);
+        } else {
+            $ret = substr($ret,1);
+        }
+    }
+
+    return $ret;
 }
 
-/*
- * Encode a string according to RFC 1522 for use in headers if it
- * contains 8-bit characters or anything that looks like it should
- * be encoded.
- */
+/**
+* Encodes header as quoted-printable
+*
+* Encode a string according to RFC 1522 for use in headers if it
+* contains 8-bit characters or anything that looks like it should
+* be encoded.
+*
+* @param string $string header string, that has to be encoded
+* @return string quoted-printable encoded string
+*/
 function encodeHeader ($string) {
-    global $default_charset;
+    global $default_charset, $languages, $squirrelmail_language;
+
+    if (isset($languages[$squirrelmail_language]['XTRA_CODE']) &&
+        function_exists($languages[$squirrelmail_language]['XTRA_CODE'] . '_encodeheader')) {
+        return  call_user_func($languages[$squirrelmail_language]['XTRA_CODE'] . '_encodeheader', $string);
+    }
 
     // Encode only if the string contains 8-bit characters or =?
-    $j = strlen( $string  );
-    $l = strstr($string, '=?');         // Must be encoded ?
+    $j = strlen($string);
+    $max_l = 75 - strlen($default_charset) - 7;
+    $aRet = array();
     $ret = '';
-    for( $i=0; $i < $j; ++$i) {
-        switch( $string{$i} ) {
-           case '=':
-          $ret .= '=3D';
-          break;
+    $iEncStart = $enc_init = false;
+    $cur_l = $iOffset = 0;
+    for($i = 0; $i < $j; ++$i) {
+        switch($string{$i})
+        {
+        case '=':
+        case '<':
+        case '>':
+        case ',':
         case '?':
-          $ret .= '=3F';
-          break;
         case '_':
-          $ret .= '=5F';
-          break;
+            if ($iEncStart === false) {
+                $iEncStart = $i;
+            }
+            $cur_l+=3;
+            if ($cur_l > ($max_l-2)) {
+                /* if there is an stringpart that doesn't need encoding, add it */
+                $aRet[] = substr($string,$iOffset,$iEncStart-$iOffset);
+                $aRet[] = "=?$default_charset?Q?$ret?=";
+                $iOffset = $i;
+                $cur_l = 0;
+                $ret = '';
+                $iEncStart = false;
+            } else {
+                $ret .= sprintf("=%02X",ord($string{$i}));
+            }
+            break;
+        case '(':
+        case ')':
+            if ($iEncStart !== false) {
+                $aRet[] = substr($string,$iOffset,$iEncStart-$iOffset);
+                $aRet[] = "=?$default_charset?Q?$ret?=";
+                $iOffset = $i;
+                $cur_l = 0;
+                $ret = '';
+                $iEncStart = false;
+            }
+            break;
         case ' ':
-          $ret .= '_';
-          break;
+            if ($iEncStart !== false) {
+                $cur_l++;
+                if ($cur_l > $max_l) {
+                    $aRet[] = substr($string,$iOffset,$iEncStart-$iOffset);
+                    $aRet[] = "=?$default_charset?Q?$ret?=";
+                    $iOffset = $i;
+                    $cur_l = 0;
+                    $ret = '';
+                    $iEncStart = false;
+                } else {
+                    $ret .= '_';
+                }
+            }
+            break;
         default:
-          $k = ord( $string{$i} );
-          if ( $k > 126 ) {
-             $ret .= sprintf("=%02X", $k);
-             $l = TRUE;
-          } else
-             $ret .= $string{$i};
+            $k = ord($string{$i});
+            if ($k > 126) {
+                if ($iEncStart === false) {
+                    // do not start encoding in the middle of a string, also take the rest of the word.
+                    $sLeadString = substr($string,0,$i);
+                    $aLeadString = explode(' ',$sLeadString);
+                    $sToBeEncoded = array_pop($aLeadString);
+                    $iEncStart = $i - strlen($sToBeEncoded);
+                    $ret .= $sToBeEncoded;
+                    $cur_l += strlen($sToBeEncoded);
+                }
+                $cur_l += 3;
+                /* first we add the encoded string that reached it's max size */
+                if ($cur_l > ($max_l-2)) {
+                    $aRet[] = substr($string,$iOffset,$iEncStart-$iOffset);
+                    $aRet[] = "=?$default_charset?Q?$ret?= "; /* the next part is also encoded => separate by space */
+                    $cur_l = 3;
+                    $ret = '';
+                    $iOffset = $i;
+                    $iEncStart = $i;
+                }
+                $enc_init = true;
+                $ret .= sprintf("=%02X", $k);
+            } else {
+                if ($iEncStart !== false) {
+                    $cur_l++;
+                    if ($cur_l > $max_l) {
+                        $aRet[] = substr($string,$iOffset,$iEncStart-$iOffset);
+                        $aRet[] = "=?$default_charset?Q?$ret?=";
+                        $iEncStart = false;
+                        $iOffset = $i;
+                        $cur_l = 0;
+                        $ret = '';
+                    } else {
+                        $ret .= $string{$i};
+                    }
+                }
+            }
+            break;
         }
     }
 
-    if ( $l ) {
-        $string = "=?$default_charset?Q?$ret?=";
+    if ($enc_init) {
+        if ($iEncStart !== false) {
+            $aRet[] = substr($string,$iOffset,$iEncStart-$iOffset);
+            $aRet[] = "=?$default_charset?Q?$ret?=";
+        } else {
+            $aRet[] = substr($string,$iOffset);
+        }
+        $string = implode('',$aRet);
     }
-
-    return( $string );
+    return $string;
 }
 
 /* This function trys to locate the entity_id of a specific mime element */
-
-function find_ent_id( $id, $message ) {
-    $ret = '';
-    for ($i=0; $ret == '' && $i < count($message->entities); $i++) {
-       if (( $message->entities[$i]->header->type1 == 'alternative') ||         
-           ( $message->entities[$i]->header->type1 == 'related') ||     
-           ( $message->entities[$i]->header->type1 == 'mixed')) {       
-           $ret = find_ent_id( $id, $message->entities[$i] );
+function find_ent_id($id, $message) {
+    for ($i = 0, $ret = ''; $ret == '' && $i < count($message->entities); $i++) {
+        if ($message->entities[$i]->header->type0 == 'multipart')  {
+            $ret = find_ent_id($id, $message->entities[$i]);
         } else {
-            if ( strcasecmp( $message->entities[$i]->header->id, $id ) == 0 )
-                $ret = $message->entities[$i]->header->entity_id;
+            if (strcasecmp($message->entities[$i]->header->id, $id) == 0) {
+//                if (sq_check_save_extension($message->entities[$i])) {
+                    return $message->entities[$i]->entity_id;
+//                }
+            } elseif (!empty($message->entities[$i]->header->parameters['name'])) {
+                /**
+                 * This is part of a fix for Outlook Express 6.x generating
+                 * cid URLs without creating content-id headers
+                 * @@JA - 20050207
+                 */
+                if (strcasecmp($message->entities[$i]->header->parameters['name'], $id) == 0) {
+                    return $message->entities[$i]->entity_id;
+                }
+            }
         }
+    }
+    return $ret;
+}
+
+function sq_check_save_extension($message) {
+    $filename = $message->getFilename();
+    $ext = substr($filename, strrpos($filename,'.')+1);
+    $save_extensions = array('jpg','jpeg','gif','png','bmp');
+    return in_array($ext, $save_extensions);
+}
+
+
+/**
+** HTMLFILTER ROUTINES
+*/
 
+/**
+* This function is more or less a wrapper around stripslashes. Apparently
+* Explorer is stupid enough to just remove the backslashes and then
+* execute the content of the attribute as if nothing happened.
+* Who does that?
+*
+* @param  attvalue   The value of the attribute
+* @return attvalue   The value of the attribute stripslashed.
+*/
+function sq_unbackslash($attvalue){
+    /**
+    * Remove any backslashes. See if there are any first.
+    */
+
+    if (strstr($attvalue, '\\') !== false){
+        $attvalue = stripslashes($attvalue);
     }
-    return( $ret );
+    return $attvalue;
 }
 
 /**
- ** HTMLFILTER ROUTINES
- */
+* Kill any tabs, newlines, or carriage returns. Our friends the
+* makers of the browser with 95% market value decided that it'd
+* be funny to make "java[tab]script" be just as good as "javascript".
+*
+* @param  attvalue  The attribute value before extraneous spaces removed.
+* @return attvalue  The attribute value after extraneous spaces removed.
+*/
+function sq_unspace($attvalue){
+    if (strcspn($attvalue, "\t\r\n") != strlen($attvalue)){
+        $attvalue = str_replace(Array("\t", "\r", "\n"), Array('', '', ''),
+                                $attvalue);
+    }
+    return $attvalue;
+}
 
 /**
- * This function returns the final tag out of the tag name, an array
- * of attributes, and the type of the tag. This function is called by 
- * sq_sanitize internally.
- *
- * @param  $tagname  the name of the tag.
- * @param  $attary   the array of attributes and their values
- * @param  $tagtype  The type of the tag (see in comments).
- * @return           a string with the final tag representation.
- */
+* This function returns the final tag out of the tag name, an array
+* of attributes, and the type of the tag. This function is called by
+* sq_sanitize internally.
+*
+* @param  $tagname  the name of the tag.
+* @param  $attary   the array of attributes and their values
+* @param  $tagtype  The type of the tag (see in comments).
+* @return           a string with the final tag representation.
+*/
 function sq_tagprint($tagname, $attary, $tagtype){
-    $me = "sq_tagprint";
+    $me = 'sq_tagprint';
+
     if ($tagtype == 2){
         $fulltag = '</' . $tagname . '>';
     } else {
@@ -1209,37 +981,37 @@ function sq_tagprint($tagname, $attary, $tagtype){
             $fulltag .= ' ' . join(" ", $atts);
         }
         if ($tagtype == 3){
-            $fulltag .= " /";
+            $fulltag .= ' /';
         }
-        $fulltag .= ">";
+        $fulltag .= '>';
     }
     return $fulltag;
 }
 
 /**
- * A small helper function to use with array_walk. Modifies a by-ref
- * value and makes it lowercase.
- *
- * @param  $val a value passed by-ref.
- * @return      void since it modifies a by-ref value.
- */
+* A small helper function to use with array_walk. Modifies a by-ref
+* value and makes it lowercase.
+*
+* @param  $val a value passed by-ref.
+* @return      void since it modifies a by-ref value.
+*/
 function sq_casenormalize(&$val){
     $val = strtolower($val);
 }
 
 /**
- * This function skips any whitespace from the current position within
- * a string and to the next non-whitespace value.
- * 
- * @param  $body   the string
- * @param  $offset the offset within the string where we should start
- *                 looking for the next non-whitespace character.
- * @return         the location within the $body where the next
- *                 non-whitespace char is located.
- */
+* This function skips any whitespace from the current position within
+* a string and to the next non-whitespace value.
+*
+* @param  $body   the string
+* @param  $offset the offset within the string where we should start
+*                 looking for the next non-whitespace character.
+* @return         the location within the $body where the next
+*                 non-whitespace char is located.
+*/
 function sq_skipspace($body, $offset){
-    $me = "sq_skipspace";
-    preg_match("/^(\s*)/s", substr($body, $offset), $matches);
+    $me = 'sq_skipspace';
+    preg_match('/^(\s*)/s', substr($body, $offset), $matches);
     if (sizeof($matches{1})){
         $count = strlen($matches{1});
         $offset += $count;
@@ -1248,18 +1020,18 @@ function sq_skipspace($body, $offset){
 }
 
 /**
- * This function looks for the next character within a string.  It's
- * really just a glorified "strpos", except it catches if failures
- * nicely.
- *
- * @param  $body   The string to look for needle in.
- * @param  $offset Start looking from this position.
- * @param  $needle The character/string to look for.
- * @return         location of the next occurance of the needle, or
- *                 strlen($body) if needle wasn't found.
- */
+* This function looks for the next character within a string.  It's
+* really just a glorified "strpos", except it catches if failures
+* nicely.
+*
+* @param  $body   The string to look for needle in.
+* @param  $offset Start looking from this position.
+* @param  $needle The character/string to look for.
+* @return         location of the next occurance of the needle, or
+*                 strlen($body) if needle wasn't found.
+*/
 function sq_findnxstr($body, $offset, $needle){
-    $me = "sq_findnxstr";
+    $me  = 'sq_findnxstr';
     $pos = strpos($body, $needle, $offset);
     if ($pos === FALSE){
         $pos = strlen($body);
@@ -1268,24 +1040,24 @@ function sq_findnxstr($body, $offset, $needle){
 }
 
 /**
- * This function takes a PCRE-style regexp and tries to match it
- * within the string.
- *
- * @param  $body   The string to look for needle in.
- * @param  $offset Start looking from here.
- * @param  $reg    A PCRE-style regex to match.
- * @return         Returns a false if no matches found, or an array
- *                 with the following members:
- *                 - integer with the location of the match within $body
- *                 - string with whatever content between offset and the match
- *                 - string with whatever it is we matched
- */
+* This function takes a PCRE-style regexp and tries to match it
+* within the string.
+*
+* @param  $body   The string to look for needle in.
+* @param  $offset Start looking from here.
+* @param  $reg    A PCRE-style regex to match.
+* @return         Returns a false if no matches found, or an array
+*                 with the following members:
+*                 - integer with the location of the match within $body
+*                 - string with whatever content between offset and the match
+*                 - string with whatever it is we matched
+*/
 function sq_findnxreg($body, $offset, $reg){
-    $me = "sq_findnxreg";
+    $me = 'sq_findnxreg';
     $matches = Array();
     $retarr = Array();
-    preg_match("%^(.*?)($reg)%s", substr($body, $offset), $matches);
-    if (!$matches{0}){
+    preg_match("%^(.*?)($reg)%si", substr($body, $offset), $matches);
+    if (!isset($matches{0}) || !$matches{0}){
         $retarr = false;
     } else {
         $retarr{0} = $offset + strlen($matches{1});
@@ -1296,21 +1068,21 @@ function sq_findnxreg($body, $offset, $reg){
 }
 
 /**
- * This function looks for the next tag.
- *
- * @param  $body   String where to look for the next tag.
- * @param  $offset Start looking from here.
- * @return         false if no more tags exist in the body, or
- *                 an array with the following members:
- *                 - string with the name of the tag
- *                 - array with attributes and their values
- *                 - integer with tag type (1, 2, or 3)
- *                 - integer where the tag starts (starting "<")
- *                 - integer where the tag ends (ending ">")
- *                 first three members will be false, if the tag is invalid.
- */
+* This function looks for the next tag.
+*
+* @param  $body   String where to look for the next tag.
+* @param  $offset Start looking from here.
+* @return         false if no more tags exist in the body, or
+*                 an array with the following members:
+*                 - string with the name of the tag
+*                 - array with attributes and their values
+*                 - integer with tag type (1, 2, or 3)
+*                 - integer where the tag starts (starting "<")
+*                 - integer where the tag ends (ending ">")
+*                 first three members will be false, if the tag is invalid.
+*/
 function sq_getnxtag($body, $offset){
-    $me = "sq_getnxtag";
+    $me = 'sq_getnxtag';
     if ($offset > strlen($body)){
         return false;
     }
@@ -1319,60 +1091,59 @@ function sq_getnxtag($body, $offset){
         return false;
     }
     /**
-     * We are here:
-     * blah blah <tag attribute="value">
-     * \---------^
-     */
+    * We are here:
+    * blah blah <tag attribute="value">
+    * \---------^
+    */
     $pos = sq_skipspace($body, $lt+1);
     if ($pos >= strlen($body)){
         return Array(false, false, false, $lt, strlen($body));
     }
     /**
-     * There are 3 kinds of tags:
-     * 1. Opening tag, e.g.:
-     *    <a href="blah">
-     * 2. Closing tag, e.g.:
-     *    </a>
-     * 3. XHTML-style content-less tag, e.g.:
-     *    <img src="blah"/>
-     */
+    * There are 3 kinds of tags:
+    * 1. Opening tag, e.g.:
+    *    <a href="blah">
+    * 2. Closing tag, e.g.:
+    *    </a>
+    * 3. XHTML-style content-less tag, e.g.:
+    *    <img src="blah" />
+    */
     $tagtype = false;
     switch (substr($body, $pos, 1)){
-    case "/":
-        $tagtype = 2;
-        $pos++;
-        break;
-    case "!":
-        /**
-         * A comment or an SGML declaration.
-         */
-        if (substr($body, $pos+1, 2) == "--"){
-            $gt = strpos($body, "-->", $pos);
-            if ($gt === false){
-                $gt = strlen($body);
+        case '/':
+            $tagtype = 2;
+            $pos++;
+            break;
+        case '!':
+            /**
+            * A comment or an SGML declaration.
+            */
+            if (substr($body, $pos+1, 2) == "--"){
+                $gt = strpos($body, "-->", $pos);
+                if ($gt === false){
+                    $gt = strlen($body);
+                } else {
+                    $gt += 2;
+                }
+                return Array(false, false, false, $lt, $gt);
             } else {
-               $gt += 2;
-           }
-            return Array(false, false, false, $lt, $gt);
-        } else {
-            $gt = sq_findnxstr($body, $pos, ">");
-            return Array(false, false, false, $lt, $gt);
-        }
-        break;
-    default:
-        /**
-         * Assume tagtype 1 for now. If it's type 3, we'll switch values
-         * later.
-         */
-        $tagtype = 1;
-        break;
+                $gt = sq_findnxstr($body, $pos, ">");
+                return Array(false, false, false, $lt, $gt);
+            }
+            break;
+        default:
+            /**
+            * Assume tagtype 1 for now. If it's type 3, we'll switch values
+            * later.
+            */
+            $tagtype = 1;
+            break;
     }
 
-    $tag_start = $pos;
     $tagname = '';
     /**
-     * Look for next [\W-_], which will indicate the end of the tag name.
-     */
+    * Look for next [\W-_], which will indicate the end of the tag name.
+    */
     $regary = sq_findnxreg($body, $pos, "[^\w\-_]");
     if ($regary == false){
         return Array(false, false, false, $lt, strlen($body));
@@ -1381,73 +1152,72 @@ function sq_getnxtag($body, $offset){
     $tagname = strtolower($tagname);
 
     /**
-     * $match can be either of these:
-     * '>'  indicating the end of the tag entirely.
-     * '\s' indicating the end of the tag name.
-     * '/'  indicating that this is type-3 xhtml tag.
-     * 
-     * Whatever else we find there indicates an invalid tag.
-     */
+    * $match can be either of these:
+    * '>'  indicating the end of the tag entirely.
+    * '\s' indicating the end of the tag name.
+    * '/'  indicating that this is type-3 xhtml tag.
+    *
+    * Whatever else we find there indicates an invalid tag.
+    */
     switch ($match){
-    case "/":
-        /**
-         * This is an xhtml-style tag with a closing / at the
-         * end, like so: <img src="blah"/>. Check if it's followed
-         * by the closing bracket. If not, then this tag is invalid
-         */
-        if (substr($body, $pos, 2) == "/>"){
-            $pos++;
-            $tagtype = 3;
-        } else {
-            $gt = sq_findnxstr($body, $pos, ">");
-            $retary = Array(false, false, false, $lt, $gt);
-            return $retary;
-        }
-    case ">":
-        return Array($tagname, false, $tagtype, $lt, $pos);
-        break;
-    default:
-        /**
-         * Check if it's whitespace
-         */
-        if (preg_match("/\s/", $match)){
-        } else {
+        case '/':
             /**
-             * This is an invalid tag! Look for the next closing ">".
-             */
-            $gt = sq_findnxstr($body, $offset, ">");
-            return Array(false, false, false, $lt, $gt);
-        }
+            * This is an xhtml-style tag with a closing / at the
+            * end, like so: <img src="blah" />. Check if it's followed
+            * by the closing bracket. If not, then this tag is invalid
+            */
+            if (substr($body, $pos, 2) == "/>"){
+                $pos++;
+                $tagtype = 3;
+            } else {
+                $gt = sq_findnxstr($body, $pos, ">");
+                $retary = Array(false, false, false, $lt, $gt);
+                return $retary;
+            }
+        case '>':
+            return Array($tagname, false, $tagtype, $lt, $pos);
+            break;
+        default:
+            /**
+            * Check if it's whitespace
+            */
+            if (!preg_match('/\s/', $match)){
+                /**
+                * This is an invalid tag! Look for the next closing ">".
+                */
+                $gt = sq_findnxstr($body, $lt, ">");
+                return Array(false, false, false, $lt, $gt);
+            }
+            break;
     }
-    
+
     /**
-     * At this point we're here:
-     * <tagname  attribute='blah'>
-     * \-------^
-     *
-     * At this point we loop in order to find all attributes.
-     */
+    * At this point we're here:
+    * <tagname  attribute='blah'>
+    * \-------^
+    *
+    * At this point we loop in order to find all attributes.
+    */
     $attname = '';
-    $atttype = false;
     $attary = Array();
 
     while ($pos <= strlen($body)){
         $pos = sq_skipspace($body, $pos);
         if ($pos == strlen($body)){
             /**
-             * Non-closed tag.
-             */
+            * Non-closed tag.
+            */
             return Array(false, false, false, $lt, $pos);
         }
         /**
-         * See if we arrived at a ">" or "/>", which means that we reached
-         * the end of the tag.
-         */
+        * See if we arrived at a ">" or "/>", which means that we reached
+        * the end of the tag.
+        */
         $matches = Array();
         if (preg_match("%^(\s*)(>|/>)%s", substr($body, $pos), $matches)) {
             /**
-             * Yep. So we did.
-             */
+            * Yep. So we did.
+            */
             $pos += strlen($matches{1});
             if ($matches{2} == "/>"){
                 $tagtype = 3;
@@ -1457,175 +1227,177 @@ function sq_getnxtag($body, $offset){
         }
 
         /**
-         * There are several types of attributes, with optional
-         * [:space:] between members.
-         * Type 1:
-         *   attrname[:space:]=[:space:]'CDATA'
-         * Type 2:
-         *   attrname[:space:]=[:space:]"CDATA"
-         * Type 3:
-         *   attr[:space:]=[:space:]CDATA
-         * Type 4:
-         *   attrname
-         *
-         * We leave types 1 and 2 the same, type 3 we check for
-         * '"' and convert to "&quot" if needed, then wrap in
-         * double quotes. Type 4 we convert into:
-         * attrname="yes".
-         */
-        $regary = sq_findnxreg($body, $pos, "[^\w\-_]");
+        * There are several types of attributes, with optional
+        * [:space:] between members.
+        * Type 1:
+        *   attrname[:space:]=[:space:]'CDATA'
+        * Type 2:
+        *   attrname[:space:]=[:space:]"CDATA"
+        * Type 3:
+        *   attr[:space:]=[:space:]CDATA
+        * Type 4:
+        *   attrname
+        *
+        * We leave types 1 and 2 the same, type 3 we check for
+        * '"' and convert to "&quot" if needed, then wrap in
+        * double quotes. Type 4 we convert into:
+        * attrname="yes".
+        */
+        $regary = sq_findnxreg($body, $pos, "[^:\w\-_]");
         if ($regary == false){
             /**
-             * Looks like body ended before the end of tag.
-             */
+            * Looks like body ended before the end of tag.
+            */
             return Array(false, false, false, $lt, strlen($body));
         }
         list($pos, $attname, $match) = $regary;
         $attname = strtolower($attname);
         /**
-         * We arrived at the end of attribute name. Several things possible
-         * here:
-         * '>'  means the end of the tag and this is attribute type 4
-         * '/'  if followed by '>' means the same thing as above
-         * '\s' means a lot of things -- look what it's followed by.
-         *      anything else means the attribute is invalid.
-         */
+        * We arrived at the end of attribute name. Several things possible
+        * here:
+        * '>'  means the end of the tag and this is attribute type 4
+        * '/'  if followed by '>' means the same thing as above
+        * '\s' means a lot of things -- look what it's followed by.
+        *      anything else means the attribute is invalid.
+        */
         switch($match){
-        case "/":
-            /**
-             * This is an xhtml-style tag with a closing / at the
-             * end, like so: <img src="blah"/>. Check if it's followed
-             * by the closing bracket. If not, then this tag is invalid
-             */
-            if (substr($body, $pos, 2) == "/>"){
-                $pos++;
-                $tagtype = 3;
-            } else {
-                $gt = sq_findnxstr($body, $pos, ">");
-                $retary = Array(false, false, false, $lt, $gt);
-                return $retary;
-            }
-        case ">":
-            $attary{$attname} = '"yes"';
-            return Array($tagname, $attary, $tagtype, $lt, $pos);
-            break;
-        default:
-            /**
-             * Skip whitespace and see what we arrive at.
-             */
-            $pos = sq_skipspace($body, $pos);
-            $char = substr($body, $pos, 1);
-            /**
-             * Two things are valid here:
-             * '=' means this is attribute type 1 2 or 3.
-             * \w means this was attribute type 4.
-             * anything else we ignore and re-loop. End of tag and
-             * invalid stuff will be caught by our checks at the beginning
-             * of the loop.
-             */
-            if ($char == "="){
-                $pos++;
-                $pos = sq_skipspace($body, $pos);
+            case '/':
                 /**
-                 * Here are 3 possibilities:
-                 * "'"  attribute type 1
-                 * '"'  attribute type 2
-                 * everything else is the content of tag type 3
-                 */
-                $quot = substr($body, $pos, 1);
-                if ($quot == "'"){
-                    $regary = sq_findnxreg($body, $pos+1, "\'");
-                    if ($regary == false){
-                        return Array(false, false, false, $lt, strlen($body));
-                    }
-                    list($pos, $attval, $match) = $regary;
-                    $pos++;
-                    $attary{$attname} = "'" . $attval . "'";
-                } else if ($quot == '"'){
-                    $regary = sq_findnxreg($body, $pos+1, '\"');
-                    if ($regary == false){
-                        return Array(false, false, false, $lt, strlen($body));
-                    }
-                    list($pos, $attval, $match) = $regary;
+                * This is an xhtml-style tag with a closing / at the
+                * end, like so: <img src="blah" />. Check if it's followed
+                * by the closing bracket. If not, then this tag is invalid
+                */
+                if (substr($body, $pos, 2) == "/>"){
                     $pos++;
-                    $attary{$attname} = '"' . $attval . '"';
+                    $tagtype = 3;
                 } else {
+                    $gt = sq_findnxstr($body, $pos, ">");
+                    $retary = Array(false, false, false, $lt, $gt);
+                    return $retary;
+                }
+            case '>':
+                $attary{$attname} = '"yes"';
+                return Array($tagname, $attary, $tagtype, $lt, $pos);
+                break;
+            default:
+                /**
+                * Skip whitespace and see what we arrive at.
+                */
+                $pos = sq_skipspace($body, $pos);
+                $char = substr($body, $pos, 1);
+                /**
+                * Two things are valid here:
+                * '=' means this is attribute type 1 2 or 3.
+                * \w means this was attribute type 4.
+                * anything else we ignore and re-loop. End of tag and
+                * invalid stuff will be caught by our checks at the beginning
+                * of the loop.
+                */
+                if ($char == "="){
+                    $pos++;
+                    $pos = sq_skipspace($body, $pos);
                     /**
-                     * These are hateful. Look for \s, or >.
-                     */
-                    $regary = sq_findnxreg($body, $pos, "[\s>]");
-                    if ($regary == false){
-                        return Array(false, false, false, $lt, strlen($body));
+                    * Here are 3 possibilities:
+                    * "'"  attribute type 1
+                    * '"'  attribute type 2
+                    * everything else is the content of tag type 3
+                    */
+                    $quot = substr($body, $pos, 1);
+                    if ($quot == "'"){
+                        $regary = sq_findnxreg($body, $pos+1, "\'");
+                        if ($regary == false){
+                            return Array(false, false, false, $lt, strlen($body));
+                        }
+                        list($pos, $attval, $match) = $regary;
+                        $pos++;
+                        $attary{$attname} = "'" . $attval . "'";
+                    } else if ($quot == '"'){
+                        $regary = sq_findnxreg($body, $pos+1, '\"');
+                        if ($regary == false){
+                            return Array(false, false, false, $lt, strlen($body));
+                        }
+                        list($pos, $attval, $match) = $regary;
+                        $pos++;
+                        $attary{$attname} = '"' . $attval . '"';
+                    } else {
+                        /**
+                        * These are hateful. Look for \s, or >.
+                        */
+                        $regary = sq_findnxreg($body, $pos, "[\s>]");
+                        if ($regary == false){
+                            return Array(false, false, false, $lt, strlen($body));
+                        }
+                        list($pos, $attval, $match) = $regary;
+                        /**
+                        * If it's ">" it will be caught at the top.
+                        */
+                        $attval = preg_replace("/\"/s", "&quot;", $attval);
+                        $attary{$attname} = '"' . $attval . '"';
                     }
-                    list($pos, $attval, $match) = $regary;
+                } else if (preg_match("|[\w/>]|", $char)) {
+                    /**
+                    * That was attribute type 4.
+                    */
+                    $attary{$attname} = '"yes"';
+                } else {
                     /**
-                     * If it's ">" it will be caught at the top.
-                     */
-                    $attval = preg_replace("/\"/s", "&quot;", $attval);
-                    $attary{$attname} = '"' . $attval . '"';
+                    * An illegal character. Find next '>' and return.
+                    */
+                    $gt = sq_findnxstr($body, $pos, ">");
+                    return Array(false, false, false, $lt, $gt);
                 }
-            } else if (preg_match("|[\w/>]|", $char)) {
-                /**
-                 * That was attribute type 4.
-                 */
-                $attary{$attname} = '"yes"';
-            } else {
-                /**
-                 * An illegal character. Find next '>' and return.
-                 */
-                $gt = sq_findnxstr($body, $pos, ">");
-                return Array(false, false, false, $lt, $gt);
-            }
+                break;
         }
     }
     /**
-     * The fact that we got here indicates that the tag end was never
-     * found. Return invalid tag indication so it gets stripped.
-     */
+    * The fact that we got here indicates that the tag end was never
+    * found. Return invalid tag indication so it gets stripped.
+    */
     return Array(false, false, false, $lt, strlen($body));
 }
 
 /**
- * This function checks attribute values for entity-encoded values
- * and returns them translated into 8-bit strings so we can run
- * checks on them.
- *
- * @param  $attvalue A string to run entity check against.
- * @return           Translated value.
- */
+* This function checks attribute values for entity-encoded values
+* and returns them translated into 8-bit strings so we can run
+* checks on them.
+*
+* @param  $attvalue A string to run entity check against.
+* @return           Translated value.
+*/
+
 function sq_deent($attvalue){
-    $me="sq_deent";
+    $me = 'sq_deent';
     /**
-     * See if we have to run the checks first. All entities must start
-     * with "&".
-     */
-    if (strpos($attvalue, "&") === false){
+    * See if we have to run the checks first. All entities must start
+    * with "&".
+    */
+    if (strpos($attvalue, '&') === false){
         return $attvalue;
     }
     /**
-     * Check named entities first.
-     */
+    * Check named entities first.
+    */
     $trans = get_html_translation_table(HTML_ENTITIES);
     /**
-     * Leave &quot; in, as it can mess us up.
-     */
+    * Leave &quot; in, as it can mess us up.
+    */
     $trans = array_flip($trans);
-    unset($trans{"&quot;"});
+    unset($trans{'&quot;'});
     while (list($ent, $val) = each($trans)){
-        $attvalue = preg_replace("/$ent*(\W)/si", "$val\\1", $attvalue);
+        $attvalue = preg_replace('/' . $ent . '*/si', $val, $attvalue);
     }
     /**
-     * Now translate numbered entities from 1 to 255 if needed.
-     */
-    if (strpos($attvalue, "#") !== false){
+    * Now translate numbered entities from 1 to 255 if needed.
+    */
+    if (strpos($attvalue, '#') !== false){
         $omit = Array(34, 39);
-        for ($asc=1; $asc<256; $asc++){
+        for ($asc = 256; $asc >= 0; $asc--){
             if (!in_array($asc, $omit)){
                 $chr = chr($asc);
-                $attvalue = preg_replace("/\&#0*$asc;*(\D)/si", "$chr\\1", 
-                                         $attvalue);
-                $attvalue = preg_replace("/\&#x0*".dechex($asc).";*(\W)/si",
-                                         "$chr\\1", $attvalue);
+                $octrule = '/\&#0*' . $asc . ';*/si';
+                $hexrule = '/\&#x0*' . dechex($asc) . ';*/si';
+                $attvalue = preg_replace($octrule, $chr, $attvalue);
+                $attvalue = preg_replace($hexrule, $chr, $attvalue);
             }
         }
     }
@@ -1633,30 +1405,31 @@ function sq_deent($attvalue){
 }
 
 /**
- * This function runs various checks against the attributes.
- *
- * @param  $tagname         String with the name of the tag.
- * @param  $attary          Array with all tag attributes.
- * @param  $rm_attnames     See description for sq_sanitize
- * @param  $bad_attvals     See description for sq_sanitize
- * @param  $add_attr_to_tag See description for sq_sanitize
- * @param  $message         message object
- * @param  $id              message id
- * @return                  Array with modified attributes.
- */
-function sq_fixatts($tagname, 
-                    $attary, 
+* This function runs various checks against the attributes.
+*
+* @param  $tagname         String with the name of the tag.
+* @param  $attary          Array with all tag attributes.
+* @param  $rm_attnames     See description for sq_sanitize
+* @param  $bad_attvals     See description for sq_sanitize
+* @param  $add_attr_to_tag See description for sq_sanitize
+* @param  $message         message object
+* @param  $id              message id
+* @return                  Array with modified attributes.
+*/
+function sq_fixatts($tagname,
+                    $attary,
                     $rm_attnames,
                     $bad_attvals,
                     $add_attr_to_tag,
                     $message,
-                    $id
+                    $id,
+                    $mailbox
                     ){
-    $me = "sq_fixatts";
+    $me = 'sq_fixatts';
     while (list($attname, $attvalue) = each($attary)){
         /**
-         * See if this attribute should be removed.
-         */
+        * See if this attribute should be removed.
+        */
         foreach ($rm_attnames as $matchtag=>$matchattrs){
             if (preg_match($matchtag, $tagname)){
                 foreach ($matchattrs as $matchattr){
@@ -1668,27 +1441,35 @@ function sq_fixatts($tagname,
             }
         }
         /**
-         * Remove any entities.
-         */
+        * Remove any backslashes, entities, and extraneous whitespace.
+        */
+        $attvalue = sq_unbackslash($attvalue);
         $attvalue = sq_deent($attvalue);
+        $attvalue = sq_unspace($attvalue);
 
         /**
-         * Now let's run checks on the attvalues.
-         * I don't expect anyone to comprehend this. If you do,
-         * get in touch with me so I can drive to where you live and
-         * shake your hand personally. :)
-         */
+        * Remove \r \n \t \0 " " "\\"
+        */
+        $attvalue = str_replace(Array("\r", "\n", "\t", "\0", " ", "\\"),
+                        Array('', '','','','',''), $attvalue);
+
+        /**
+        * Now let's run checks on the attvalues.
+        * I don't expect anyone to comprehend this. If you do,
+        * get in touch with me so I can drive to where you live and
+        * shake your hand personally. :)
+        */
         foreach ($bad_attvals as $matchtag=>$matchattrs){
             if (preg_match($matchtag, $tagname)){
                 foreach ($matchattrs as $matchattr=>$valary){
                     if (preg_match($matchattr, $attname)){
                         /**
-                         * There are two arrays in valary.
-                         * First is matches.
-                         * Second one is replacements
-                         */
+                        * There are two arrays in valary.
+                        * First is matches.
+                        * Second one is replacements
+                        */
                         list($valmatch, $valrepl) = $valary;
-                        $newvalue = 
+                        $newvalue =
                             preg_replace($valmatch, $valrepl, $attvalue);
                         if ($newvalue != $attvalue){
                             $attary{$attname} = $newvalue;
@@ -1697,16 +1478,38 @@ function sq_fixatts($tagname,
                 }
             }
         }
+
+
         /**
-         * Turn cid: urls into http-friendly ones.
+         * Replace empty src tags with the blank image.  src is only used
+         * for frames, images, and image inputs.  Doing a replace should
+         * not affect them working as should be, however it will stop
+         * IE from being kicked off when src for img tags are not set
          */
+        if (($attname == 'src') && ($attvalue == '""')) {
+            $attary{$attname} = '"' . SM_PATH . 'images/blank.png"';
+        }
+
+        /**
+        * Turn cid: urls into http-friendly ones.
+        */
         if (preg_match("/^[\'\"]\s*cid:/si", $attvalue)){
-            $attary{$attname} = sq_cid2http($message, $id, $attvalue);
+            $attary{$attname} = sq_cid2http($message, $id, $attvalue, $mailbox);
+        }
+
+        /**
+         * "Hack" fix for Outlook using propriatary outbind:// protocol in img tags.
+         * One day MS might actually make it match something useful, for now, falling
+         * back to using cid2http, so we can grab the blank.png.
+         */
+        if (preg_match("/^[\'\"]\s*outbind:\/\//si", $attvalue)) {
+            $attary{$attname} = sq_cid2http($message, $id, $attvalue, $mailbox);
         }
+
     }
     /**
-     * See if we need to append any attributes to this tag.
-     */
+    * See if we need to append any attributes to this tag.
+    */
     foreach ($add_attr_to_tag as $matchtag=>$addattary){
         if (preg_match($matchtag, $tagname)){
             $attary = array_merge($attary, $addattary);
@@ -1716,112 +1519,170 @@ function sq_fixatts($tagname,
 }
 
 /**
- * This function edits the style definition to make them friendly and
- * usable in squirrelmail.
- * 
- * @param  $message  the message object
- * @param  $id       the message id
- * @param  $content  a string with whatever is between <style> and </style>
- * @return           a string with edited content.
- */
-function sq_fixstyle($message, $id, $content){
+* This function edits the style definition to make them friendly and
+* usable in SquirrelMail.
+*
+* @param  $message  the message object
+* @param  $id       the message id
+* @param  $content  a string with whatever is between <style> and </style>
+* @param  $mailbox  the message mailbox
+* @return           a string with edited content.
+*/
+function sq_fixstyle($body, $pos, $message, $id, $mailbox){
     global $view_unsafe_images;
-    $me = "sq_fixstyle";
+    $me = 'sq_fixstyle';
+    $ret = sq_findnxreg($body, $pos, '</\s*style\s*>');
+    if ($ret == FALSE){
+        return array(FALSE, strlen($body));
+    }
+    $newpos = $ret[0] + strlen($ret[2]);
+    $content = $ret[1];
     /**
-     * First look for general BODY style declaration, which would be
-     * like so:
-     * body {background: blah-blah}
-     * and change it to .bodyclass so we can just assign it to a <div>
-     */
+    * First look for general BODY style declaration, which would be
+    * like so:
+    * body {background: blah-blah}
+    * and change it to .bodyclass so we can just assign it to a <div>
+    */
     $content = preg_replace("|body(\s*\{.*?\})|si", ".bodyclass\\1", $content);
-    $secremoveimg = "../images/" . _("sec_remove_eng.png");
+    $secremoveimg = '../images/' . _("sec_remove_eng.png");
     /**
-     * Fix url('blah') declarations.
-     */
-    $content = preg_replace("|url\(([\'\"])\s*\S+script\s*:.*?([\'\"])\)|si",
+    * Fix url('blah') declarations.
+    */
+    $content = preg_replace("|url\s*\(\s*([\'\"])\s*\S+script\s*:.*?([\'\"])\s*\)|si",
                             "url(\\1$secremoveimg\\2)", $content);
     /**
-     * Fix url('https*://.*) declarations but only if $view_unsafe_images
-     * is false.
-     */
+    * Fix url('https*://.*) declarations but only if $view_unsafe_images
+    * is false.
+    */
     if (!$view_unsafe_images){
-        $content = preg_replace("|url\(([\'\"])\s*https*:.*?([\'\"])\)|si",
+        $content = preg_replace("|url\s*\(\s*([\'\"])\s*https*:.*?([\'\"])\s*\)|si",
                                 "url(\\1$secremoveimg\\2)", $content);
     }
-    
+
     /**
-     * Fix urls that refer to cid:
-     */
-    while (preg_match("|url\(([\'\"]\s*cid:.*?[\'\"])\)|si", $content, 
-                      $matches)){
+    * Fix urls that refer to cid:
+    */
+    while (preg_match("|url\s*\(\s*([\'\"]\s*cid:.*?[\'\"])\s*\)|si",
+                    $content, $matches)){
         $cidurl = $matches{1};
-        $httpurl = sq_cid2http($message, $id, $cidurl);
-        $content = preg_replace("|url\($cidurl\)|si",
+        $httpurl = sq_cid2http($message, $id, $cidurl, $mailbox);
+        $content = preg_replace("|url\s*\(\s*$cidurl\s*\)|si",
                                 "url($httpurl)", $content);
     }
 
     /**
-     * Fix stupid css declarations which lead to vulnerabilities
-     * in IE.
-     */
-    $match   = Array('/expression/si',
-                    '/behaviou*r/si',
-                    '/binding/si');
-    $replace = Array('idiocy', 'idiocy', 'idiocy');
+    * Fix stupid css declarations which lead to vulnerabilities
+    * in IE.
+    */
+    $match   = Array('/expression/i',
+                    '/behaviou*r/i',
+                    '/binding/i',
+                    '/include-source/i');
+    $replace = Array('idiocy', 'idiocy', 'idiocy', 'idiocy');
     $content = preg_replace($match, $replace, $content);
-    return $content;
+    return array($content, $newpos);
 }
 
 /**
- * This function converts cid: url's into the ones that can be viewed in
- * the browser.
- *
- * @param  $message  the message object
- * @param  $id       the message id
- * @param  $cidurl   the cid: url.
- * @return           a string with a http-friendly url
- */
-function sq_cid2http($message, $id, $cidurl){
+* This function converts cid: url's into the ones that can be viewed in
+* the browser.
+*
+* @param  $message  the message object
+* @param  $id       the message id
+* @param  $cidurl   the cid: url.
+* @param  $mailbox  the message mailbox
+* @return           a string with a http-friendly url
+*/
+function sq_cid2http($message, $id, $cidurl, $mailbox){
     /**
-     * Get rid of quotes.
-     */
+    * Get rid of quotes.
+    */
     $quotchar = substr($cidurl, 0, 1);
-    $cidurl = str_replace($quotchar, "", $cidurl);
+    if ($quotchar == '"' || $quotchar == "'"){
+        $cidurl = str_replace($quotchar, "", $cidurl);
+    } else {
+        $quotchar = '';
+    }
     $cidurl = substr(trim($cidurl), 4);
-    $httpurl = $quotchar . "../src/download.php?absolute_dl=true&amp;" .
-        "passed_id=$id&amp;mailbox=" . urlencode($message->header->mailbox) .
-        "&amp;passed_ent_id=" . find_ent_id($cidurl, $message) . $quotchar;
+    $linkurl = find_ent_id($cidurl, $message);
+    /* in case of non-save cid links $httpurl should be replaced by a sort of
+    unsave link image */
+    $httpurl = '';
+
+   /**
+    * This is part of a fix for Outlook Express 6.x generating
+    * cid URLs without creating content-id headers. These images are
+    * not part of the multipart/related html mail. The html contains
+    * <img src="cid:{some_id}/image_filename.ext"> references to
+    * attached images with as goal to render them inline although
+    * the attachment disposition property is not inline.
+    **/
+
+    if (empty($linkurl)) {
+        if (preg_match('/{.*}\//', $cidurl)) {
+            $cidurl = preg_replace('/{.*}\//','', $cidurl);
+            if (!empty($cidurl)) {
+                $linkurl = find_ent_id($cidurl, $message);
+            }
+        }
+    }
+    if (!empty($linkurl)) {
+        $httpurl = $quotchar . SM_PATH . 'src/download.php?absolute_dl=true&amp;' .
+                   "passed_id=$id&amp;mailbox=" . urlencode($mailbox) .
+                   '&amp;ent_id=' . $linkurl . $quotchar;
+    } else {
+        /**
+         * If we couldn't generate a proper img url, drop in a blank image
+         * instead of sending back empty, otherwise it causes unusual behaviour
+         */
+        $httpurl = $quotchar . SM_PATH . 'images/blank.png';
+    }
     return $httpurl;
 }
 
 /**
- * This function changes the <body> tag into a <div> tag since we
- * can't really have a body-within-body.
- *
- * @param  $attary  an array of attributes and values of <body>
- * @return          a modified array of attributes to be set for <div>
- */
-function sq_body2div($attary){
-    $me = "sq_body2div";
-    $divattary = Array("class"=>"'bodyclass'");
-    $bgcolor="#ffffff";
-    $text="#000000";
-    $styledef="";
+* This function changes the <body> tag into a <div> tag since we
+* can't really have a body-within-body.
+*
+* @param  $attary   an array of attributes and values of <body>
+* @param  $mailbox  mailbox we're currently reading (for cid2http)
+* @param  $message  current message (for cid2http)
+* @param  $id       current message id (for cid2http)
+* @return           a modified array of attributes to be set for <div>
+*/
+function sq_body2div($attary, $mailbox, $message, $id){
+    $me = 'sq_body2div';
+    $divattary = Array('class' => "'bodyclass'");
+    $text = '#000000';
+    $has_bgc_stl = $has_txt_stl = false;
+    $styledef = '';
     if (is_array($attary) && sizeof($attary) > 0){
         foreach ($attary as $attname=>$attvalue){
             $quotchar = substr($attvalue, 0, 1);
             $attvalue = str_replace($quotchar, "", $attvalue);
             switch ($attname){
-            case "background":
-                $styledef .= "background-image: url('$attvalue'); ";
-                break;
-            case "bgcolor":
-                $styledef .= "background-color: $attvalue; ";
-                break;
-            case "text":
-                $styledef .= "color: $attvalue; ";
+                case 'background':
+                    $attvalue = sq_cid2http($message, $id,
+                                            $attvalue, $mailbox);
+                    $styledef .= "background-image: url('$attvalue'); ";
+                    break;
+                case 'bgcolor':
+                    $has_bgc_stl = true;
+                    $styledef .= "background-color: $attvalue; ";
+                    break;
+                case 'text':
+                    $has_txt_stl = true;
+                    $styledef .= "color: $attvalue; ";
+                    break;
             }
         }
+        // Outlook defines a white bgcolor and no text color. This can lead to
+        // white text on a white bg with certain themes.
+        if ($has_bgc_stl && !$has_txt_stl) {
+            $styledef .= "color: $text; ";
+        }
         if (strlen($styledef) > 0){
             $divattary{"style"} = "\"$styledef\"";
         }
@@ -1830,126 +1691,135 @@ function sq_body2div($attary){
 }
 
 /**
- * This is the main function and the one you should actually be calling.
- * There are several variables you should be aware of an which need
- * special description.
- *
- * Since the description is quite lengthy, see it here:
- * http://www.mricon.com/html/phpfilter.html
- *
- * @param $body                 the string with HTML you wish to filter
- * @param $tag_list             see description above
- * @param $rm_tags_with_content see description above
- * @param $self_closing_tags    see description above
- * @param $force_tag_closing    see description above
- * @param $rm_attnames          see description above
- * @param $bad_attvals          see description above
- * @param $add_attr_to_tag      see description above
- * @param $message              message object
- * @param $id                   message id
- * @return                      sanitized html safe to show on your pages.
- */
-function sq_sanitize($body, 
-                     $tag_list, 
-                     $rm_tags_with_content,
-                     $self_closing_tags,
-                     $force_tag_closing,
-                     $rm_attnames,
-                     $bad_attvals,
-                     $add_attr_to_tag,
-                     $message,
-                     $id
-                     ){
-    $me = "sq_sanitize";
+* This is the main function and the one you should actually be calling.
+* There are several variables you should be aware of an which need
+* special description.
+*
+* Since the description is quite lengthy, see it here:
+* http://linux.duke.edu/projects/mini/htmlfilter/
+*
+* @param $body                 the string with HTML you wish to filter
+* @param $tag_list             see description above
+* @param $rm_tags_with_content see description above
+* @param $self_closing_tags    see description above
+* @param $force_tag_closing    see description above
+* @param $rm_attnames          see description above
+* @param $bad_attvals          see description above
+* @param $add_attr_to_tag      see description above
+* @param $message              message object
+* @param $id                   message id
+* @return                      sanitized html safe to show on your pages.
+*/
+function sq_sanitize($body,
+                    $tag_list,
+                    $rm_tags_with_content,
+                    $self_closing_tags,
+                    $force_tag_closing,
+                    $rm_attnames,
+                    $bad_attvals,
+                    $add_attr_to_tag,
+                    $message,
+                    $id,
+                    $mailbox
+                    ){
+    $me = 'sq_sanitize';
+    $rm_tags = array_shift($tag_list);
     /**
-     * Normalize rm_tags and rm_tags_with_content.
-     */
-    @array_walk($rm_tags, 'sq_casenormalize');
+    * Normalize rm_tags and rm_tags_with_content.
+    */
+    @array_walk($tag_list, 'sq_casenormalize');
     @array_walk($rm_tags_with_content, 'sq_casenormalize');
     @array_walk($self_closing_tags, 'sq_casenormalize');
     /**
-     * See if tag_list is of tags to remove or tags to allow.
-     * false  means remove these tags
-     * true   means allow these tags
-     */
-    $rm_tags = array_shift($tag_list);
+    * See if tag_list is of tags to remove or tags to allow.
+    * false  means remove these tags
+    * true   means allow these tags
+    */
     $curpos = 0;
     $open_tags = Array();
-    $trusted = "<!-- begin sanitized html -->\n";
+    $trusted = "\n<!-- begin sanitized html -->\n";
     $skip_content = false;
     /**
-     * Take care of netscape's stupid javascript entities like
-     * &{alert('boo')};
-     */
+    * Take care of netscape's stupid javascript entities like
+    * &{alert('boo')};
+    */
     $body = preg_replace("/&(\{.*?\};)/si", "&amp;\\1", $body);
 
-    while (($curtag=sq_getnxtag($body, $curpos)) != FALSE){
+    while (($curtag = sq_getnxtag($body, $curpos)) != FALSE){
         list($tagname, $attary, $tagtype, $lt, $gt) = $curtag;
         $free_content = substr($body, $curpos, $lt-$curpos);
         /**
-         * Take care of <style>
-         */
-        if ($tagname == "style" && $tagtype == 2){
-            /**
-             * This is a closing </style>. Edit the
-             * content before we apply it.
-             */
-            $free_content = sq_fixstyle($message, $id, $free_content);
+        * Take care of <style>
+        */
+        if ($tagname == "style" && $tagtype == 1){
+            list($free_content, $curpos) =
+                sq_fixstyle($body, $gt+1, $message, $id, $mailbox);
+            if ($free_content != FALSE){
+                $trusted .= sq_tagprint($tagname, $attary, $tagtype);
+                $trusted .= $free_content;
+                $trusted .= sq_tagprint($tagname, false, 2);
+            }
+            continue;
         }
         if ($skip_content == false){
             $trusted .= $free_content;
-        } else {
         }
         if ($tagname != FALSE){
             if ($tagtype == 2){
                 if ($skip_content == $tagname){
                     /**
-                     * Got to the end of tag we needed to remove.
-                     */
+                    * Got to the end of tag we needed to remove.
+                    */
                     $tagname = false;
                     $skip_content = false;
                 } else {
                     if ($skip_content == false){
                         if ($tagname == "body"){
                             $tagname = "div";
+                        }
+                        if (isset($open_tags{$tagname}) &&
+                            $open_tags{$tagname} > 0){
+                            $open_tags{$tagname}--;
                         } else {
-                            if (isset($open_tags{$tagname}) && 
-                                $open_tags{$tagname} > 0){
-                                $open_tags{$tagname}--;
-                            } else {
-                                $tagname = false;
-                            }
+                            $tagname = false;
                         }
-                    } else {
                     }
                 }
             } else {
                 /**
-                 * $rm_tags_with_content
-                 */
+                * $rm_tags_with_content
+                */
                 if ($skip_content == false){
                     /**
-                     * See if this is a self-closing type and change
-                     * tagtype appropriately.
-                     */
+                    * See if this is a self-closing type and change
+                    * tagtype appropriately.
+                    */
                     if ($tagtype == 1
                         && in_array($tagname, $self_closing_tags)){
-                        $tagtype=3;
+                        $tagtype = 3;
                     }
                     /**
-                     * See if we should skip this tag and any content
-                     * inside it.
-                     */
+                    * See if we should skip this tag and any content
+                    * inside it.
+                    */
                     if ($tagtype == 1 &&
                         in_array($tagname, $rm_tags_with_content)){
                         $skip_content = $tagname;
                     } else {
-                        if (($rm_tags == false 
-                             && in_array($tagname, $tag_list)) ||
+                        if (($rm_tags == false
+                            && in_array($tagname, $tag_list)) ||
                             ($rm_tags == true &&
-                             !in_array($tagname, $tag_list))){
+                            !in_array($tagname, $tag_list))){
                             $tagname = false;
                         } else {
+                            /**
+                            * Convert body into div.
+                            */
+                            if ($tagname == "body"){
+                                $tagname = "div";
+                                $attary = sq_body2div($attary, $mailbox,
+                                                    $message, $id);
+                            }
                             if ($tagtype == 1){
                                 if (isset($open_tags{$tagname})){
                                     $open_tags{$tagname}++;
@@ -1958,34 +1828,26 @@ function sq_sanitize($body,
                                 }
                             }
                             /**
-                             * This is where we run other checks.
-                             */
+                            * This is where we run other checks.
+                            */
                             if (is_array($attary) && sizeof($attary) > 0){
                                 $attary = sq_fixatts($tagname,
-                                                     $attary,
-                                                     $rm_attnames,
-                                                     $bad_attvals,
-                                                     $add_attr_to_tag,
-                                                     $message,
-                                                     $id
-                                                     );
-                            }
-                            /**
-                             * Convert body into div.
-                             */
-                            if ($tagname == "body"){
-                                $tagname = "div";
-                                $attary = sq_body2div($attary, $message, $id);
+                                                    $attary,
+                                                    $rm_attnames,
+                                                    $bad_attvals,
+                                                    $add_attr_to_tag,
+                                                    $message,
+                                                    $id,
+                                                    $mailbox
+                                                    );
                             }
                         }
                     }
-                } else {
                 }
             }
             if ($tagname != false && $skip_content == false){
                 $trusted .= sq_tagprint($tagname, $attary, $tagtype);
             }
-        } else {
         }
         $curpos = $gt+1;
     }
@@ -2004,124 +1866,335 @@ function sq_sanitize($body,
 }
 
 /**
- * This is a wrapper function to call html sanitizing routines.
- *
- * @param  $body  the body of the message
- * @param  $id    the id of the message
- * @return        a string with html safe to display in the browser.
- */
-function magicHTML($body, $id, $message){
+* This is a wrapper function to call html sanitizing routines.
+*
+* @param  $body  the body of the message
+* @param  $id    the id of the message
+* @param  $message
+* @param  $mailbox
+* @param  boolean $take_mailto_links When TRUE, converts mailto: links
+*                                    into internal SM compose links
+*                                    (optional; default = TRUE)
+* @return        a string with html safe to display in the browser.
+*/
+function magicHTML($body, $id, $message, $mailbox = 'INBOX', $take_mailto_links = true) {
+
+    require_once(SM_PATH . 'functions/url_parser.php');  // for $MailTo_PReg_Match
+
     global $attachment_common_show_images, $view_unsafe_images,
         $has_unsafe_images;
+
     /**
-     * Don't display attached images in HTML mode.
-     */
+    * Don't display attached images in HTML mode.
+    */
     $attachment_common_show_images = false;
     $tag_list = Array(
-                      false,
-                      "object",
-                      "meta",
-                      "html",
-                      "head",
-                      "base"
-                      );
+                    false,
+                    "object",
+                    "meta",
+                    "html",
+                    "head",
+                    "base",
+                    "link",
+                    "frame",
+                    "iframe",
+                    "plaintext",
+                    "marquee"
+                    );
 
     $rm_tags_with_content = Array(
-                                  "script",
-                                  "applet",
-                                  "embed",
-                                  "title"
-                                  );
+                                "script",
+                                "applet",
+                                "embed",
+                                "title",
+                                "frameset",
+                                "xml"
+                                );
 
     $self_closing_tags =  Array(
                                 "img",
                                 "br",
                                 "hr",
-                                "input"
+                                "input",
+                                "outbind"
                                 );
 
-    $force_tag_closing = false;
+    $force_tag_closing = true;
 
     $rm_attnames = Array(
-                         "/.*/" =>
-                         Array(
-                               "/target/si",
-                               "/^on.*/si",
-                              "/^dynsrc/si",
-                              "/^data.*/si"
-                               )
-                         );
+                        "/.*/" =>
+                        Array(
+                            "/target/i",
+                            "/^on.*/i",
+                            "/^dynsrc/i",
+                            "/^data.*/i",
+                            "/^lowsrc.*/i"
+                            )
+                        );
 
     $secremoveimg = "../images/" . _("sec_remove_eng.png");
     $bad_attvals = Array(
         "/.*/" =>
             Array(
-                "/^src|background|href|action/i" =>
+                "/^src|background/i" =>
                     Array(
-                          Array(
-                                "|^([\'\"])\s*\.\./.*([\'\"])|si",
+                        Array(
                                 "/^([\'\"])\s*\S+script\s*:.*([\'\"])/si",
-                               "/^([\'\"])\s*mocha\s*:*(.*)([\'\"])/si",
-                               "/^([\'\"])\s*about\s*:(.*)([\'\"])/si"
+                                "/^([\'\"])\s*mocha\s*:*.*([\'\"])/si",
+                                "/^([\'\"])\s*about\s*:.*([\'\"])/si"
                                 ),
-                          Array(
+                        Array(
                                 "\\1$secremoveimg\\2",
                                 "\\1$secremoveimg\\2",
-                               "\\1$secremoveimg\\2",
-                               "\\1$secremoveimg\\2"
+                                "\\1$secremoveimg\\2",
+                                "\\1$secremoveimg\\2"
+                                )
+                        ),
+                "/^href|action/i" =>
+                    Array(
+                        Array(
+                                "/^([\'\"])\s*\S+script\s*:.*([\'\"])/si",
+                                "/^([\'\"])\s*mocha\s*:*.*([\'\"])/si",
+                                "/^([\'\"])\s*about\s*:.*([\'\"])/si"
+                                ),
+                        Array(
+                                "\\1#\\1",
+                                "\\1#\\1",
+                                "\\1#\\1",
+                                "\\1#\\1"
                                 )
                         ),
-                "/^style/si" =>
+                "/^style/i" =>
                     Array(
-                          Array(
-                                "/expression/si",
-                               "/binding/si",
-                               "/behaviou*r/si",
-                                "|url\(([\'\"])\s*\.\./.*([\'\"])\)|si",
-                                "/url\(([\'\"])\s*\S+script:.*([\'\"])\)/si"
-                               ),
-                          Array(
+                        Array(
+                                "/expression/i",
+                                "/binding/i",
+                                "/behaviou*r/i",
+                                "/include-source/i",
+                                "/position\s*:\s*absolute/i",
+                                "/url\s*\(\s*([\'\"])\s*\S+script\s*:.*([\'\"])\s*\)/si",
+                                "/url\s*\(\s*([\'\"])\s*mocha\s*:.*([\'\"])\s*\)/si",
+                                "/url\s*\(\s*([\'\"])\s*about\s*:.*([\'\"])\s*\)/si",
+                                "/(.*)\s*:\s*url\s*\(\s*([\'\"]*)\s*\S+script\s*:.*([\'\"]*)\s*\)/si"
+                            ),
+                        Array(
                                 "idiocy",
-                               "idiocy",
-                               "idiocy",
-                                "url(\\1$secremoveimg\\2)",
-                                "url(\\1$secremoveimg\\2)"
-                               )
-                          )
+                                "idiocy",
+                                "idiocy",
+                                "idiocy",
+                                "",
+                                "url(\\1#\\1)",
+                                "url(\\1#\\1)",
+                                "url(\\1#\\1)",
+                                "url(\\1#\\1)",
+                                "\\1:url(\\2#\\3)"
+                            )
+                        )
                 )
         );
+    if( !sqgetGlobalVar('view_unsafe_images', $view_unsafe_images, SQ_GET) ) {
+        $view_unsafe_images = false;
+    }
     if (!$view_unsafe_images){
         /**
-         * Remove any references to http/https if view_unsafe_images set
-         * to false.
-         */
-         array_push($bad_attvals{'/.*/'}{'/^src|background|href|action/i'}[0],
+        * Remove any references to http/https if view_unsafe_images set
+        * to false.
+        */
+        array_push($bad_attvals{'/.*/'}{'/^src|background/i'}[0],
                     '/^([\'\"])\s*https*:.*([\'\"])/si');
-         array_push($bad_attvals{'/.*/'}{'/^src|background|href|action/i'}[1],
-                    "\\1$secremoveimg\\2");
-         array_push($bad_attvals{'/.*/'}{'/^style/si'}[0],
+        array_push($bad_attvals{'/.*/'}{'/^src|background/i'}[1],
+                    "\\1$secremoveimg\\1");
+        array_push($bad_attvals{'/.*/'}{'/^style/i'}[0],
                     '/url\(([\'\"])\s*https*:.*([\'\"])\)/si');
-         array_push($bad_attvals{'/.*/'}{'/^style/si'}[1],
-                    "url(\\1$secremoveimg\\2)");
+        array_push($bad_attvals{'/.*/'}{'/^style/i'}[1],
+                    "url(\\1$secremoveimg\\1)");
     }
 
     $add_attr_to_tag = Array(
-                             "/^a$/si" => Array('target'=>'"_new"')
-                             );
-    $trusted = sq_sanitize($body, 
-                           $tag_list, 
-                           $rm_tags_with_content,
-                           $self_closing_tags,
-                           $force_tag_closing,
-                           $rm_attnames,
-                           $bad_attvals,
-                           $add_attr_to_tag,
-                           $message,
-                           $id
-                           );
-    if (preg_match("|$secremoveimg|si", $trusted)){
+        "/^a$/i" =>
+            Array('target'=>'"_blank"',
+                'title'=>'"'._("This external link will open in a new window").'"'
+            )
+    );
+    $trusted = sq_sanitize($body,
+                        $tag_list,
+                        $rm_tags_with_content,
+                        $self_closing_tags,
+                        $force_tag_closing,
+                        $rm_attnames,
+                        $bad_attvals,
+                        $add_attr_to_tag,
+                        $message,
+                        $id,
+                        $mailbox
+                        );
+    if (preg_match("|$secremoveimg|i", $trusted)){
         $has_unsafe_images = true;
     }
+
+
+    // we want to parse mailto's in HTML output, change to SM compose links
+    // this is a modified version of code from url_parser.php... but Marc is
+    // right: we need a better filtering implementation; adding this randomly
+    // here is not a great solution
+    //
+    if ($take_mailto_links) {
+        // parseUrl($trusted);   // this even parses URLs inside of tags... too aggressive
+        global $MailTo_PReg_Match;
+        $MailTo_PReg_Match = '/mailto:' . substr($MailTo_PReg_Match, 1);
+        if ((preg_match_all($MailTo_PReg_Match, $trusted, $regs)) && ($regs[0][0] != '')) {
+            foreach ($regs[0] as $i => $mailto_before) {
+                $mailto_params = $regs[10][$i];
+     
+                // get rid of any tailing quote since we have to add send_to to the end
+                //
+                if (substr($mailto_before, strlen($mailto_before) - 1) == '"')
+                    $mailto_before = substr($mailto_before, 0, strlen($mailto_before) - 1);
+                if (substr($mailto_params, strlen($mailto_params) - 1) == '"')
+                    $mailto_params = substr($mailto_params, 0, strlen($mailto_params) - 1);
+    
+                if ($regs[1][$i]) {    //if there is an email addr before '?', we need to merge it with the params
+                    $to = 'to=' . $regs[1][$i];
+                    if (strpos($mailto_params, 'to=') > -1)    //already a 'to='
+                        $mailto_params = str_replace('to=', $to . '%2C%20', $mailto_params);
+                    else {
+                        if ($mailto_params)    //already some params, append to them
+                            $mailto_params .= '&amp;' . $to;
+                        else
+                            $mailto_params .= '?' . $to;
+                    }
+                }
+    
+                $url_str = preg_replace(array('/to=/i', '/(?<!b)cc=/i', '/bcc=/i'), array('send_to=', 'send_to_cc=', 'send_to_bcc='), $mailto_params);
+    
+                // we'll already have target=_blank, no need to allow comp_in_new
+                // here (which would be a lot more work anyway)
+                //
+                global $compose_new_win;
+                $temp_comp_in_new = $compose_new_win;
+                $compose_new_win = 0;
+                $comp_uri = makeComposeLink('src/compose.php' . $url_str, $mailto_before);
+                $compose_new_win = $temp_comp_in_new;
+    
+                // remove <a href=" and anything after the next quote (we only
+                // need the uri, not the link HTML) in compose uri
+                //
+                $comp_uri = substr($comp_uri, 9);
+                $comp_uri = substr($comp_uri, 0, strpos($comp_uri, '"', 1));
+                $trusted = str_replace($mailto_before, $comp_uri, $trusted);
+            }
+        }
+    }
+
     return $trusted;
 }
+
+/**
+* function SendDownloadHeaders - send file to the browser
+*
+* Original Source: SM core src/download.php
+* moved here to make it available to other code, and separate
+* front end from back end functionality.
+*
+* @param string $type0 first half of mime type
+* @param string $type1 second half of mime type
+* @param string $filename filename to tell the browser for downloaded file
+* @param boolean $force whether to force the download dialog to pop
+* @param optional integer $filesize send the Content-Header and length to the browser
+* @return void
+*/
+function SendDownloadHeaders($type0, $type1, $filename, $force, $filesize=0) {
+    global $languages, $squirrelmail_language;
+    $isIE = $isIE6 = 0;
+
+    sqgetGlobalVar('HTTP_USER_AGENT', $HTTP_USER_AGENT, SQ_SERVER);
+
+    if (strstr($HTTP_USER_AGENT, 'compatible; MSIE ') !== false &&
+        strstr($HTTP_USER_AGENT, 'Opera') === false) {
+        $isIE = 1;
+    }
+
+    if (strstr($HTTP_USER_AGENT, 'compatible; MSIE 6') !== false &&
+        strstr($HTTP_USER_AGENT, 'Opera') === false) {
+        $isIE6 = 1;
+    }
+
+    if (isset($languages[$squirrelmail_language]['XTRA_CODE']) &&
+        function_exists($languages[$squirrelmail_language]['XTRA_CODE'] . '_downloadfilename')) {
+        $filename =
+        call_user_func($languages[$squirrelmail_language]['XTRA_CODE'] . '_downloadfilename', $filename, $HTTP_USER_AGENT);
+    } else {
+        $filename = ereg_replace('[\\/:\*\?"<>\|;]', '_', str_replace('&nbsp;', ' ', $filename));
+    }
+
+    // A Pox on Microsoft and it's Internet Explorer!
+    //
+    // IE has lots of bugs with file downloads.
+    // It also has problems with SSL.  Both of these cause problems
+    // for us in this function.
+    //
+    // See this article on Cache Control headers and SSL
+    // http://support.microsoft.com/default.aspx?scid=kb;en-us;323308
+    //
+    // The best thing you can do for IE is to upgrade to the latest
+    // version
+    //set all the Cache Control Headers for IE
+    if ($isIE) {
+        $filename=rawurlencode($filename);
+        header ("Pragma: public");
+        header ("Cache-Control: no-store, max-age=0, no-cache, must-revalidate"); # HTTP/1.1
+        header ("Cache-Control: post-check=0, pre-check=0", false);
+        header ("Cache-control: private");
+
+        //set the inline header for IE, we'll add the attachment header later if we need it
+        header ("Content-Disposition: inline; filename=$filename");
+    }
+
+    if (!$force) {
+        // Try to show in browser window
+        header ("Content-Disposition: inline; filename=\"$filename\"");
+        header ("Content-Type: $type0/$type1; name=\"$filename\"");
+    } else {
+        // Try to pop up the "save as" box
+
+        // IE makes this hard.  It pops up 2 save boxes, or none.
+        // http://support.microsoft.com/support/kb/articles/Q238/5/88.ASP
+        // http://support.microsoft.com/default.aspx?scid=kb;EN-US;260519
+        // But, according to Microsoft, it is "RFC compliant but doesn't
+        // take into account some deviations that allowed within the
+        // specification."  Doesn't that mean RFC non-compliant?
+        // http://support.microsoft.com/support/kb/articles/Q258/4/52.ASP
+
+        // all browsers need the application/octet-stream header for this
+        header ("Content-Type: application/octet-stream; name=\"$filename\"");
+
+        // http://support.microsoft.com/support/kb/articles/Q182/3/15.asp
+        // Do not have quotes around filename, but that applied to
+        // "attachment"... does it apply to inline too?
+        header ("Content-Disposition: attachment; filename=\"$filename\"");
+
+        if ($isIE && !$isIE6) {
+            // This combination seems to work mostly.  IE 5.5 SP 1 has
+            // known issues (see the Microsoft Knowledge Base)
+
+            // This works for most types, but doesn't work with Word files
+            header ("Content-Type: application/download; name=\"$filename\"");
+
+            // These are spares, just in case.  :-)
+            //header("Content-Type: $type0/$type1; name=\"$filename\"");
+            //header("Content-Type: application/x-msdownload; name=\"$filename\"");
+            //header("Content-Type: application/octet-stream; name=\"$filename\"");
+        } else {
+            // another application/octet-stream forces download for Netscape
+            header ("Content-Type: application/octet-stream; name=\"$filename\"");
+        }
+    }
+
+    //send the content-length header if the calling function provides it
+    if ($filesize > 0) {
+        header("Content-Length: $filesize");
+    }
+
+}  // end fn SendDownloadHeaders
+
 ?>
\ No newline at end of file