diff options
author | Michael Stahl <michael.stahl@allotropia.de> | 2023-10-18 15:17:29 +0200 |
---|---|---|
committer | Michael Stahl <michael.stahl@allotropia.de> | 2023-10-18 20:18:03 +0200 |
commit | adef72f8411a49920a75d3ceee7c1dc61ca0a470 (patch) | |
tree | e796f74673d1cf127f2b1de4c12fc0a9c4475a8c /embedserv | |
parent | 51678d9cf57085bac3c7eba998e0a6d364d36cb2 (diff) |
sw: fix stack overflow on kde45196-1.html
Previously, the shell cursor was put onto a SwGrfNode in the top left
corner, which is invalid, but didn't crash immediately.
With commit b20ca8d951e8205c8b963c6b7407f984053b4094 the cursor is
instead put onto the first SwTextNode in the body, which happens to be
in a table, and the entire table is hidden; this causes an infinite
recursion in SwCursorShell::UpdateCursorPos() because
GetModelPositionForViewPoint() no longer moves it to SwGrfNode.
So try to move the cursor out of a hidden node, which requires an
additional change in SwCursor::IsSelOvr() to allow moving from a node
without a frame to another node without a frame.
Change-Id: Ia33c7b558755f7e8b65ea3ff2c46aea20be577dd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/158119
Tested-by: Michael Stahl <michael.stahl@allotropia.de>
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Diffstat (limited to 'embedserv')
0 files changed, 0 insertions, 0 deletions