Issue 103720 - OOO crashes on selection of a certain slide
Summary: OOO crashes on selection of a certain slide
Alias: None
Product: Draw
Classification: Application
Component: code (show other issues)
Version: DEV300m52
Hardware: PC All
: P2 Trivial (vote)
Target Milestone: OOo 3.2
Assignee: wolframgarten
QA Contact: issues@graphics
Keywords: crash, regression
Depends on:
Reported: 2009-07-22 15:21 UTC by oliverburghard
Modified: 2010-04-22 12:55 UTC (History)
4 users (show)

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

This file crashes ooo. (67.27 KB, text/plain)
2009-07-22 15:21 UTC, oliverburghard
no flags Details
possible fix (671 bytes, patch)
2009-07-24 13:05 UTC, dtardon
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description oliverburghard 2009-07-22 15:21:10 UTC
* open attached ooo file
* select 2nd slide
* crashes ooo
Comment 1 oliverburghard 2009-07-22 15:21:42 UTC
Created attachment 63691 [details]
This file crashes ooo.
Comment 2 kpalagin 2009-07-22 20:32:29 UTC
Also 3.1 and m52 on Windows (no crash reporter).
2.4.2 does not crash.
Comment 3 wolframgarten 2009-07-23 08:07:15 UTC
Crashes m53, too. Ok in 3.0 final. Reproducible. Reassigned.
Comment 4 oliverburghard 2009-07-23 14:53:29 UTC
When saving the file as a ppt, a totally transparent (transparency = 100%) png
turns up. Its transparency is automatically set to 0%. Afterwards the file works
Comment 5 dtardon 2009-07-24 13:04:42 UTC
The problem is in asynchronous loading of graphic objects. In this case, the
sdr::event::EventHandler executes sdr::event::AsynchGraphicLoadingEvent, that
delegates the loading to sdr::contact::ViewObjectContactOfGraphic. There the
object is loaded and ActionChanged() is called. But that can cause calling
createPrimitive2DSequence() and impPrepareGraphicWithAsynchroniousLoading(),
where the event is deleted, if the object has been already loaded, like in this
case. Later, the event is deleted again by EventHandler, leading to crash.
Comment 6 dtardon 2009-07-24 13:05:30 UTC
Created attachment 63721 [details]
possible fix
Comment 7 dtardon 2009-07-24 13:08:51 UTC
changing component
Comment 8 ooo 2009-07-24 13:35:10 UTC
ka=>aw: could you take a closer look, please?
Comment 9 wolframgarten 2009-07-27 08:15:11 UTC
The ID of the error report is rmw5vkc.
Comment 10 Armin Le Grand 2009-07-29 12:25:50 UTC
AW: Checked validity of the fix; it's the right fix at the right place. Thanks
to dtardon for finding and suggesting.
Comment 11 Armin Le Grand 2009-07-29 12:43:05 UTC
AW: Adapted the assertion in forgetAsynchGraphicLoadingEvent which now may
locally have a 0 == mpAsynchLoadEvent state. Commited changes, done.
Comment 12 Armin Le Grand 2009-07-29 12:43:34 UTC
AW: Adapted target
Comment 13 Armin Le Grand 2009-08-05 14:58:10 UTC
AW: sd was not built. With sd built, all is okay. Checked on test build.
Comment 14 Armin Le Grand 2009-08-06 13:50:35 UTC
AW->WG: Please review as described.
Comment 15 wolframgarten 2009-08-10 08:16:29 UTC
Verified in CWS.
Comment 16 dtardon 2010-04-22 12:55:29 UTC
integrated in DEV300_m57