[#LANG-369] ExceptionUtils not thread-safe

Commons Lang

ExceptionUtils not thread-safe

Created: 30/Oct/07 05:36 PM Updated: 16/May/09 07:32 AM

Return to search

Component/s:

None

Affects Version/s:

None

Fix Version/s:

3.0, 2.x

Time Tracking:

Not Specified

File Attachments:

File Name	Date Attached	Attached By	Size
LANG-369-2.patch	2009-03-14 02:39 PM	Sebb	3 kB
LANG-369.patch	2007-11-09 08:20 AM	Henri Yandell	4 kB

Resolution Date:

17/Mar/09 08:55 PM

Description

« Hide

The ExceptionUtils class does not appear to be thread-safe:

CAUSE_METHOD_NAMES is not final, so may not be visible to all threads
addCauseMethodName() and removeCauseMethodName() can update CAUSE_METHOD_NAMES but are not synch.
all accesses to CAUSE_METHOD_NAMES probably need to be synchronised

The documentation does not state whether or not the class is thread-safe, but given that it only has static methods it does not make any sense unless it is thread-safe.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Henri Yandell made changes - 03/Nov/07 08:06 AM

Field	Original Value	New Value
Fix Version/s		2.4 [ 12312481 ]

Henri Yandell made changes - 09/Nov/07 08:20 AM

Attachment

LANG-369.patch [ 12369216 ]

Henri Yandell made changes - 12/Nov/07 07:58 PM

Status	Open [ 1 ]	Closed [ 6 ]
Resolution		Fixed [ 1 ]

Sebb made changes - 14/Mar/09 02:33 PM

Status	Closed [ 6 ]	Reopened [ 4 ]
Resolution	Fixed [ 1 ]

Sebb made changes - 14/Mar/09 02:39 PM

Attachment

LANG-369-2.patch [ 12402204 ]

Henri Yandell made changes - 15/Mar/09 02:39 AM

Fix Version/s	2.4 [ 12312481 ]
Fix Version/s		2.x [ 12313695 ]
Fix Version/s		3.0 [ 12311714 ]

Sebb made changes - 17/Mar/09 08:55 PM

Resolution		Fixed [ 1 ]
Status	Reopened [ 4 ]	Resolved [ 5 ]

Henri Yandell made changes - 16/May/09 07:32 AM

Status

Resolved [ 5 ]

Closed [ 6 ]