Do not repeat processing invalidly encoded address

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

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