If mailto: string isn't double URL encoded, the values don't correctly get

fed to compose.php.

ChangeLog

@@ -231,6 +231,8 @@ Version 1.5.1 -- CVS
     option includes.
   - Fixed bug #1124764, view unsafe images inside printer friendly view.
   - Fixed bug #1032366, remove NUL characters in text attachments on sent.
+  - URL Encode required for string being passed in mailto: links to pass on
+    additional values (cc, body, subject etc).
 
 Version 1.5.0
 --------------------

src/redirect.php

@@ -141,7 +141,7 @@ if ( sqgetGlobalVar('session_expired_location', $session_expired_location, SQ_SE
 }
 if($mailto != '') {
     $redirect_url  = $location . '/webmail.php?right_frame=compose.php&mailto=';
-    $redirect_url .= $mailto;
+    $redirect_url .= urlencode($mailto);
 }
 
 /* Write session data and send them off to the appropriate page. */
@@ -176,4 +176,4 @@ function attachment_common_parse($str, $debug) {
     sqsession_register($attachment_common_types, 'attachment_common_types');
 }
 
-?>
+?>