Issue 85444 - Extension updates fail
Summary: Extension updates fail
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: OOH680m3
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 2.4
Assignee: jsc
QA Contact: issues@framework
URL:
Keywords:
Depends on:
Blocks: 84957
  Show dependency tree
 
Reported: 2008-01-21 15:34 UTC by joachim.lingner
Modified: 2008-06-20 12:21 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
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