diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2019-01-25 14:47:02 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2019-01-25 16:57:34 +0100 |
commit | b9d5c32b687d2747e7c4fe91b8890942f4548a5c (patch) | |
tree | be3fe55e8026c5f762076267af168b2b6fbb9670 /sc/source/ui/view/output2.cxx | |
parent | 5b2f1243bd44b450da26c8344b3ee459f8e88f6c (diff) |
CppunitTest_desktop_lib: avoid use-after-free
The LOK callback registers a DesktopLOKTest instance, but depending on
timing, VCL may invoke that callback after that instance is already
deleted. The symptom is that all tests run fine, but the testsuite still
crashes at the end.
This can happen because DeInitVCL() indirectly calls
Scheduler::ProcessEventsToIdle(), which
desktop::CallbackFlushHandler::Invoke(), which may or may not invoke the
(now gone) callback, depending on timing. This typically doesn't happen
in debug builds, but it happens in release builds.
Change-Id: I00fc07046c46ad602a8b26c022ffcc916d211346
Reviewed-on: https://gerrit.libreoffice.org/66909
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'sc/source/ui/view/output2.cxx')
0 files changed, 0 insertions, 0 deletions