Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Component/s: Core/Parsing
-
Labels:None
-
Environment:Load a document page that takes a long time to initialize. Close the document immediately after opening. The viewer RI will appear to hang until the page(s) are done loading. Capture a stack trace to see the lock issue.
Description
A client has reported a thread lock issue when trying to close a document that contains pages that take a long time to initialize. See attached stack trace. The report is valid.
When a page is loaded the page initializer thread gets a lock on the Page object. When the awt-event thread gets the command to dispose the document it can't call page.removePaintPagelistener because of the page initializer lock. Moving the add/removePaintPageListeners synchronization to the vector object gets around this issue.
When a page is loaded the page initializer thread gets a lock on the Page object. When the awt-event thread gets the command to dispose the document it can't call page.removePaintPagelistener because of the page initializer lock. Moving the add/removePaintPageListeners synchronization to the vector object gets around this issue.
Activity
Patrick Corless
created issue -
Patrick Corless
made changes -
Field | Original Value | New Value |
---|---|---|
Salesforce Case | [] | |
Component/s | Core [ 10022 ] | |
Fix Version/s | 4.0.2 [ 10234 ] |
Tyler Johnson
made changes -
Salesforce Case | [5007000000Ar2To] |
Dirk Maass
made changes -
Attachment | icepdf4.0.1-fixSynchronizationInPage.patch [ 12304 ] |
Patrick Corless
made changes -
Fix Version/s | 4.1 [ 10227 ] | |
Fix Version/s | 4.0.2 [ 10234 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #21610 | Wed Jun 02 06:36:04 MDT 2010 | patrick.corless | |
Files Changed | ||||
MODIFY
/icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/Page.java
|
Patrick Corless
made changes -
Fix Version/s | 4.0.2 [ 10234 ] | |
Fix Version/s | 4.1 [ 10227 ] |
Patrick Corless
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Patrick Corless
made changes -
Fix Version/s | 4.1 [ 10227 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Managed to reproduce the error and test the fix, all seem to be good.