[JAMES-609] MailImpl.setMessage and possible NPE: regression from 2.2.0 and 2.3.0rc1

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.3.0
• Fix Version/s: 2.3.0
• Component/s: James Core
• Assignee: Stefano Bagnara
• Reporter: Stefano Bagnara
• Created: 08/Sep/06 17:28
• Updated: 21/Nov/07 08:31
• Resolved: 12/Sep/06 12:44

Description

The change introduced in rc2 to fix the file leaks has produced an NPE in a custom mailet.

I suggest to revert the setMessage changes removing the dispose() call for the old message and to put the dispose code in the bundled calling mailets.

This is not a good fix in the long term, but for 2.3.0 I would prefer to keep compatibility with 2.2.0 and to have a leak instead of NPE (difficult to understand by most users).