- Fix handling of message bodies (quoted-printable encoded) with NULL characters (#1486189)

program/include/rcube_imap_generic.php

@@ -2080,6 +2080,7 @@ class rcube_imap_generic
 		        }
             	$bytes -= $len;
 
+                // BASE64
 		        if ($mode == 1) {
 					$line = rtrim($line, "\t\r\n\0\x0B");
 					// create chunks with proper length for base64 decoding
@@ -2092,40 +2093,30 @@ class rcube_imap_generic
 					}
 					else
 						$prev = '';
-
+					$line = base64_decode($line);
-					if ($file)
+                // QUOTED-PRINTABLE
-						fwrite($file, base64_decode($line));
-            		else if ($print)
-						echo base64_decode($line);
-					else
-						$result .= base64_decode($line);
 				} else if ($mode == 2) {
 					$line = rtrim($line, "\t\r\0\x0B");
-					if ($file)
+                    $line = quoted_printable_decode($line);
-						fwrite($file, quoted_printable_decode($line));
+                    // Remove NULL characters (#1486189)
-            		else if ($print)
+                    $line = str_replace("\x00", '', $line);
-						echo quoted_printable_decode($line);
+                // UUENCODE
-					else
-						$result .= quoted_printable_decode($line);
 				} else if ($mode == 3) {
 					$line = rtrim($line, "\t\r\n\0\x0B");
 					if ($line == 'end' || preg_match('/^begin\s+[0-7]+\s+.+$/', $line))
 						continue;
-					if ($file)
+                    $line = convert_uudecode($line);
-						fwrite($file, convert_uudecode($line));
+                // default
-            		else if ($print)
-						echo convert_uudecode($line);
-					else
-						$result .= convert_uudecode($line);
 				} else {
-					$line = rtrim($line, "\t\r\n\0\x0B");
+					$line = rtrim($line, "\t\r\n\0\x0B") . "\n";
+				}
+
     			if ($file)
-						fwrite($file, $line . "\n");
+					fwrite($file, $line);
            		else if ($print)
-						echo $line . "\n";
+					echo $line;
 				else
-						$result .= $line . "\n";
+					$result .= $line;
-				}
         	}
     	}