Issue 121188 - [non-pro] Crash with Undo immediately after insert new slide
Summary: [non-pro] Crash with Undo immediately after insert new slide
Status: CONFIRMED
Alias: None
Product: Impress
Classification: Application
Component: editing (show other issues)
Version: 4.0.0-dev
Hardware: All All
: P3 Normal (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-09 13:30 UTC by Regina Henschel
Modified: 2013-06-20 09:53 UTC (History)
6 users (show)

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


Attachments
Screenshot of error message. (22.66 KB, image/png)
2012-10-09 13:30 UTC, Regina Henschel
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description Regina Henschel 2012-10-09 13:30:42 UTC
Created attachment 79753 [details]
Screenshot of error message.

I use a non-pro build of r1395635 on WinXP.
The error occurs the same way in Draw.

Take an Impress or a Draw document.
Insert new slide/page.
Undo.

The application crashes in MS c++ Runtime Library.
Assertion failed!
In file main/sd/source/ui/slidesorter/controller/SlsPageSelector.cxx
Expression nSelectionCount==mnSelectedPageCount

I'll attach a screenshot of the error message.
Comment 1 Ilari Henrik Aegerter 2012-10-27 22:57:53 UTC
I could not reproduce this bug with the following latest build
AOO350m1(Build:9611)  -  Rev. 1400866 (German)

Configuration:
Windows 7 Ultimate, 32Bit (Adding this piece of information because above under platform it states: All)

Tried the following workflows:

a)
1. Open Impress
2. Empty Presentation -> Create
3. Added a new slide
4. Undo (both with the arrow icon and through the menu)

b)
1. Open an existing Impress document
2. Added a new slide
3. Undo (both with the arrow icon and through the menu)
Comment 2 Regina Henschel 2012-10-28 10:37:13 UTC
Do you use a non-pro build?
Comment 3 Rainer Bielefeld 2013-05-26 06:29:19 UTC
<http://wiki.openoffice.org/wiki/Non_Product_Build>

@Regina:
Still a problem?
Comment 4 Regina Henschel 2013-05-26 11:54:24 UTC
Yes, the problem still exists with AOO400m1(Build:9700)  -  Rev. 1481886.
It is a "Non Product Build" problem.
Comment 5 Armin Le Grand 2013-05-29 16:27:26 UTC
ALG: Could not reproduce neither on Mac nor on Windows. How do you insert the slide? I used context menu over empty space in SlideSorter view (left of edit view).
Comment 6 Regina Henschel 2013-05-29 16:52:51 UTC
You have to use a non-pro build to see the crash.

I use menu Insert > Slide and then undo-button in the standard toolbar.

It still crashes with r1486846.
Comment 7 Armin Le Grand 2013-05-29 17:18:19 UTC
ALG: Indeed, that way it crashes. Interesting...
Comment 8 Armin Le Grand 2013-05-30 09:53:00 UTC
ALG: If it only happens with non-pro build it's not a too high priority.
Looking at the stack shows that PageSelector::CheckConsistency calls SlideSorterModel::GetPageDescriptor somehow with an invalid nPageIndex. This seems to happen inside the 'assert' after the (nSelectionCount!=mnSelectedPageCount) comparison.

        assert(nSelectionCount==mnSelectedPageCount);

I could not really find out where the 'assert' is implemented/comes from, but it is probably implemented empty in the pro builds, so it does not crash there.

@awf.aoo (adding to cc): Any hints?
Comment 9 Andre 2013-05-30 12:35:16 UTC
A quick search with OpenGrok finds the the assertion in
PageSelector::CheckConsistency().  But that is, of course, not the root cause of the problem.  It just tells you that the slidesorter and the Impress model have different ideas of the number of selected slides.
Comment 10 Oliver-Rainer Wittmann 2013-06-20 09:53:31 UTC
as this is clearly a problem of the non-pro environment it is not a release blocker.