Bug 146614 - Crash swlo!SwFrame::SetInvalidVert+0x4397
Summary: Crash swlo!SwFrame::SetInvalidVert+0x4397
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.4.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: wantBacktrace
Depends on:
Blocks: Crash
  Show dependency treegraph
 
Reported: 2022-01-06 13:08 UTC by Telesto
Modified: 2024-05-02 18:19 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Example file (16.40 KB, application/vnd.oasis.opendocument.text)
2022-01-06 13:08 UTC, Telesto
Details
BT without symbols (14.11 KB, text/plain)
2022-01-06 13:09 UTC, Telesto
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2022-01-06 13:08:36 UTC
Description:
Crash swlo!SwFrame::SetInvalidVert+0x4397

Steps to Reproduce:
1. Open attached file (slightly modified version of attachment 143995 [details] bug 119126)
2. Press CTRL+A
3. Press CTRL+C
4. Go to second page.. and click somewhere at green highlighting (different table distribution)
5. Scroll up. Place cursor somewhere at yellow marking 
6. CTRL+V (wait)
7. CTRL+Z -> crash

Actual Results:
Crash

Expected Results:
No crash


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.4.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 1bb0e177124d5d6661b72df6c7d848fb23639652
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL
Comment 1 Telesto 2022-01-06 13:08:53 UTC
Created attachment 177347 [details]
Example file
Comment 2 Telesto 2022-01-06 13:09:11 UTC
Created attachment 177348 [details]
BT without symbols
Comment 3 Telesto 2022-01-06 15:50:34 UTC
Alternative
1. Open attached file
2. Place the cursor in the top left table cell
3. CTRL+A
4. F12 (toggle numbering on)
5. CTRL+Z

Does throw different warning SAL warning, prior crash, but well BT stack nearly identical
Comment 4 Julien Nabet 2022-01-06 17:45:44 UTC
On pc Debian x86-64 with master sources updated today, I don't reproduce the crash with first mechanism.
However, I don't have any part in green anymore after Ctrl-Z

I noticed different kinds of warns:
warn:legacy.osl:55675:55675:sw/source/core/layout/tabfrm.cxx:2710: debug assertion: <SwTabFrame::MakeAll()> - format of table lowers suppressed by fix i44910
warn:legacy.osl:55675:55675:sw/source/core/layout/flowfrm.cxx:2563: <SwFlowFrame::MoveBwd(..)> - layout loop control for layout action <Move Backward> applied
warn:sw.layout:55675:55675:sw/source/core/layout/tabfrm.cxx:900: Cannot remove in-use Follow Flow Line
warn:legacy.osl:55675:55675:sw/source/core/layout/tabfrm.cxx:893: There should be a flowline in the follow
warn:legacy.osl:55675:55675:sw/source/core/layout/tabfrm.cxx:1368: Joining follow flow line

I suppose there are already similar bugtrackers with this kind of doc (complicated, at least for LO, layout with tabs) which crashes so I suppose it needs someone to care about these before creating new bugtrackers.

It also needs to speed up the layout mechanism, it's really slow.
Comment 5 Telesto 2022-01-06 18:12:00 UTC
(In reply to Julien Nabet from comment #4)
> On pc Debian x86-64 with master sources updated today, I don't reproduce the
> crash with first mechanism.
> However, I don't have any part in green anymore after Ctrl-Z
> 
> I noticed different kinds of warns:
> warn:legacy.osl:55675:55675:sw/source/core/layout/tabfrm.cxx:2710: debug
> assertion: <SwTabFrame::MakeAll()> - format of table lowers suppressed by
> fix i44910
> warn:legacy.osl:55675:55675:sw/source/core/layout/flowfrm.cxx:2563:
> <SwFlowFrame::MoveBwd(..)> - layout loop control for layout action <Move
> Backward> applied
> warn:sw.layout:55675:55675:sw/source/core/layout/tabfrm.cxx:900: Cannot
> remove in-use Follow Flow Line
> warn:legacy.osl:55675:55675:sw/source/core/layout/tabfrm.cxx:893: There
> should be a flowline in the follow
> warn:legacy.osl:55675:55675:sw/source/core/layout/tabfrm.cxx:1368: Joining
> follow flow line
> 
> I suppose there are already similar bugtrackers with this kind of doc
> (complicated, at least for LO, layout with tabs) which crashes so I suppose
> it needs someone to care about these before creating new bugtrackers.
> 
> It also needs to speed up the layout mechanism, it's really slow.

Well those warnings are around for a while.. but mostly doesn't end up with crash, as far I recall.
Comment 6 Telesto 2022-01-09 09:52:35 UTC
Confirming based on BT at bug 146621
Comment 7 Stéphane Guillou (stragu) 2024-05-02 15:12:26 UTC
Comment 0 steps on Linux:
- 7.4.0.3: no crash, but poor performance
- 7.6.6.3: frozen
- 24.2.2.2: not frozen anymore, although poor performance
- 24.8 alpha0+: no freeze, no performance issue / loop

Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: ce454f382d0d005dd3de021c7820be3ffa0bb582
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

On Windows:
- 7.5.0.3: no crash, but poor performance
- 24.8 alpha0+: no freeze, no performance issue / loop

Closing as "works for me", but please confirm it's good on your end too, Telesto.
Comment 8 Telesto 2024-05-02 18:19:18 UTC
(In reply to Stéphane Guillou (stragu) from comment #7)
> Closing as "works for me", but please confirm it's good on your end too,
> Telesto.

I'm unable to repro the exact steps. Performance is reasonable. However the same/similar issue is still present in multipage view (bug 160913)

Didn't check for warnings a noted in comment 4. I expect those to be present. I even assume that all bugs throwing "format of table lowers suppressed by fix i44910" having the same root cause (as noted in comment 4)