Created attachment 134425 [details] Example file This bug was filed from the crash reporting server and is br-c0c82391-5686-4477-be72-026f89b03482. ========================================= 1. Open the attached file 2. Copy the formula somewhere around C6 3. Close Calc (gray cross) 4. Open New calc document 5. Paste -> Crash
Repro. Version: 5.4.0.1 (x64) Build ID: 962a9c4e2f56d1dbdd354b1becda28edd471f4f2 CPU threads: 4; OS: Windows 6.19; UI render: GL; Locale: es-ES (es_ES); Calc: group
regression, no crash in Version 4.1.0.0.alpha0
commit 36d197c0646cd054145c7edcd7fb906b13c610a8 Author: Matthew Francis <mjay.francis@gmail.com> Date: Fri Sep 18 10:21:27 2015 +0800 source-hash-c55d52262ea1d5f869a9528fd051ee19e687f1cc commit c55d52262ea1d5f869a9528fd051ee19e687f1cc Author: Kohei Yoshida <kohei.yoshida@gmail.com> AuthorDate: Fri Jan 18 14:36:33 2013 -0500 Commit: Kohei Yoshida <kohei.yoshida@gmail.com> CommitDate: Fri Jan 18 14:41:42 2013 -0500 fdo#58988, fdo#58562: Populate draw clip document with data for charts. Without populating the clip document, copying a chart (or charts) into clipboard makes the charts lose all their referenced data, which causes the pasted chart to appear empty. Change-Id: I3675f76baed19b48cec403964c19df00725a044b
the same commit as in bug 108881 *** This bug has been marked as a duplicate of bug 108881 ***
Back to NEW, still reproducible with: Version: 6.0.0.0.alpha0+ Build ID: b8e26ebe2c37640212ef663a483cddf1639af742 CPU threads: 4; OS: Windows 6.19; UI render: default; TinderBox: Win-x86@39, Branch:master, Time: 2017-07-20_07:15:12 Locale: nl-NL (nl_NL); Calc: CL
Hi Dennis, I guess the fix for this is semilar to what you did for charts in bug 108881 but for formulas, isn't it ?
(In reply to Xisco Faulí from comment #6) > Hi Dennis, > I guess the fix for this is semilar to what you did for charts in bug 108881 > but for formulas, isn't it ? I think so, I will work on it next.
Only for the record, the crashing is not limited to Formula's. The crashing is happening for (nearly) all OLE-objects I tested in Calc (for example): * LibreofficeDev 6.0 Spreadsheet * LibreofficeDev 6.0 Drawing * LibreofficeDev 6.0 Presentation Steps to reproduce 1. Open Calc 2. Insert -> Object -> Ole-Object -> Insert a new Ole-object (for example spreadsheet) 3. Cut/Copy the Object 3. Close Calc (gray cross) 4. Open New calc document 5. Paste -> Crash ------------- I also noticed a crash when copying LibreOfficeDev 6.0 Chart OLE-object (cutting it is working fine) 1. Open Calc 2. Insert -> Object -> Ole-Object -> Insert LibreOfficeDec 6.0 Chart (create new) 3. Copy the Object Chart -> Crash
Submitted a patch on this for review @ https://gerrit.libreoffice.org/40278
*** Bug 109292 has been marked as a duplicate of this bug. ***
Dennis Francis committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=04461743d75f8cffb5906ab52d772089c44a7780 tdf#108887 : Move the code of ScDrawView::CheckOle()... It will be available in 6.0.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.
Thank you very much for this fix. Do you plan to backport it to 5.4 branch? Best regards. JBF
I will soon get the patch for 5-4 in gerrit after testing.
submitted @ https://gerrit.libreoffice.org/#/c/40579/ for libreoffice-5-4 branch
Dennis Francis committed a patch related to this issue. It has been pushed to "libreoffice-5-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=5b7a08939986da68b44d92099c7a9ac47351675c&h=libreoffice-5-4 tdf#108887 : Move the code of ScDrawView::CheckOle()... It will be available in 5.4.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.
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/6bda2e11c553b4032a33b95dd89e762db23e60f3 tdf#130614, tdf#108887: sc_uicalc: Add unittest It will be available in 7.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.