Issue 100718 - XUIConfigurationPersistence.store() crashes OOo
Summary: XUIConfigurationPersistence.store() crashes OOo
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: OOO310m6
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 3.1
Assignee: stefan.baltzer
QA Contact: issues@framework
URL:
Keywords: crash, regression
Depends on:
Blocks:
 
Reported: 2009-03-31 15:01 UTC by mux2005
Modified: 2009-05-06 13:58 UTC (History)
5 users (show)

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


Attachments
Launch the JAR file to crash your OOo 3.1 (22.80 KB, application/x-compressed)
2009-03-31 15:02 UTC, mux2005
no flags Details
To make this easier to confirm I've put the code into a BeanShell macro. Simply open the document and press the button => OOo 3.1 crashes (9.12 KB, application/vnd.oasis.opendocument.text)
2009-04-02 14:03 UTC, mux2005
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description mux2005 2009-03-31 15:01:42 UTC
Steps to reproduce: 

1. Launch OpenOffice.org
2. Start the attached program:
   java -jar ShortcutPersistenceBug.jar
3. OpenOffice.org crashes

The call triggering the crash is XUIConfigurationPersistence.store() on the
ShortCutManager. See the source code included in the JAR file for details.

Verified on Linux and Windows.
Comment 1 mux2005 2009-03-31 15:02:42 UTC
Created attachment 61316 [details]
Launch the JAR file to crash your OOo 3.1
Comment 2 mux2005 2009-03-31 15:03:30 UTC
This bug was not present in OOo 3.0.1 => regression
Comment 3 Olaf Felka 2009-03-31 15:38:04 UTC
@ sb: Something for you? Please have a look and/or assign.
Comment 4 Stephan Bergmann 2009-03-31 16:06:19 UTC
.
Comment 5 carsten.driesner 2009-03-31 16:52:46 UTC
cd: Set issue to OOo 3.2 Looks like a problem with the transition from the XML
file based implementation to the configuration.

cd->wuy: Could you please check this issue, too. You are much more familiar with
the code.
Comment 6 Frank Schönheit 2009-04-02 13:19:10 UTC
should this be set to NEW instead of UNCONFIRMED?
Comment 7 carsten.driesner 2009-04-02 13:21:47 UTC
cd->fs: Can you confirm this issue?
Comment 8 Frank Schönheit 2009-04-02 13:29:54 UTC
I didn't try :)
I just assumed that if it already has a target of 3.2, then somebody could
reproduce it.
Comment 9 carsten.driesner 2009-04-02 13:32:52 UTC
cd: This is just a reminder for me to check it for OOo 3.2, but no real
confirmation. Wuyan will check this issue when is back from vacation.
Comment 10 mux2005 2009-04-02 14:03:11 UTC
Created attachment 61363 [details]
To make this easier to confirm I've put the code into a BeanShell macro. Simply open the document and press the button => OOo 3.1 crashes
Comment 11 kpalagin 2009-04-02 14:56:28 UTC
Confirming - m44 crashes on button push.
Keyword.
Comment 12 carsten.driesner 2009-04-02 15:23:35 UTC
cd->kpalagin: Thanks for the confirmation. Could you or someone else confirm
this issue for OOO310, too. I am currently busy with a show stopper therefore
not able to do it.
Comment 13 kpalagin 2009-04-02 15:42:28 UTC
m8 crashes as well.
Comment 14 carsten.driesner 2009-04-02 16:40:47 UTC
cd->kpalagin: Thanks for the confirmation.

cd: Accepted. Looks like a regression due to the fact that the keyboard settings
are now stored in the Office configuration. I will have a look as soon as
possible to see if this can be fixed easily.
Comment 15 carsten.driesner 2009-04-06 13:19:34 UTC
cd: There is an easy hot-fix for this issue. Therefore I will try to get it on
the show stopper list.
Comment 16 andreas.schluens 2009-04-06 13:50:00 UTC
as: Code reviewed ... workaround was accepted.
Comment 17 clutz 2009-04-07 09:30:18 UTC
This issue still occurs in OOO310_m9 (RC 1). Will the fix be integrated in RC 2?
Comment 18 carsten.driesner 2009-04-07 09:47:55 UTC
cd->clutz: It's logical that this crash occurs on RC1. The issue was fixed
yesterday afternoon, while the RC1 has been available since last week. Yes, I
think that this issue will be fixed for RC2.
Comment 19 carsten.driesner 2009-04-07 16:17:56 UTC
cd->sba: Please verify. You can use the attached BeanShell script.
Comment 20 stefan.baltzer 2009-04-09 09:47:14 UTC
Verified in CWS fwk111.
Comment 21 stefan.baltzer 2009-05-06 13:58:46 UTC
Re-verified in OOO310_m11 (OOo 3.1 RC2).
Closed.

Note: CWS fwk111 was just cloned for Integration into DEV300 tree, so this issue
should be fixed for Target OOo 3.2 in forthcoming DEV300_m48 as well.