Bug 162492 - Changing relation of the position of an image, while displaying current selection's properties in Development tools, causes stack overflow
Summary: Changing relation of the position of an image, while displaying current selec...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
7.2.0.4 release
Hardware: All All
: medium critical
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: implementationError
Depends on:
Blocks: UNO-Object-Inspector
  Show dependency tree / graph
 
Reported: 2024-08-16 11:10 UTC by Mike Kaganski
Modified: 2024-12-01 13:19 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Kaganski 2024-08-16 11:10:14 UTC 
In Writer:

1. Tools->Development Tools;
2. In Development Tools panel, select "Current Selection", and open "Properties" tab;
3. Insert->Image, and insert some PNG;
4. Right-click the inserted image -> Properties;
5. In "Position", change the "Horizontal" from "Center" to "From left" -> OK.

=> Crash (stack overflow).

Does not happen with shaped (e.g., rectangles) instead of images.
Comment 1 Mike Kaganski 2024-08-16 11:16:46 UTC 
Adding a re-entry guard to DevelopmentToolDockingWindow::selectionChanged workarounds the problem, but that looks like a hack.

Caolan, Quikee: do you have an idea what is wrong here - likely in the object's property getter? I hope there is a better fix. Thank you!
Comment 2 Buovjaga 2024-10-02 12:15:26 UTC 
Repro. Also with master of linux-64-7.2 repo

Arch Linux 64-bit
Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: cb7e14bf370647a3fe618492ff79d0202c64c71f
CPU threads: 8; OS: Linux 6.10; UI render: default; VCL: kf6 (cairo+wayland)
Locale: fi-FI (fi_FI.UTF-8); UI: en-US
Calc: CL threaded
Built on 2 October 2024