Bug 100143 - FILEOPEN: LibO crashes opening an .XLS file
Summary: FILEOPEN: LibO crashes opening an .XLS file
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
5.1.0.3 release
Hardware: All Windows (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: OpenCL
  Show dependency treegraph
 
Reported: 2016-05-30 17:27 UTC by Petr Pisar
Modified: 2018-06-03 19:02 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
Offending XLS file (11.50 KB, application/vnd.ms-excel)
2016-05-30 17:27 UTC, Petr Pisar
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Petr Pisar 2016-05-30 17:27:58 UTC
Created attachment 125394 [details]
Offending XLS file

I have an XLS file (attached) that causes libreoffice 5.1 to crash whenever I will try to open the file.

Affected versions are 32-bit Windows builds of 5.1:

5.1.1.3
5.1.2.2
5.1.3.2

Opening the file works with 5.0 versions. It also works on 64-bit Linux with the 5.1.
Comment 1 Julien Nabet 2016-05-30 20:09:31 UTC
Is Java 32 bits (1.6, 1.7, 1.8) installed or only Java 64 bits?

Indeed, LO 32 bits can only work with Java 32 bits.
(and LO 64 bits can only work with Java 64 bits)

Also, for the test, could you rename your LO directory profile (see https://wiki.documentfoundation.org/UserProfile#Windows)?
Comment 2 Petr Pisar 2016-05-31 04:44:45 UTC
The Windows machine has 64-bit CPU, but all the software including operating system is 32-bit. There is Oracle JRE 1.8.

The Linux machine has everything 64-bit and OpenJDK JRE 1.8.

I will check the clean profile later.
Comment 3 Petr Pisar 2016-06-05 18:17:51 UTC
I renamed the directory with the profile ("4") and then started LO. It did not help. (I kept the old "3" directory. I believe it does not matter.)

I did other tests and I found that the crash happens only if software OpenCL interpreter is enabled. Since LO 1.5, there are two options for OpenCL in the Options → LibreOffice → OpenCL menu. LO 1.4 has only "Allow use of OpenCL". If I disable the new "Allow use of software interpreter", the crash disappear.

The machine where it happens has an 64-bit Intel CPU although the Windows Vista Pro operating system is 32-bit. The machine has ATI Radeon HD 4300/4500 Series graphics card (and an Intel integrated that is disabled), Windows identify it as "PCI\VEN_1002&DEV_954F&SUBSYS_E113174B&REV_00". The driver is the latest one 8.970.100.1100 version from AMD.

The crash does not change if I disable JRE.
Comment 4 Michael Meeks 2016-07-14 13:24:18 UTC
Thanks for your analysis and report - we now have code in 5.1.5.1 and onwards (any build from yesterday/today should do) which exercises the OpenCL driver early, and if/when that crashes disables OpenCL - which should (I hope) help your case.

Is there any chance you can test 5.1.5.1 which is currently here: http://dev-builds.libreoffice.org/pre-releases/win/ =)

Thanks !
Comment 5 Petr Pisar 2016-07-17 18:53:11 UTC
With 5.1.5.1, if the "Allow use of OpenCL" is enabled, the program crashes. On next start, the option is disabled. Although the first crash looks scary, it's probably what you wanted to hear. Maybe a notice "OpenCL was disabled because it seems to crash LibreOffice" on next start would be great. People would then know what happened that it will not happen anymore.
Comment 6 How can I remove my account? 2016-07-28 08:11:50 UTC
Comments 3 and 5 are inconsistent. Comment 3 indicates that it would be the "software interpreter" (which is *NOT* an "OpenCL software interpreter", there is no such thing) that causes the crash, while comment 5 indicates that it would be the OpenCL code that causes the crash.

Anyway, I can't reproduce any problem with a fresh build from the 5.1 branch. In the SAL_LOG output I don't see OpenCL being used after the document is loaded. Also, there doesn't seem to be much calculation going on in the sheet, certainly not 100 or more copies of a formula in a column, which is needed for OpenCL to be used. Or is there some hidden sheet that I don't see?
Comment 7 How can I remove my account? 2016-08-22 07:24:32 UTC
Petr, please respond to my questions. If no new information is available, this bug can be resolved as WORKSFORME.
Comment 8 Petr Pisar 2016-08-30 17:22:53 UTC
I'm sorry for the delay. I had long vacation and I do not carry all the credentials with myself.

I verified with official 5.1.5.2 release that it behaves exactly as the 5.1.5.1 prerelease.

I have Czech localization and I'm not sitting at the machine right know, so my description can be misleading. The OpenCL configuration tab has two options. If I remember correctly, since the fixed release 5.1.5.1 the lower option (Allow use of OpenCL) gets automatically disabled after the initial crash, while the upper one (Software interpreter) keeps enabled.

I'm fine with the current solution that disables the offending feature on first crash.
Comment 9 Aron Budea 2016-09-10 22:29:35 UTC
'Maybe a notice "OpenCL was disabled because it seems to crash LibreOffice" on next start would be great.'

Tor, what's your opinion on a similar notification?
Comment 10 Xisco Faulí 2016-09-10 23:10:03 UTC
Hi Petr,
Please, Don't change the status to ASSIGNED at least you intend to work on it.
Regards
Comment 11 Aron Budea 2016-09-10 23:45:52 UTC
Correct status is NEW, since Petr provided information. The only reason why I didn't close the bug report based on that is because of the question I wanted to ask Tor.
Comment 12 How can I remove my account? 2016-09-11 08:04:54 UTC
I still see no response to my question "there doesn't seem to be much calculation going on in the sheet, certainly not 100 or more copies of a formula in a column, which is needed for OpenCL to be used. Or is there some hidden sheet that I don't see?".

That was a honest question, it is entirely possible that I missed something when I looked at the spreadsheet. (I did not look at it again now.) Unless there are 100 or more copies of a formula below each other in a column, neither OpenCL nor the "software interpreter" will be used for that. ("copies" in a loose sense; formulas that refer cells positioned in an identical relative position count as copies.)
Comment 13 Petr Pisar 2016-09-11 18:09:17 UTC
I have no idea how many calculations or hidden sheets are in the the file. I'm not author of the file.
Comment 14 Buovjaga 2017-05-19 17:46:34 UTC
(correct status is unconfirmed, as this was never confirmed by someone else)

Petr: does this still happen with 5.3 and OpenCL enabled? If yes, does it perhaps produce a crash report (which can be allowed to be uploaded to the crash report server)?
Comment 15 Xisco Faulí 2017-05-20 09:54:17 UTC
Setting to NEEDINFO until we get reporter's feedback
Comment 16 Petr Pisar 2017-06-04 19:44:56 UTC
There is no change. After enabling OpenCL and restarting LibreOffice 5.3.3, it immediately crashes on startup. The crash dialog offers sending a report, but subsequent window with crash details review displays only "Error!" as the report content and does not have any button to continue with reporting. After dismissing the crash dialog, LibreOffice starts with auto-disabled OpenCL.
Comment 17 tommy27 2017-11-26 17:22:46 UTC
no crash under Win8.1 x61 using LibO 5.3.7

try upgrading to 5.3.7 or 5.4.3 and if it doesn't fix the issue consider resetting the user profile ( https://wiki.documentfoundation.org/UserProfile )

back to NEEDINFO
Comment 18 QA Administrators 2018-05-30 16:39:57 UTC Comment hidden (obsolete)
Comment 19 Petr Pisar 2018-06-03 18:31:21 UTC
I confirm that LibreOffice 5.4.7 as well as 6.0.4 do not crash. If I enable OpenCL, then after restarting LibreOffice the OpenCL will become disabled again without any crash.
Comment 20 Buovjaga 2018-06-03 19:02:07 UTC
Very nice, let's close.