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
Stefano Bagnara made changes - 08/Sep/06 06:50 PM
Field Original Value New Value
Attachment JAMES-609.diff [ 12340476 ]
Stefano Bagnara made changes - 12/Sep/06 12:44 PM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]
Assignee Stefano Bagnara [ bago ]
Danny Angus made changes - 21/Nov/07 08:31 AM
Status Resolved [ 5 ] Closed [ 6 ]