To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=110567 Issue #|110567 Summary|desktop: possible thread-unsafe call of Invalidate() Component|framework Version|DEV300m75 Platform|All URL| OS/Version|All Status|NEW Status whiteboard| Keywords| Resolution| Issue type|PATCH Priority|P3 Subcomponent|code Assigned to|mba Reported by|dtardon
------- Additional comments from [email protected] Fri Apr 2 15:03:01 +0000 2010 ------- dp_gui::ExtensionBox_Impl does have five methods that may call Invalidate() -- these are addEntry, removeEntry, updateEntry, checkEntries and selectEntry --, but just one of them is called with the SolarMutex locked -- checkEntries, from dp_gui::ExtMgrDialog::checkEntries. I suppose the other should be guarded too. Anyway, I suppose that concurrent call of Invalidate from addEntry and from the main thread may have caused the crash in the attached stack trace. The attached patch locks the SolarMutex at the place of call of each of these methods, in the same way it's already been done for checkEntries. --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
