Bug 54558

Summary: FILEOPEN: Crashes immediately upon opening Excel Macro file.
Product: LibreOffice Reporter: ayr035+bugs
Component: CalcAssignee: Markus Mohrhard <markus.mohrhard>
Status: RESOLVED FIXED    
Severity: normal    
Priority: medium    
Version: 3.6.1.2 release   
Hardware: Other   
OS: All   
Whiteboard: BSA target:3.7.0 target:3.6.3
Crash report or crash signature: Regression By:
Attachments: The excel macro file that causes the crash.
The excel macro file that causes the crash.

Description ayr035+bugs 2012-09-05 13:34:08 UTC
Created attachment 66675 [details]
The excel macro file that causes the crash.

Problem description: 
Libreoffice crashes immediately upon opening the attached document.  The document opens fine in Libreoffice 3.5.x.

Steps to reproduce:
1. Open attached file.
2. Calc goes BOOM!

Current behavior: Calc crashes.

Expected behavior: Expect at least the behavior of Calc 3.5.6

Platform (if different from the browser): Linux 3.5
              
Browser: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20100101 Firefox/15.0
Comment 1 ayr035+bugs 2012-09-05 13:39:41 UTC
Created attachment 66677 [details]
The excel macro file that causes the crash.
Comment 2 Markus Mohrhard 2012-09-26 23:54:10 UTC
Ok, so I needed some time to understand the issue here and had to read ooxml spec about extLst and ext entries again.

The spec allows to have ext entries in an extLst that don't map any more to an exisiting entry. The standard even requires that consumers write all extLst entries they can't import out again into the document.

I need to change the extLst handling in 3.6 to work with that.
Comment 3 Not Assigned 2012-09-28 00:02:52 UTC
Markus Mohrhard committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=fcd85be5d5e44b00c6fa5717c7ec13ea657483d4

ooxml spec allow ext ref to non existent entry, fdo#54558



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 4 Not Assigned 2012-10-01 09:19:01 UTC
Markus Mohrhard committed a patch related to this issue.
It has been pushed to "libreoffice-3-6":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=6232eeaa90643f2e067145296f8e97fb16f7ecf5&g=libreoffice-3-6

ooxml spec allow ext ref to non existent entry, fdo#54558


It will be available in LibreOffice 3.6.3.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 5 Not Assigned 2012-10-01 10:58:33 UTC
Markus Mohrhard committed a patch related to this issue.
It has been pushed to "libreoffice-3-6":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=1ba25c3ada91963f095c976764d68db039be0fd8&g=libreoffice-3-6

fix vell value import from ooxml with array formulas, fdo#54558


It will be available in LibreOffice 3.6.3.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.