[
https://issues.apache.org/jira/browse/DERBY-3308?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kristian Waagan updated DERBY-3308:
-----------------------------------
Attachment: derby-3308-1b-eventlisteners_synchronization.diff
Thanks for commenting on the patch Knut Anders.
I have incorporated you suggestion about replacing Vector with ArrayList in
revision 1b.
Regarding the suggested changes in the parent class, there are some methods
that are synchronized and some that are not. A patch could be attached to the
subtask of this issue. Feel free to add it, or maybe I will if I get around to
it.
> Broken synchronization for event handling in ClientPooledConnection40
> ---------------------------------------------------------------------
>
> Key: DERBY-3308
> URL: https://issues.apache.org/jira/browse/DERBY-3308
> Project: Derby
> Issue Type: Bug
> Components: JDBC, Network Client
> Affects Versions: 10.2.2.0, 10.3.2.1, 10.4.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.4.0.0
>
> Attachments: derby-3308-1a-eventlisteners_synchronization.diff,
> derby-3308-1b-eventlisteners_synchronization.diff
>
>
> 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).
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
