diff options
author | Justin Luth <justin_luth@sil.org> | 2019-07-23 13:53:21 +0300 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2019-07-30 09:35:37 +0200 |
commit | c27987010c6fd67ae68ddc2a3d316b62bdd81d84 (patch) | |
tree | a0663be93a63c89f37ec5cef4e3042d5e82f8afc /include/vcl | |
parent | f111aba46cbb037bd82d7246bb0a111b87931f03 (diff) |
tdf#125609 c14: vcl button: don't modify style directly
LO 6.4/6.3 commit d35171456bc230efdaa9426da1398b2db7fa0df8
could cause a recursive loop. ImplInitStyle RETURNS the
suggested style, while ImplUncheckAll MODIFYS the
actual style. Since the call to set WB_TABSTOP
was already redundant in this case (and many others),
I provided an option to avoid that direct modification.
An example crashing document is attached to the bug report,
which will eventually find its way into automatic crash testing.
For this particular example document, it also requires
LO 6.4 commit 5cf057c3365a0feafc8f2e4f4a9e24d69a581999
in order to trigger a crash.
Change-Id: I26dddca27471dec10650f848f787363505b16c0f
Reviewed-on: https://gerrit.libreoffice.org/76186
Reviewed-by: Justin Luth <justin_luth@sil.org>
Tested-by: Justin Luth <justin_luth@sil.org>
(cherry picked from commit a9f4913f283d34c610c4b73c755fdc828857bfce)
Reviewed-on: https://gerrit.libreoffice.org/76574
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Diffstat (limited to 'include/vcl')
-rw-r--r-- | include/vcl/button.hxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/vcl/button.hxx b/include/vcl/button.hxx index 2a63e47a8c9b..4e782aa7a437 100644 --- a/include/vcl/button.hxx +++ b/include/vcl/button.hxx @@ -297,7 +297,7 @@ private: const Size& rImageSize, tools::Rectangle& rStateRect, tools::Rectangle& rMouseRect ); SAL_DLLPRIVATE void ImplDrawRadioButton(vcl::RenderContext& rRenderContext ); - SAL_DLLPRIVATE void ImplUncheckAllOther(); + SAL_DLLPRIVATE void ImplUncheckAllOther( const bool bSetStyle = true); SAL_DLLPRIVATE Size ImplGetRadioImageSize() const; SAL_DLLPRIVATE long ImplGetImageToTextDistance() const; |