Issue 103720 - OOO crashes on selection of a certain slide
Summary: OOO crashes on selection of a certain slide
Status: CLOSED FIXED
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
URL:
Keywords: crash, regression
Depends on:
Blocks:
 
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: ---

Attachments
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
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 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
again.
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