Issue 107615 - Customize Toolbar for DataSourceBrowser crashes Office
Summary: Customize Toolbar for DataSourceBrowser crashes Office
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: DEV300m67
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 3.3
Assignee: Olaf Felka
QA Contact: issues@framework
URL:
Keywords: crash
Depends on:
Blocks: 111112
  Show dependency tree
 
Reported: 2009-12-11 11:23 UTC by Ariel Constenla-Haile
Modified: 2017-05-20 10:22 UTC (History)
3 users (show)

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


Attachments
gdb backtrace (13.06 KB, text/plain)
2009-12-11 11:24 UTC, Ariel Constenla-Haile
no flags Details
Patch to solve the crash (1.65 KB, text/plain)
2009-12-11 14:19 UTC, carsten.driesner
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description Ariel Constenla-Haile 2009-12-11 11:23:02 UTC
Steps to reproduce:

* open an OOo Writer document
* press F4 to open the DataSourceBrowser
* right click on the DSB toolbar, and select "Customize Toolbar..."
* press the "Menues" tab
* OOo crashes

The problem seems to be that MenuSaveInData::LoadSubMenus is invoked with a null
reference 
http://svn.services.openoffice.org/opengrok/xref/DEV300_m67/svx/source/cui/cfg.cxx#1185

(gdb) print xMenuSettings
$1 = (const
com::sun::star::uno::Reference<com::sun::star::container::XIndexAccess> &)
@0x7f95b06ac978:
    {<com::sun::star::uno::BaseReference> = {_pInterface = 0x0}, <No data fields>}
(gdb) print xMenuSettings.is()
$2 = 0 '\000'

so xMenuSettings->getCount() crashes OOo.
Comment 1 Ariel Constenla-Haile 2009-12-11 11:24:14 UTC
Created attachment 66606 [details]
gdb backtrace
Comment 2 carsten.driesner 2009-12-11 11:39:49 UTC
cd->arielch: Although you found a crash this is by no means a P1. Please have a
look at the following web page which describes the priority system.
http://qa.openoffice.org/scdocs/ddIssues_EnterModify.html#priority

cd: Set priority to P2 and tm on CC.
Comment 3 Ariel Constenla-Haile 2009-12-11 11:45:03 UTC
arielch_>cd: ups, I thought crashes were always P1; thought P2 makes sense here
"Crashs or freezes during normal operations of the application"
Comment 4 carsten.driesner 2009-12-11 14:19:17 UTC
Created attachment 66612 [details]
Patch to solve the crash
Comment 5 carsten.driesner 2009-12-11 14:19:56 UTC
cd: Added patch to fix the crash.
Comment 6 carsten.driesner 2009-12-14 14:49:56 UTC
cd: Set mav on CC.
Comment 7 carsten.driesner 2009-12-14 14:51:31 UTC
cd: Set cd on CC.

cd->mav: Please take over as I will be in vacation.
Comment 8 mikhail.voytenko 2010-07-06 13:26:10 UTC
Sending to you back. Unfortunately it was not marked as a patch, so it still was
not integrated in 3.3 branch together with other patches.
Comment 9 carsten.driesner 2010-08-25 14:39:49 UTC
cd: Change title to better describe the problem.
Comment 10 mikhail.voytenko 2010-08-31 08:22:29 UTC
The patch is integrated in cws fwk155.
Comment 11 carsten.driesner 2010-09-01 09:28:26 UTC
cd->of: Please verify.
Comment 12 Olaf Felka 2010-09-02 07:06:15 UTC
of: Looks good in cws fwk155.