Issue Details (XML | Word | Printable)

Key: JAMES-609
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Stefano Bagnara
Reporter: Stefano Bagnara
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
JAMES Server

MailImpl.setMessage and possible NPE: regression from 2.2.0 and 2.3.0rc1

Created: 08/Sep/06 05:28 PM   Updated: 21/Nov/07 08:31 AM
Return to search
Component/s: James Core
Affects Version/s: 2.3.0
Fix Version/s: 2.3.0

Time Tracking:
Not Specified

File Attachments:
  Size
File Licensed for inclusion in ASF works JAMES-609.diff 2006-09-08 06:50 PM Stefano Bagnara 20 kB

Resolution Date: 12/Sep/06 12:44 PM


 Description  « Hide
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).


 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
No work has yet been logged on this issue.