Issue 32314 - crash when window new and make a reload.
Summary: crash when window new and make a reload.
Status: CLOSED DUPLICATE of issue 38341
Alias: None
Product: Draw
Classification: Application
Component: ui (show other issues)
Version: 680m42
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: christian.guenther
QA Contact: issues@graphics
URL:
Keywords:
Depends on: 35524
Blocks:
  Show dependency tree
 
Reported: 2004-07-29 17:22 UTC by christian.guenther
Modified: 2005-01-24 10:44 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 christian.guenther 2004-07-29 17:22:55 UTC 
- occures in the cws impress15ea
- start an Impress / Draw
- click on 'Window/New'
- save the (new) doc
- create an object (e.g. a rectangle)
- make a reload
- click on Yes
- you get a crash.
- was ok in src680m40
Comment 1 groucho266 2004-07-30 15:15:06 UTC 
Accepted.
Comment 2 christian.guenther 2004-08-06 09:28:15 UTC 
change prio
Comment 3 groucho266 2004-08-23 16:59:14 UTC 
The reason for the crash is maybe an invalid controller:
The stacked shells are created asynchronously for a new ViewShellBase.  This is
also true for the controller.  In the short time between creation of
ViewShellBase and that of ViewShell (and with it the controller) a default
SfxBaseController is used.  For some reason this seems to be used as
SdUnoDrawView and thus crashes.

If this is true, then to fix this bug we have to make the creation of the
initial stacked shells synchronous. This has to be done anyway for making Draw
and Impress as OLE objects work.
Comment 4 groucho266 2004-10-15 14:15:39 UTC 
Seems like issue 35524 fixed this crash.

What remains are assertions probably caused by other bugs (see issue 35524 for
details.)
To be sure I have to speak with MBA about them.
Comment 5 groucho266 2004-10-15 15:47:56 UTC 
The other problems have to be solved under their own issues.
Comment 6 groucho266 2004-10-22 15:10:28 UTC 
Step 1/3 of sending issue to QA.
Comment 7 groucho266 2004-10-22 15:10:57 UTC 
Step 2/3 of sending issue to QA.
Comment 8 groucho266 2004-10-22 15:11:12 UTC 
Step 3/3 of sending issue to QA.
Comment 9 wolframgarten 2004-10-27 09:59:51 UTC 
Verified in CWS.
Comment 10 wolframgarten 2004-11-25 15:20:18 UTC 
Reopened.
Comment 11 wolframgarten 2004-11-25 15:20:54 UTC 
Crashes master m63. Back to AF.
Comment 12 groucho266 2004-11-30 10:50:15 UTC 
This new crash is related to but not the same as the original crash.  It still
is an invalid controller that causes the crash.  But this time it is the
ViewShellBase::UpdateController() method that is called once too often.  It is
called from ViewShell::Init() and thus right after the creation of a new
ViewShellBase object.  This is unnecessary since the SFX code that creates the
ViewShellBase also registers the new controller (which is done in
UpdateController()).  In the case of the reload it is not just unnecessary but
dangerous and causes the crash.

Can be fixed by simply remove the call to UpdateController() from
ViewShell::Init().  There is still the call from the PaneManager.
Comment 13 christian.guenther 2004-11-30 14:07:56 UTC 
back to AF
Comment 14 groucho266 2004-11-30 16:13:32 UTC 
Fixed in CWS impress21beta by removing the call to UpdateController() as
described above.
Comment 15 groucho266 2004-12-03 14:46:52 UTC 
See issue 38341 for a related issue that may even cover this one.
Comment 16 groucho266 2004-12-03 17:24:39 UTC 
Reopening...
Comment 17 groucho266 2004-12-03 17:25:06 UTC 
...reassigning...
Comment 18 groucho266 2004-12-03 17:25:25 UTC 
...resolving
Comment 19 groucho266 2004-12-06 09:40:45 UTC 
According to WG the crash is fixed on Windows and Linux but still occurs on Solaris.
Comment 20 groucho266 2004-12-06 09:41:53 UTC 
The crash that still occurs on Solaris has the same root cause as issue 38341.

*** This issue has been marked as a duplicate of 38341 ***
Comment 21 christian.guenther 2005-01-24 10:44:27 UTC 
According to AF I close this issue as duplicate.