Bug 161664

Summary:	[CRASH] Assert fails when changing object area fill with the color tool in Impress
Product:	LibreOffice	Reporter:	Rafael Lima <rafael.palma.lima>
Component:	Impress	Assignee:	Not Assigned <libreoffice-bugs>
Status:	NEW ---
Severity:	normal	CC:	Armin.Le.Grand, miguelangelrv, rb.henschel, serval2412, stephane.guillou
Priority:	medium	Keywords:	haveBacktrace
Version:	24.8.0.0 alpha1+
Hardware:	All
OS:	All
Whiteboard:
Crash report or crash signature:		Regression By:
Bug Depends on:
Bug Blocks:	85184, 105537
Attachments:	Callstack by VS gdb bt

Description Rafael Lima 2024-06-19 17:44:22 UTC

In Impress and Draw, changing the background color using the Color tool (from the toolbar or from the Properties sidebar) will crash LO.

Steps to reproduce
1) Open Impress or Draw
2) Insert a simple shape
3) Change its background color using the color tool (in the toolbar or sidebar)
4) Crash

It works fine if you right-click and go to Area to change the background color.

On the terminal I'm getting:

warn:svl:25358:25358:svl/source/items/poolitem.cxx:554: comparing different pool item subclasses 14XFillColorItem && 14XFillColorItem
soffice.bin: /home/rafael/Programming/libreoffice/svl/source/items/poolitem.cxx:557: virtual bool SfxPoolItem::operator==(const SfxPoolItem&) const: Assertion `rCmp.ItemType() == ItemType() && "comparing different pool item subclasses"' failed.

System info

Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 6b2aaaa6be71b1289521bd2ffd2d4abacecabb87
CPU threads: 16; OS: Linux 6.8; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Calc: CL threaded

Comment 1 m_a_riosv 2024-06-20 01:11:10 UTC

I can't reproduce
Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 3909a1186cf423105c30c91136f131daf66e18c6
CPU threads: 16; OS: Windows 11 X86_64 (10.0 build 22631); UI render: Skia/Raster; VCL: win
Locale: es-ES (es_ES); UI: en-US
Calc: CL threaded

Comment 2 Stéphane Guillou (stragu) 2024-06-21 14:22:22 UTC

No crash with:

Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 151d997365f7bf271d63af535d29a9c3439c6d46
CPU threads: 8; OS: Linux 6.5; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: CL threaded

Nor with:

Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 151d997365f7bf271d63af535d29a9c3439c6d46
CPU threads: 8; OS: Linux 6.5; UI render: default; VCL: kf5 (cairo+xcb)
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: CL threaded

Is that a debug build, Rafael?

Comment 3 Rafael Lima 2024-06-21 14:30:23 UTC

(In reply to Stéphane Guillou (stragu) from comment #2)
> Is that a debug build, Rafael?

Yeah... it is a debug build.

I'll update my repo and build again... if the crash persists, I'll attach a backtrace. It is possible that the crash only happens with the debug build

Comment 4 Stéphane Guillou (stragu) 2024-06-21 14:40:50 UTC

Reproduced in latest daily debug build:

Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 151d997365f7bf271d63af535d29a9c3439c6d46
CPU threads: 8; OS: Linux 6.5; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: CL threaded

Comment 5 Regina Henschel 2024-06-21 19:27:19 UTC

Created attachment 194900 [details]
Callstack by VS

I see the same problem with line color drop-line list from toolbar.
Draw a line, open the drop-down, select color -> assert fails.

I have used a clean debug-build from today master.
Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: a820696263e62ffc7ebe0a9f6d51bec2e1354926
CPU threads: 32; OS: Windows 11 X86_64 (10.0 build 22631); UI render: default; VCL: win
Locale: de-DE (de_DE); UI: en-US
Calc: threaded

Comment 6 Regina Henschel 2024-06-21 19:33:19 UTC

It happens too for the color drop-down list in the properties part in the Sidebar.

It does not happen, using the color drop-down list in the style dialog when creating a new style.

Comment 7 Julien Nabet 2024-06-22 09:23:45 UTC

Created attachment 194907 [details]
gdb bt

On pc Debian x86-64 with master sources updated yesterday, I got an assertion.

Comment 8 Julien Nabet 2024-06-22 09:24:25 UTC

Armin: seeing poolitem, thought you might be interested in this one.