Bug 331879 - Libreoffice crash when selecting "save as" document
Summary: Libreoffice crash when selecting "save as" document
Status: RESOLVED FIXED
Alias: None
Product: Oxygen
Classification: Plasma
Component: gtk2-engine (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR crash
Target Milestone: ---
Assignee: Hugo Pereira Da Costa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-08 15:07 UTC by diogomsantos
Modified: 2014-03-09 18:44 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description diogomsantos 2014-03-08 15:07:39 UTC 
Libreoffice crashes when selecting "open" or "save as" in menu. No output from terminal.



Reproducible: Always




This is in Arch Linux KDE 4.12.3. Libreoffice 4.1.5. oxygen-gtk2 1.4.4. The same happens with oxygen-gtk2 compiled from git.

Does not happen with oxygen-gtk2 1.4.3
Comment 1 diogomsantos 2014-03-08 15:32:05 UTC 
When compiled from git (oxygen-gtk2) this is the crash output:

soffice.bin: /tmp/packerbuild-1000/oxygen-gtk2-git/oxygen-gtk2-git/src/oxygen-gtk/src/animations/oxygendatamap.h:87: T& Oxygen::DataMap<T>::value(GtkWidget*) [with T = Oxygen::InnerShadowData; GtkWidget = _GtkWidget]: Assertion `iter != _map.end()' failed.
Comment 2 Hugo Pereira Da Costa 2014-03-08 17:13:35 UTC 
Can reproduce.
Failed assertion. Should be easily fixed. Will keep you posted.
Thanks for reporting and sorry for the trouble
Comment 3 Hugo Pereira Da Costa 2014-03-09 16:29:03 UTC 
Git commit b39931fa3ad7a2756bcb1494d5c3559513406ce2 by Hugo Pereira Da Costa.
Committed on 09/03/2014 at 12:00.
Pushed by hpereiradacosta into branch '1.4'.

make sure parent is registered to innerShadowEngine before registering child

M  +6    -2    src/oxygenstylewrapper.cpp

http://commits.kde.org/oxygen-gtk/b39931fa3ad7a2756bcb1494d5c3559513406ce2
Comment 4 Hugo Pereira Da Costa 2014-03-09 16:31:16 UTC 
patch from comment #3 fixes the crash.
it will be included in the next release (soon) of oxygen-gtk2
Similar patch has also been pushed to the gtk3 branch (although the corresponding crash did not bite us yet).

Thanks for reporting.
Comment 5 diogomsantos 2014-03-09 18:44:19 UTC 
Thanks! It is fixed!