Bug 101726 - crash when selecting column and pressing ⌘1 or clicking menubar Format > Cells (macOS only)
Summary: crash when selecting column and pressing ⌘1 or clicking menubar Format > Cell...
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
5.3.0.0.alpha0+
Hardware: All macOS (All)
: highest critical
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisectRequest, haveBacktrace, regression
Depends on:
Blocks:
 
Reported: 2016-08-25 10:38 UTC by steve
Modified: 2016-10-20 10:32 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments
crash document (8.27 KB, application/vnd.oasis.opendocument.spreadsheet)
2016-08-25 10:38 UTC, steve
Details
screencast (4.20 MB, video/mp4)
2016-08-26 06:42 UTC, steve
Details
full bt debug build osx 10.11.6 (30.96 KB, text/plain)
2016-08-26 07:40 UTC, Alex Thurgood
Details

Note You need to log in before you can comment on or make changes to this bug.
Description steve 2016-08-25 10:38:44 UTC
Created attachment 127023 [details]
crash document

reproduce

1. open attached file
2. select sheet "Monat"
3. select column A and press ⌘1 (menubar Format > Cells)

Sadly I am not presented with a crash log. The recovery dialog automatically shows up after the crash, recovery works, repeat and enjoy endless loop.
Comment 1 steve 2016-08-25 11:07:43 UTC
Version: 5.3.0.0.alpha0+
Build ID: 3e6052ce1b9a24f82358e574de32919fa2b6499f
CPU Threads: 4; OS Version: Mac OS X 10.11.6; UI Render: default; 
TinderBox: MacOSX-x86_64@49-TDF, Branch:master, Time: 2016-08-23_01:56:05
Locale: de-DE (de.UTF-8); Calc: group

and persisting with nightly from today
Comment 2 Buovjaga 2016-08-25 11:47:33 UTC
No crash.. maybe I should update to the latest.

Win 7 Pro 64-bit Version: 5.3.0.0.alpha0+ (x64)
Build ID: 264466bd604d7613aa0a5e4b34c149ad66414580
CPU Threads: 4; OS Version: Windows 6.1; UI Render: default; 
TinderBox: Win-x86_64@62-TDF, Branch:MASTER, Time: 2016-08-21_21:59:38
Locale: fi-FI (fi_FI); Calc: CL
Comment 3 Julien Nabet 2016-08-25 13:24:55 UTC
Might be MacOs only bug, let's specify hardware for the moment.
Comment 4 Buovjaga 2016-08-26 05:30:22 UTC
No crash.

Win 7 Pro 64-bit Version: 5.3.0.0.alpha0+
Build ID: 1e84b23839d96068c862e746c9162db79d2c8c62
CPU Threads: 4; OS Version: Windows 6.1; UI Render: default; 
TinderBox: Win-x86@39, Branch:master, Time: 2016-08-25_05:49:05
Locale: fi-FI (fi_FI); Calc: CL
Comment 5 steve 2016-08-26 06:33:34 UTC
Comment on attachment 127023 [details]
crash document

crash not related to any specific document
Comment 6 steve 2016-08-26 06:42:45 UTC
Created attachment 127031 [details]
screencast
Comment 7 steve 2016-08-26 06:44:27 UTC
Not related to specific document. Screencast added showing instant crash on selecting Format > Cells with empty sheet.

@Alex could you try to repor this on your macOS machine?
Comment 8 Alex Thurgood 2016-08-26 07:23:35 UTC
Version: 5.3.0.0.alpha0+
Build ID: 03f576466c9f110fbef8923604908654d59484a6
CPU Threads: 2; OS Version: Mac OS X 10.11.6; UI Render: default; 
Locale: fr-FR (fr.UTF-8); Calc: group


Confirming, spectacular !

Will try and get a bt. it makes no difference whether the shortcut or the menu Format > Cell is used, both produce a crash.
Comment 9 Alex Thurgood 2016-08-26 07:40:26 UTC
Created attachment 127032 [details]
full bt debug build osx 10.11.6
Comment 10 Michael Meeks 2016-10-04 09:05:45 UTC
This assertion fails:

include/svl/itemset.hxx: 

class SVL_DLLPUBLIC SfxItemSet
...
    template<class T> const T* GetItem(sal_uInt16 nWhich, bool bSearchInParent = true) const
    {
        const SfxPoolItem* pItem = GetItem(nWhich, bSearchInParent);
        const T* pCastedItem = dynamic_cast<const T*>(pItem);

        assert(!pItem || pCastedItem); // if it exists, must have the correct type
        return pCastedItem;
    }

Kendy - any thoughts on that ? =)
Comment 11 Jan Holesovsky 2016-10-05 10:22:59 UTC
My guess would be that it is down to commit 200e336d1d5c036b5ecd308d93e4131274b1efb8 (only the change in sc/source/ui/inc/attrdlg.hxx), but of course bibisect / bisect would be most appreciated.
Comment 12 Jan Holesovsky 2016-10-05 10:27:51 UTC
Then again, the commit is a year old (was confused that it was in August too, but did not check the year).  Really, need a bibisect here I fear.
Comment 13 Buovjaga 2016-10-05 10:38:24 UTC
Reverting status change as bibisectRequest is not a valid reason for NEEDINFO (and will cause the report to be ignored by bibisecters most likely).
Comment 14 steve 2016-10-20 10:32:54 UTC
works in Version: 5.3.0.0.alpha0+
Build ID: 40fc2c1a0d2ebdf47131651045107c9d5abb850d
CPU Threads: 4; OS Version: Mac OS X 10.12; UI Render: default; 
TinderBox: MacOSX-x86_64@49-TDF, Branch:master, Time: 2016-10-20_06:13:56
Locale: de-DE (de_DE.UTF-8); Calc: group