Issue Details (XML | Word | Printable)

Key: DERBY-3308
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Kristian Waagan
Reporter: Kristian Waagan
Votes: 0
Watchers: 0
Operations

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

Broken synchronization for event handling in ClientPooledConnection40

Created: 09/Jan/08 01:15 PM   Updated: 11/Feb/08 01:28 PM
Return to search
Component/s: JDBC, Network Client
Affects Version/s: 10.2.2.0, 10.3.2.1, 10.4.1.3
Fix Version/s: 10.2.2.1, 10.3.3.0, 10.4.1.3

Time Tracking:
Issue & Sub-Tasks
Issue Only
Not Specified

File Attachments:
  Size
File Licensed for inclusion in ASF works derby-3308-1a-eventlisteners_synchronization.diff 2008-01-23 12:30 PM Kristian Waagan 4 kB
File Licensed for inclusion in ASF works derby-3308-1b-eventlisteners_synchronization.diff 2008-01-23 01:58 PM Kristian Waagan 4 kB

Resolution Date: 23/Jan/08 02:13 PM

Sub-Tasks  All   Open   
No sub-tasks match this view.

 Description  « Hide
Access to the the list of event listeners is not synchronized properly.

Taken the rather infrequent use of the relevant methods and the small critical sections, I mean it is sufficient to add synchronization to all the methods that access it at the method level. The same approach is taken in ClientPooledConnection (although not followed through consistently, some of the methods are unsynchronized).

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Repository Revision Date User Message
ASF #614536 Wed Jan 23 14:12:02 UTC 2008 kristwaa DERBY-3308: Broken synchronization for event handling in ClientPooledConnection40.
Patch file: DERBY-3308-1b-eventlisteners_synchronization.diff
Files Changed
MODIFY /db/derby/code/trunk/java/client/org/apache/derby/client/ClientPooledConnection40.java

Repository Revision Date User Message
ASF #618639 Tue Feb 05 13:15:55 UTC 2008 kristwaa DERBY-3308: Broken synchronization for event handling in ClientPooledConnection40. Merged revision 614536 from trunk (10.4).
Files Changed
MODIFY /db/derby/code/branches/10.2/java/client/org/apache/derby/client/ClientPooledConnection40.java

Repository Revision Date User Message
ASF #618640 Tue Feb 05 13:15:56 UTC 2008 kristwaa DERBY-3308: Broken synchronization for event handling in ClientPooledConnection40. Merged revision 614536 from trunk (10.4).
Files Changed
MODIFY /db/derby/code/branches/10.3/java/client/org/apache/derby/client/ClientPooledConnection40.java