Summary: | Crash after undo with experimental features enabled | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Telesto <telesto> |
Component: | Writer | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | lo_bugs, shivamhere247, xiscofauli |
Priority: | medium | Keywords: | haveBacktrace |
Version: | 7.1.0.0.alpha0+ | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=135179 | ||
Whiteboard: | target:7.1.0 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 134554 | ||
Attachments: |
Example file
Screencast backtrace on Linux |
Description
Telesto
2020-07-15 08:17:09 UTC
Created attachment 163043 [details]
Example file
I can't reproduce it in Version: 7.1.0.0.alpha0+ Build ID: d851a02df57ab378ed0cc6d9362516de09c3279c CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Calc: threaded I believe you use ESC or click somewhere in the document before undoing the chart insertion Created attachment 163079 [details]
Screencast
Was confused myself a bit, Style Inspector tab should be visible
Created attachment 163332 [details]
backtrace on Linux
The backtrace is from commit 0d45380c (2020-07-17) built and running
on debian-buster.
Terminal messages include
warn:chart2.tools:17729:17729:chart2/source/tools/WrappedPropertySet.cxx:137: found no inner property set to map to
warn:chart2:17729:17729:chart2/source/tools/LifeTime.cxx:117: This component is already disposed
Application Error
I am adding keyword haveBacktrace.
The code leading up to svx/source/sidebar/inspector/InspectorTextPanel.cxx line 63 reads 61: void InspectorTextPanel::updateEntries(const std::vector<TreeNode>& rStore) 62: { 63: mpListBoxStyles->freeze(); This function is frame #14 in the previously attached backtrace. In the second call to this function, mpListBoxStyles = std::unique_ptr<weld::TreeView> = {get() = 0x9999999999999999} and the call to freeze() results in an immediate segfault. I am adding Shivam Kumar Singh to cc. I think the Styles Inspector is your area, is it not? (In reply to Terrence Enger from comment #6) > I am adding Shivam Kumar Singh to cc. I think the Styles Inspector is > your area, is it not? Yes, Thank You. I confirm this with :- Version: 7.1.0.0.alpha0+ Build ID: 6e89b8aa81af8197cf9c826792e99938d582ae14 CPU threads: 8; OS: Linux 4.15; UI render: default; VCL: gtk3 Locale: en-IN (en_IN); UI: en-US Calc: threaded Likely the same problem as in tdf#135179. Inserting a chart changes the component temporarily; that likely re-creates all panels of sidebar, invalidating previously set link. Shivam Kumar Singh committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/3b8c893b487a3ee27bac710da74856225fb72950 tdf#135178 tdf#135179 tdf#134820 Issue in SetChgLnk in Inspector It will be available in 7.1.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. Verified in Version: 7.1.0.0.alpha0+ Build ID: 231e1e416c039d1f9724962a89cf0573a3db48a2 CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Calc: threaded @Shivam Kumar Singh, thanks for fixing this issue!! |