Bug 147211 - Link to External Data with an update interval causes Calc to hang
Summary: Link to External Data with an update interval causes Calc to hang
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.3.0.3 release
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks:
 
Reported: 2022-02-05 08:14 UTC by Howard Feldman
Modified: 2022-06-28 13:05 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Howard Feldman 2022-02-05 08:14:33 UTC
Description:
Using a link to an external file with an update interval causes Calc to hang and nothing can make it respond to any input or menu selection.  The only option is to close the window and force a quit.

Steps to Reproduce:
1.open Calc and select any cell (e.g. A1)
2.select from menu Sheet -> Link to External Data
3.choose a file that contains some data.  important: check the box that activates the update every XX seconds and choose an update time (for testing use a short time e.g. 5 seconds)
4.Press OK
5.Observe that spreadsheet will be unresponsive after the update time expires.  However, the user may use the pull down menus but they won't do anything.

Actual Results:
The spreadsheet was unresponsive

Expected Results:
expected result is that the spreadsheet should allow the user all functionality and when the update time expires it should reread the file and update the cell


Reproducible: Always


User Profile Reset: Yes


OpenGL enabled: Yes

Additional Info:
Version: 7.3.0.3 / LibreOffice Community
Build ID: 30(Build:3)
CPU threads: 8; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Ubuntu package version: 1:7.3.0~rc3-0ubuntu0.20.04.1~lo1
Calc: threaded
Comment 1 raal 2022-02-06 08:24:37 UTC
I can confirm with Version: 7.4.0.0.alpha0+ / LibreOffice Community
Build ID: d5f015185240a7bddfed7ddf10d6b5426e35fb72
CPU threads: 4; OS: Linux 5.11; UI render: default; VCL: gtk3
Locale: cs-CZ (cs_CZ.UTF-8); UI: en-US
Calc: threaded Jumbo


works in Version: 7.1.7.2 / LibreOffice Community
Build ID: 10(Build:2)
CPU threads: 4; OS: Linux 5.11; UI render: default; VCL: gtk3
Locale: cs-CZ (cs_CZ.UTF-8); UI: cs-CZ
Ubuntu package version: 1:7.1.7-0ubuntu0.21.04.1
Calc: threaded
Comment 2 raal 2022-02-06 09:07:02 UTC
This seems to have begun at the below commit.
Adding Cc: to Noel Grandin; Could you possibly take a look at this one?
Thanks
 4b6c08c0459142c1ea3966e88a2f692383127a47 is the first bad commit
commit 4b6c08c0459142c1ea3966e88a2f692383127a47
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Mon Aug 2 12:35:01 2021 +0200

    source 287680683ca266f1fb4f447ac9bdaf76669d559d

https://git.libreoffice.org/core/+/287680683ca266f1fb4f447ac9bdaf76669d559d
   osl::Mutex->std::mutex in ScRefreshTimer
Comment 3 Noel Grandin 2022-06-28 13:05:41 UTC
This was fixed by
    commit 4361ee2585e616cb3c504eb719deca4076de78da
    Author: Eike Rathke <erack@redhat.com>
    Date:   Sat Feb 26 01:57:51 2022 +0100
    Resolves: tdf#147448 ScRefreshTimerControl mutex must be std::recursive_mutex