Merge branch 'jonny64-v3.4.0_hang'

Author: andris9

lib/mail-parser.js

@@ -509,14 +509,18 @@ class MailParser extends Transform {
     }
 
     decodeAddresses(addresses) {
+        let processedAddress = new WeakSet();
         for (let i = 0; i < addresses.length; i++) {
             let address = addresses[i];
             address.name = (address.name || '').toString().trim();
 
-            if (!address.address && /^(=\?([^?]+)\?[Bb]\?[^?]*\?=)(\s*=\?([^?]+)\?[Bb]\?[^?]*\?=)*$/.test(address.name)) {
+            if (!address.address && /^(=\?([^?]+)\?[Bb]\?[^?]*\?=)(\s*=\?([^?]+)\?[Bb]\?[^?]*\?=)*$/.test(address.name) && !processedAddress.has(address)) {
                 let parsed = addressparser(this.libmime.decodeWords(address.name));
                 if (parsed.length) {
-                    parsed.forEach(entry => addresses.push(entry));
+                    parsed.forEach(entry => {
+                        processedAddress.add(entry);
+                        addresses.push(entry);
+                    });
                 }
 
                 // remove current element

test/issue-337-test.js

@@ -0,0 +1,8 @@
+'use strict';
+
+const simpleParser = require('..').simpleParser;
+
+module.exports['Hangs when Reply-To invalid encoded'] = async test => {
+    await simpleParser('Content-type: text/plain\r\nReply-To: =?U� -8?B?0=D0=BA=D0=BE=D0=B2?=\n\r\n1234');
+    test.done();
+};