Issue 85444

Summary: Extension updates fail
Product: General Reporter: joachim.lingner
Component: codeAssignee: jsc
Status: CLOSED FIXED QA Contact: issues@framework <issues>
Severity: Trivial    
Priority: P2 CC: issues
Version: OOH680m3   
Target Milestone: OOo 2.4   
Hardware: All   
OS: All   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on:    
Issue Blocks: 84957    

Description joachim.lingner 2008-01-21 15:34:33 UTC
The fix for issue 85290 brought up another threading issue in the bootstrap
variable code in rtl. I installed the test extensions from
desktop/test/deployment/update/publisher and ran the extension update. About 30
-50% of the updates failed.

The reason is, that the code in sal/rtl/source/macro.cxx is not thread safe. 
The code was reimplemented for issue 84200 which will go into 3.0. Because the
reimplementation bears a higher risk of regression, I will only remove a little
code which causes the problem. Then in 3.0 we will have the better solution.
Comment 1 joachim.lingner 2008-01-21 15:36:10 UTC
.
Comment 2 joachim.lingner 2008-01-21 16:09:25 UTC
.
Comment 3 Stephan Bergmann 2008-01-21 16:10:28 UTC
verified by code review
Comment 4 joachim.lingner 2008-01-24 08:44:06 UTC
jl->jsk: Just to make sure, please verify that the installation of the extension
updates works properly. The problem was visible on Windows. When running the
update, the description box contained error messages indicating something like
"*** recursion detected***".

I used the test extensions in desktop/test/deployment/update/publisher.
Comment 5 jsc 2008-02-08 11:15:21 UTC
verified on cws jl87
Comment 6 jsc 2008-02-08 11:26:52 UTC
change owner
Comment 7 jsc 2008-06-20 12:21:54 UTC
closed