| Summary: | Crash while opening a document | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Elmar <roberg> |
| Component: | Calc | Assignee: | Caolán McNamara <caolan.mcnamara> |
| Status: | VERIFIED FIXED | ||
| Severity: | critical | CC: | aron.budea, dennisfrancis.in, rene.linder, serval2412, xiscofauli |
| Priority: | highest | Keywords: | bibisected, bisected, haveBacktrace, regression |
| Version: | 6.0.0.0.beta1 | ||
| Hardware: | All | ||
| OS: | Linux (All) | ||
| Whiteboard: | target:6.1.0 target:6.0.0.1 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 114159 | ||
| Attachments: |
doc crashes on opening in v6.1
gdb backtrace bt with debug symbols |
||
|
Description
Elmar
2017-12-08 13:37:33 UTC
Created attachment 138308 [details]
doc crashes on opening in v6.1
I then opened the document in v5.4 opened up a new spreadsheet in v6.1 Then I selected all the cells that are used Copy the cells to new doc save the file open the new saved file opens fine in v6.1 Regression introduced by: author Tor Lillqvist <tml@collabora.com> 2017-10-04 12:40:20 +0300 committer Dennis Francis <dennis.francis@collabora.co.uk> 2017-11-21 16:09:42 +0530 commit 5222910f969390c64c18866834d9af53e7c4c189 (patch) tree 51b5104ddb0a1bde71cccd99b0e97584e4b0e536 parent fd9d991ec02c3df178220031be6bd8b38c6c2c23 (diff) Make threaded calculation the default (when OpenCL is not used) Introduce a configuration setting to turn it off. For now, can also be turned off with the environment variable SC_NO_THREADED_CALCULATION, but that is probably not something we want to keep or guarantee staility of. (LO looks at way too many environment variables already.) Bisected with: bibisect-linux64-6.0 Adding Cc: to Tor Lillqvist Created attachment 138313 [details]
gdb backtrace
Created attachment 138332 [details]
bt with debug symbols
On pc Debian x86-64 with master sources updated today + enable-dbgutil, I also got a crash but with a different bt.
It's a stack overflow.
After having found that UseThreadedCalculationForFormulaGroups returned 1, I used this patch just for the test:
diff --git a/sc/source/core/data/formulacell.cxx b/sc/source/core/data/formulacell.cxx
index 955be80b4c59..9d4c3a97cd40 100644
--- a/sc/source/core/data/formulacell.cxx
+++ b/sc/source/core/data/formulacell.cxx
@@ -4344,7 +4344,7 @@ bool ScFormulaCell::InterpretFormulaGroup()
if (!bThreadingProhibited && !ScCalcConfig::isOpenCLEnabled() &&
pCode->GetVectorState() == FormulaVectorEnabledForThreading &&
- officecfg::Office::Calc::Formula::Calculation::UseThreadedCalculationForFormulaGroups::get())
+ officecfg::Office::Calc::Formula::Calculation::UseThreadedCalculationForFormulaGroups::get() && false)
{
// iterate over code in the formula ...
// ensure all input is pre-calculated -
I could open the file.
Now I don't know how to fix this.
Caolán McNamara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e1bd0dd9966c0d525477443bc54c1d2bd6f7d30f tdf#114342 move recursion guard to encompass threaded path too It will be available in 6.1.0. 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. backport to 6-0 in gerrit Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-6-0": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a63e8a7fa3b0f98501e8bc20a1adf6ca235a7dd1&h=libreoffice-6-0 tdf#114342 move recursion guard to encompass threaded path too It will be available in 6.0.0.1. 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. Verified in Version: 6.1.0.0.alpha0+ Build ID: 3af500580b1c82eabd60335c9ebc458a3f68850c CPU threads: 4; OS: Linux 4.10; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: group threaded *** Bug 114514 has been marked as a duplicate of this bug. *** |