[CONTROLLER-572] DOMForwardedWriteTransaction<T> Is not Thread Safe Created: 20/Jun/14 Updated: 01/Jul/14 Due: 24/Jun/14 Resolved: 01/Jul/14 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | mdsal |
| Affects Version/s: | Helium |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Devin Avery | Assignee: | Tony Tkacik |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: Mac OS |
||
| External issue ID: | 1227 |
| Description |
|
The class DOMForwardedWriteTransaction is not thread safe. private variables "canceled" and commitImpl are not synchronized in all cases where they are accessed (specifically, put, delete, merge). Any place where you access write to, OR read from a variable on multiple threads you need to synchronize, or mark the variable as volatile. |
| Comments |
| Comment by Tony Tkacik [ 23/Jun/14 ] |
| Comment by Tony Tkacik [ 26/Jun/14 ] |