Bug 143387 - CRASH: undoing paste ( debug build )
Summary: CRASH: undoing paste ( debug build )
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.3.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:24.8.0
Keywords: haveBacktrace
: 144716 149276 (view as bug list)
Depends on:
Blocks: Crash-Assert
  Show dependency treegraph
 
Reported: 2021-07-15 13:20 UTC by Xisco Faulí
Modified: 2024-04-30 19:14 UTC (History)
7 users (show)

See Also:
Crash report or crash signature:


Attachments
sample file (9.64 KB, application/vnd.oasis.opendocument.text)
2021-07-15 13:20 UTC, Xisco Faulí
Details
gdb backtrace (37.37 KB, text/plain)
2021-07-15 13:22 UTC, Xisco Faulí
Details
gdb bt with gen rendering (9.57 KB, text/plain)
2021-07-21 18:20 UTC, Julien Nabet
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Xisco Faulí 2021-07-15 13:20:09 UTC
Created attachment 173598 [details]
sample file

I found this issue while creating a unittest for bug 143320

Steps to reproduce:
1. Open attached document
2. Select all
3. Copy
4. Open new document
5. Paste
6. Undo

-> Crash

Reproduced in

Version: 7.3.0.0.alpha0+ / LibreOffice Community
Build ID: 717dc8e3575a18e1e18dc446031e6db4e60a873d
CPU threads: 4; OS: Linux 5.7; UI render: default; VCL: x11
Locale: en-US (en_US.UTF-8); UI: en-US
TinderBox: Linux-rpm_deb-x86_64@86-TDF-dbg, Branch:master, Time: 2021-07-15_06:02:12
Calc: threaded
Comment 1 Xisco Faulí 2021-07-15 13:22:30 UTC
Created attachment 173599 [details]
gdb backtrace
Comment 2 Telesto 2021-07-15 14:19:29 UTC
So the reverted commit at bug 143320 may not be the true cause of the crash?
Comment 3 Aron Budea 2021-07-21 18:11:23 UTC
Confirmed in a fresh debug build.
This is the assert:

sw/source/core/txtnode/ndtxt.cxx:5270: void SwTextNode::TriggerNodeUpdate(const sw::LegacyModifyHint&): Assertion `dynamic_cast<SwTextFormatColl const*>(static_cast<const SwFormatChg*>(pOldValue)->pChangedFormat)' failed.
Comment 4 Julien Nabet 2021-07-21 18:20:49 UTC
Created attachment 173762 [details]
gdb bt with gen rendering

On pc Debian x86-64 with master sources updated today, I could reproduce this.
Comment 5 Julien Nabet 2021-07-21 18:21:31 UTC
Michael: thought you might be interested in this one since it concerns Writer + undo.
Comment 6 Telesto 2021-07-21 18:39:23 UTC
(In reply to Aron Budea from comment #3)
sw::LegacyModifyHint ... isn't Bjoern working in that area?
Comment 7 Xisco Faulí 2021-11-29 08:33:39 UTC
*** Bug 144716 has been marked as a duplicate of this bug. ***
Comment 8 Xisco Faulí 2021-11-29 09:42:00 UTC
Unittest added in https://gerrit.libreoffice.org/c/core/+/118908 but still failing
Comment 9 Xisco Faulí 2022-05-27 08:19:48 UTC
*** Bug 149276 has been marked as a duplicate of this bug. ***
Comment 10 Stéphane Guillou (stragu) 2024-04-30 02:31:02 UTC
No repro in this debug build:

Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 0957ee9f5d379c80fca4027c187b471118d0490d
CPU threads: 8; OS: Linux 6.5; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: CL threaded

Xisco/Julien/Aron, can you double-check?
Comment 11 Julien Nabet 2024-04-30 08:56:39 UTC
On pc Debian x86-64 with master sources updated today, I don't reproduce this anymore.
Comment 12 Xisco Faulí 2024-04-30 09:17:03 UTC
yes, it doesn't crash anymore. In fact the unittest <https://gerrit.libreoffice.org/c/core/+/118908> no longer fails either. Closing as RESOLVED WORKSFORME
Comment 13 Buovjaga 2024-04-30 09:22:40 UTC
No repro with steps from bug 144716, so let's close.

Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 63e0680e29ea8d414ca23d10304d5564fe2aa99c
CPU threads: 8; OS: Linux 6.8; UI render: default; VCL: kf6 (cairo+wayland)
Locale: fi-FI (fi_FI.UTF-8); UI: en-US
Calc: threaded
Comment 14 Commit Notification 2024-04-30 19:14:25 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/17b2f3d96fb21dcfc2fb8e54ca9670a8c58840f1

tdf#143320, tdf#143387: sw_uiwriter4: Add unittest

It will be available in 24.8.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.