Description: Crash swlo!SwAnchoredDrawObject::IsValidPos Steps to Reproduce: 1. Open attachment 151991 [details] 2. Enable tracking changes toolbar 3. Start Recording changes 4. CTRL+A & CTRL+X 5. Hide Track changes 6. Unhide Track changes -> Crash Actual Results: Crash Expected Results: No crash Reproducible: Always User Profile Reset: No Additional Info: Version: 6.4.0.0.alpha0+ (x64) Build ID: 95462a02a3aee1e3e7f9aa8fc50ba25fee3fa592 CPU threads: 4; OS: Windows 6.3; UI render: default; VCL: win; TinderBox: Win-x86_64@42, Branch:master, Time: 2019-06-03_07:09:38 Locale: nl-NL (nl_NL); UI-Language: en-US Calc: CL
Created attachment 151993 [details] BT without symbols
I don't know if it's the fact to use enable-dbgutil but with a Ryzen 2600 + 32GB, it's too long to try to reproduce this. Would it be possible to have a minimal file instead of a 305 pages file?
Created attachment 151997 [details] Example file 43 pages.. more trimming is probably possible, but i'm lazy
(In reply to Telesto from comment #3) > Created attachment 151997 [details] > Example file > > 43 pages.. more trimming is probably possible, but i'm lazy Sorry 86 pages
I can't confirm it with Version: 6.3.0.0.beta1 (x64) Build ID: a187af327633f5f00363be5131bd21a13e0f1a7b CPU threads: 4; OS: Windows 10.0; UI render: default; VCL: win; Locale: en-US (de_DE); UI-Language: en-GB Calc: threaded
Created attachment 152018 [details] Screencast
Reproduced in Version: 6.4.0.0.alpha0+ Build ID: 6b911ae9eb9484bebbdc1323210020486f5ef33f CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US Calc: threaded
Also reproduced in Versión: 6.2.4.2 Id. de compilación: 2412653d852ce75f65fbfa83fb7e7b669a126d64 Subprocs. CPU: 1; SO: Windows 6.1; Repres. IU: predet.; VCL: win; Configuración regional: es-ES (es_ES); Idioma de IU: es-ES Calc: threaded
Reproduced in Version: 5.0.0.0.alpha1+ Build ID: 0db96caf0fcce09b87621c11b584a6d81cc7df86 Locale: ca-ES (ca_ES.UTF-8) but not in Version: 4.3.0.0.alpha1+ Build ID: c15927f20d4727c3b8de68497b6949e72f9e6e9e
Actually this is a more recent regression, it doesn't crash in Version: 6.0.0.0.alpha1+ Build ID: 6eeac3539ea4cac32d126c5e24141f262eb5a4d9 CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: group threaded
LibreOffice started to crash at step 6 in https://cgit.freedesktop.org/libreoffice/core/commit/?id=f609eba8b979620dc8f1c4a47d710360f8aae9e6, before that, it crashes hiding track changes ( step 5 ). I'll bisect that as well but I don't have time right now for that... Adding Cc: to Michael Stahl
Created attachment 152039 [details] bt with debug symbols (gtk3) On pc Debian x86-64 with master sources updated today, I could reproduce this.
Michael Stahl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/dd489bc01adc22fc5015ea56b61d66104af184a8%5E%21 tdf#125754 sw_redlinehide: avoid recursive layout in SetHideRedlines() It will be available in 6.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.
fixed on master
Michael Stahl committed a patch related to this issue. It has been pushed to "libreoffice-6-3": https://git.libreoffice.org/core/+/f75ba02bb18cc10e4ca8ca766f82ab37957189d9%5E%21 tdf#125754 sw_redlinehide: avoid recursive layout in SetHideRedlines() It will be available in 6.3.0.1. 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.
Michael Stahl committed a patch related to this issue. It has been pushed to "libreoffice-6-2": https://git.libreoffice.org/core/+/83263c0428410bd8aa2344aa83e0388fdd3f7342%5E%21 tdf#125754 sw_redlinehide: avoid recursive layout in SetHideRedlines() It will be available in 6.2.6. 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.
Verified in Version: 6.4.0.0.alpha0+ Build ID: 52800731baf9fcb88e54d60de5585640c8f97f7b CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US Calc: threaded @Michael Stahl, thank for fixing this issue!!
Michael Stahl committed a patch related to this issue. It has been pushed to "libreoffice-6-2-5": https://git.libreoffice.org/core/+/bc95445e08e07a5d23e2c833b47a31ba0a217517%5E%21 tdf#125754 sw_redlinehide: avoid recursive layout in SetHideRedlines() It will be available in 6.2.5. 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.
Let's simplify targets a bit.