Deleting any of the first 2 sheets with the comments causes a crash. Deleting the 3rd sheet ("test2") without the comments doesn't causes a crash (while tested with comments, used to also crash on delete). Tested with LO from 3.5.7 over 4.2.7 and 4.3.3 up to 4.4.0 master in Win64. Spreadsheet and debug log from WinDBG attached.
Created attachment 109301 [details] crash on delete sheet 1 or 2 with comments, sheet 3 can be deleted
Created attachment 109302 [details] crash on delete sheet - debug WinDGB.xt
I can confirm with Version: 4.3.4.0.0+ Build ID: e812aaaf31050c29a9499d3936cf07079b1a5f86 TinderBox: Linux-rpm_deb-x86_64@46-TDF, Branch:libreoffice-4-3, Time: 2014-11-10_10:51:25
confirmed in 4.3.3.2 thanks Timur. A fresh file doesn't have the problem. The attached one is a converted xls.
I have created a segfault with daily dbgutil repository versions 2014-11-12 and 2014-05-21 and with a parallel installation of 4.3.2.2 on Linux. However, I can delete the first two worksheets without any obvious problem with bibisect-43all repository versions oldest and latest. So, setting whiteboard notBibisectable. In all my experiments, I disabled macros.
Tested in 3.3.4.1 and not reproduced, and reproduced earliest in 3.4.0.1. Added "regression".
Markus Mohrhard committed a patch related to this issue. It has been pushed to "libreoffice-4-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d80a59a22dc993be8259e43f4d46e4471f1b7cc3&h=libreoffice-4-4 we need the SdrPage for deleting the notes, fdo#86166 It will be available in 4.4.0.0.beta3. 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.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=b45e1b58700c8303596a0aac9705d8c1577903b3 we need the SdrPage for deleting the notes, fdo#86166 It will be available in 4.5.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.
Very nice. Can it be backported to 4.3? If not late for 4.3.5?
(In reply to Timur from comment #9) > Very nice. Can it be backported to 4.3? If not late for 4.3.5? It has been pushed to 4-3 review together with the other commits. Stable branches just need to go through review in contrast to development branches.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "libreoffice-4-3": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8efdd9909cc46b4a00236f148f2f5c8349377c33&h=libreoffice-4-3 we need the SdrPage for deleting the notes, fdo#86166 It will be available in 4.3.6. 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.
Migrating Whiteboard tags to Keywords: (notBibisectable) [NinjaEdit]
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/9f0e19721bb598c75835cfa94f4158085f81288e tdf#86166: sc_uicalc: Add unittest It will be available in 7.4.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.