Issue 38297 - Crash when loading company takeover sample
Summary: Crash when loading company takeover sample
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: ui (show other issues)
Version: 680m64
Hardware: All All
: P1 (highest) Trivial (vote)
Target Milestone: OOo 2.0
Assignee: wolframgarten
QA Contact: issues@graphics
URL:
Keywords:
: 38580 (view as issue list)
Depends on:
Blocks:
 
Reported: 2004-12-02 09:25 UTC by wolframgarten
Modified: 2004-12-16 11:29 UTC (History)
1 user (show)

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

Attachments
patch for svx/source/svdraw/svdoole2.cxx (734 bytes, patch)
2004-12-02 15:59 UTC, groucho266 		no flags Details | Diff
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 wolframgarten 2004-12-02 09:25:53 UTC 
Load the sample company takeover: Office crashes. Other samples work well.
Comment 1 groucho266 2004-12-02 10:38:06 UTC 
This crash can be reproduced by other operations as well, for example create a
new slide.
Comment 2 groucho266 2004-12-02 10:38:26 UTC 
Accepted.
Comment 3 groucho266 2004-12-02 13:42:18 UTC 
In SdrOle2Obj::ImpSetVisAreaSize() the xObjRef member is accessed without
testing it for existance of the referenced object.  This referenced object does
indeed not exist which causes the crash.

The missing referenced object is a legal case for a newly created presentation
object.
Comment 4 groucho266 2004-12-02 13:44:52 UTC 
I set the priority to P1 because the same code causes crashes in other
circumstances, too.  So it is better to fix it at once instead of waiting for
the time when we can create a CWS.
Comment 5 groucho266 2004-12-02 15:57:13 UTC 
Mailed the simple fix to RT so that it can be checked in top level.
Comment 6 groucho266 2004-12-02 15:59:45 UTC 
Created attachment 20017 [details]
patch for svx/source/svdraw/svdoole2.cxx
Comment 7 groucho266 2004-12-03 14:55:41 UTC 
Fixed on CWS impress22.
Comment 8 groucho266 2004-12-08 09:54:19 UTC 
Reopening...
Comment 9 groucho266 2004-12-08 09:54:46 UTC 
...reassigning...
Comment 10 groucho266 2004-12-08 09:55:09 UTC 
...resolving.
Comment 11 wolframgarten 2004-12-08 10:31:19 UTC 
Verified in CWS.
Comment 12 wolframgarten 2004-12-10 09:11:17 UTC 
Tested in master m65. Closed.
Comment 13 christian.guenther 2004-12-16 11:29:13 UTC 
*** Issue 38580 has been marked as a duplicate of this issue. ***