Bug 74556

Summary: LibO CALC crashes when manipulating comments
Product: LibreOffice Reporter: Dominique Boutry <dominique.boutry3>
Component: CalcAssignee: Kohei Yoshida <kohei>
Status: RESOLVED FIXED    
Severity: critical CC: jbfaure, sophi, uwolf+libreoffice
Priority: highest Keywords: regression
Version: 4.2.0.4 release   
Hardware: Other   
OS: All   
Whiteboard: target:4.3.0 target:4.2.1
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 65675    
Attachments: backtrace of the crash

Description Dominique Boutry 2014-02-05 10:32:13 UTC
Steps to reproduce :
- open a new Spreadsheet,
- fill in a comment for cell A1, then another comment for cell A2,
- quickly press twice on the "Undo" icon.
CALC crashes.
Comment 1 sophie 2014-02-05 10:56:59 UTC
Confirmed with Version: 4.2.0.4
Build ID: 05dceb5d363845f2cf968344d7adab8dcfb2ba71 Ubuntu 13.10 x64 -
I will try to add a backtrace.
Change Plateform, importance, set to New - Sophie
Comment 2 sophie 2014-02-05 12:17:56 UTC
Created attachment 93442 [details]
backtrace of the crash

attaching the bt of the crash - Sophie
Comment 3 Jean-Baptiste Faure 2014-02-05 20:55:31 UTC
Crash reproducible with version 4.2.1.0.0+ (Build ID: 8cab77b3b26a5469d67235dd233d3996a1fa3677) on Ubuntu 13.10 x86-64.

Best regards. JBF
Comment 4 Björn Michaelsen 2014-02-06 00:12:28 UTC
not reproducible on 4.1.4 on Ubuntu 13.10 -- thus a regression.
Comment 5 Kohei Yoshida 2014-02-06 00:22:05 UTC
I'm looking into this.
Comment 6 Kohei Yoshida 2014-02-06 04:00:26 UTC
I think I fixed the originally reported problem, and also fixed a similar crasher during undo and redo after copy and paste.  I'll push my fix shortly.
Comment 7 Commit Notification 2014-02-06 04:03:12 UTC
Kohei Yoshida committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=771aadfce7d4b0dca678b0722c8f951371816082

fdo#74556: Write test for ReleaseNote().



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 8 Commit Notification 2014-02-06 04:03:25 UTC
Kohei Yoshida committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=fe5d604ecf6de4935c622e0e95efc085c4a3cbfd

fdo#74556: Have ReleaseNote() really release note rather than destroying it.



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 9 Commit Notification 2014-02-06 04:03:38 UTC
Kohei Yoshida committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=bc504b5adfaeeac0b910b89b0c98ae564f1ff5b8

fdo#74556: Correctly handle note captions life cycles.



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 10 Kohei Yoshida 2014-02-06 04:08:30 UTC
For 4.2 backport: https://gerrit.libreoffice.org/7887
Comment 11 Commit Notification 2014-02-06 14:40:06 UTC
Kohei Yoshida committed a patch related to this issue.
It has been pushed to "libreoffice-4-2":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=a0fe08c6862f5debaa375cc35c6bcec64f1008dc&h=libreoffice-4-2

fdo#74556: Correctly handle note captions life cycles.


It will be available in LibreOffice 4.2.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.
Comment 12 Commit Notification 2014-02-06 14:40:14 UTC
Kohei Yoshida committed a patch related to this issue.
It has been pushed to "libreoffice-4-2":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=a0fe08c6862f5debaa375cc35c6bcec64f1008dc&h=libreoffice-4-2

fdo#74556: Correctly handle note captions life cycles.


It will be available in LibreOffice 4.2.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.
Comment 13 Kohei Yoshida 2014-02-06 14:48:03 UTC
This is now in 4.2.x.
Comment 14 Jacques Guilleron 2014-02-10 09:48:05 UTC
*** Bug 74776 has been marked as a duplicate of this bug. ***