Bug 128001 - Assertion when accessing animation properties from sidebar
Summary: Assertion when accessing animation properties from sidebar
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
6.4.0.0.alpha1+
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: haveBacktrace
Depends on:
Blocks: Crash-Assert
  Show dependency treegraph
 
Reported: 2019-10-07 15:11 UTC by Julien Nabet
Modified: 2019-10-30 07:55 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
bt Windows (windbg) (36.01 KB, text/plain)
2019-10-07 15:11 UTC, Julien Nabet
Details
bt with debug symbols (14.68 KB, text/plain)
2019-10-07 19:00 UTC, Julien Nabet
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Nabet 2019-10-07 15:11:12 UTC
Description:
On Win10 with master sources updated today, I got a crash when accessing animation properties from sidebar

Steps to Reproduce:
1. Launch Impress
2. Display sidebar
3. Click "Animation" button

Actual Results:
Crash

Expected Results:
No crash


Reproducible: Always


User Profile Reset: Yes



Additional Info:
I don't reproduce this on LO 6.3.2.2
Comment 1 Julien Nabet 2019-10-07 15:11:35 UTC
Created attachment 154802 [details]
bt Windows (windbg)
Comment 2 Julien Nabet 2019-10-07 19:00:41 UTC
Created attachment 154805 [details]
bt with debug symbols

On pc Debian x86-64 with master sources updated today, I got an assertion too.
Comment 3 Aron Budea 2019-10-08 05:11:51 UTC
Confirmed in Ubuntu 19.04. The following is the assertion, perhaps a missing file? I wonder if it's related to bug 126293's fix.

soffice.bin: /home/valaki/projects/libreoffice/vcl/source/gdi/bmpacc.cxx:38: BitmapInfoAccess::BitmapInfoAccess(Bitmap&, BitmapAccessMode): Assertion `xImpBmp && "Forbidden Access to empty bitmap!"' failed.
Comment 4 Julien Nabet 2019-10-08 07:05:58 UTC
Caolán/Noel: thought you might be interested in this one (a bt is included) since it concerns vcl.
Comment 5 Stephan Bergmann 2019-10-09 10:10:16 UTC
For my Linux build using Elementary icon theme, the issue appears to be a missing extensions/res/buttonminus.png.  The links.txt in instdir/share/config/images_elementary.zip contains

  extensions/res/buttonminus.png extensions/res/scanner/minus.png

but instdir/share/config/images_elementary.zip contains no extensions/res/scanner/minus.png, even though there is a source file icon-themes/elementary/extensions/res/scanner/minus.png.  Must be some issue with how postprocess/CustomTarget_images.mk generates those images_*.zip?
Comment 6 Caolán McNamara 2019-10-09 10:35:56 UTC
Is it an incremental build dependency bug ?, we had that before e.g. bug 124023.

FWIW I'm not getting the assert locally
Comment 7 Julien Nabet 2019-10-09 12:49:34 UTC
(In reply to Stephan Bergmann from comment #5)
> For my Linux build using Elementary icon theme, the issue appears to be a
> missing extensions/res/buttonminus.png.  The links.txt in
> instdir/share/config/images_elementary.zip contains
> 
>   extensions/res/buttonminus.png extensions/res/scanner/minus.png
> ...

Just for curiosity, how did you find there was a missing file? Indeed, I don't see the file name concerned in the bt (or perhaps missed it)
Comment 8 Julien Nabet 2019-10-09 12:50:10 UTC
(In reply to Caolán McNamara from comment #6)
> Is it an incremental build dependency bug ?, we had that before e.g. bug
> 124023.
> 
> FWIW I'm not getting the assert locally

I did a make clean on Win10 this morning (I'm UTC+2) and I could reproduce the assertion.
Comment 9 Stephan Bergmann 2019-10-09 16:02:16 UTC
(In reply to Caolán McNamara from comment #6)
> Is it an incremental build dependency bug ?, we had that before e.g. bug
> 124023.

I can still reproduce hitting the "vcl/source/gdi/bmpacc.cxx:38: BitmapInfoAccess::BitmapInfoAccess(Bitmap &, BitmapAccessMode): Assertion `xImpBmp && "Forbidden Access to empty bitmap!"' failed" assert after a full rebuild (on Linux).  (And instdir/share/config/images_elementary.zip still contains no extensions/res/scanner/minus.png.)
Comment 10 Stephan Bergmann 2019-10-09 16:06:57 UTC
(In reply to Julien Nabet from comment #7)
> Just for curiosity, how did you find there was a missing file? Indeed, I
> don't see the file name concerned in the bt (or perhaps missed it)

Once the assert is hit in gdb:

> (gdb) f 16
> #16 0x00007fffc422bd43 in Button::ImplDrawAlignedImage (this=0x6170001bba80, pDev=0x6170001bba80, rPos=Point = {...}, rSize=Size = {...}, nImageSep=7, nTextStyle=5411, pSymbolRect=0x0, bAddImageSep=true) at core/vcl/source/control/button.cxx:439
> 439	        pDev->DrawImage(aImagePos, *pImage, nStyle);
> (gdb) p *pImage->mpImplData.get()
> $1 = {
>   maBitmapChecksum = 0, 
>   maSizePixel = Size = {
>     width = 0,
>     height = 0
>   }, 
>   maPreferedSizePixel = Size = {
>     width = 0,
>     height = 0
>   }, 
>   maStockName = "extensions/res/buttonminus.png", 
[...]
Comment 11 Caolán McNamara 2019-10-29 15:21:10 UTC
$ rm -rf workdir/CustomTarget/postprocess/
$ ls -l icon-themes/elementary/extensions/res/scanner/minus.png
-rw-r--r--. 1 caolan caolan 188 Oct 29 14:59 icon-themes/elementary/extensions/res/scanner/minus.png
$ make postprocess.build
$ unzip -l workdir/CustomTarget/postprocess/images/images_elementary.zip|grep minus
      188  10-29-2019 15:16   extensions/res/scanner/minus.png
      488  10-29-2019 15:16   res/minus.png

no idea why it seems extensions/res/scanner/minus.png goes missing for you, someone who is afflicted by this would have to debug solenv/bin/pack_images.py to see why that file does not get included
Comment 12 Stephan Bergmann 2019-10-29 15:30:23 UTC
Indeed, retrying with my local Linux master build from today, `unzip -l instdir/share/config/images_elementary.zip | grep extensions/res/scanner/minus.png` shows that the file is there, and the reproducer from comment 0 no longer causes a crash for me.
Comment 13 Julien Nabet 2019-10-30 07:55:48 UTC
On pc Debian x86-64 with master sources updated today (and full rebuild some days ago), I don't reproduce this. Let's put this one to WFM then.