Issue 87199 - crash when pasting a control from Writer to Calc
Summary: crash when pasting a control from Writer to Calc
Status: CLOSED FIXED
Alias: None
Product: Base
Classification: Application
Component: code (show other issues)
Version: DEV300m3
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 3.0
Assignee: wolframgarten
QA Contact: issues@dba
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2008-03-18 16:12 UTC by Frank Schönheit
Modified: 2008-04-22 14:36 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 Frank Schönheit 2008-03-18 16:12:15 UTC 
- open a new text document
- insert a button from the "Form Controls" toolbar
- copy this button to the clipboard (Edit/Copy)
- open a new spreadsheet document
- paste the button (Edit/Paste)
=> crash
Comment 1 Frank Schönheit 2008-03-18 21:40:33 UTC 
fs->cl: the crash is in XMLStyleExport::exportStyleFamily, where the model which
is being exported (GetExport().GetModel()) is queried for the
XStyleFamiliesSupplier interface, which it does not support - nonetheless, the
NULL pointer resulting from the query is used without checks.

While I think that the this code is too generous - it should check the
XStyleFamiliesSupplier interface it got -, nothing relevant changed between m2
and m3 in xmloff/source/style, so the root cause of the problem must be
somewhere in the model implementation, which is a SvxUnoDrawingModel.

For both classes - SvxUnoDrawingModel and XMLStyleExport -, I think you're
better suited in finding out the cause, or a responsible person, than I am.
Thanks :)
Comment 2 clippka 2008-04-01 12:34:05 UTC 
fixed, same root cause as issue 87243
Comment 3 clippka 2008-04-04 09:16:02 UTC 
back to qa
Comment 4 wolframgarten 2008-04-04 10:04:24 UTC 
Verified in CWS.
Comment 5 wolframgarten 2008-04-22 14:36:18 UTC 
Tested in m10. Closed.