Bug 54558 - FILEOPEN: Crashes immediately upon opening Excel Macro file.
Summary: FILEOPEN: Crashes immediately upon opening Excel Macro file.
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
3.6.1.2 release
Hardware: Other All
: medium normal
Assignee: Markus Mohrhard
URL:
Whiteboard: BSA target:3.7.0 target:3.6.3
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-05 13:34 UTC by ayr035+bugs
Modified: 2012-10-01 12:26 UTC (History)
0 users

See Also:
Crash report or crash signature:


Attachments
The excel macro file that causes the crash. (129.56 KB, application/vnd.ms-excel.sheet.macroEnabled.12)
2012-09-05 13:34 UTC, ayr035+bugs
Details
The excel macro file that causes the crash. (110.80 KB, application/vnd.ms-excel.sheet.macroEnabled.12)
2012-09-05 13:39 UTC, ayr035+bugs
Details

Note You need to log in before you can comment on or make changes to this bug.
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.