diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2020-02-11 10:31:18 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2020-02-12 09:13:02 +0100 |
| commit | d84eaea950f276e7ba0d155cdbed056bcc815255 (patch) | |
| tree | dc8a4fd84f1faff7d2b7b5ad7142f9c3a6cde911 /vcl | |
| parent | c2a5346b19c57f93f210b76c15cdba64f6871203 (diff) | |
tdf#130555: Prevent negative aSelection.Min()
...which would violate the the preconditions of the later call to maText.remove.
When BreakIteratorImpl::previousWord
(i18npool/source/breakiterator/breakiteratorImpl.cxx) is called to e.g. move
back over a single space at the start of the text, at least for an en-US locale
it will fall through to the call to BreakIterator_Unicode::previousWord
(i18npool/source/breakiterator/breakiterator_unicode.cxx) at the bottom of the
function. That in turn calls icu::BreakIterator::preceding, which is documented
(workdir/UnpackedTarball/icu/source/common/unicode/brkiter.h) to return
icu::BreakIterator::DONE (i.e., -1, see
workdir/UnpackedTarball/icu/source/common/unicode/brkiter.h) in that case, which
causes BreakIterator_Unicode::previousWord to return a Boundary with startPos ==
endPos == -1.
The documentation of UNO method css.i18n.XBreakIterator::previousWord
(offapi/com/sun/star/i18n/XBreakIterator.idl) is silent about the expected
return value in such a case. But lets assume that returning such a [-1..-1]
Boundary is as intended, and locally address this case in Edit::ImplDelete,
making aSelection start at the start of the text.
Change-Id: I40e17ba602088e72aa6962cb41dd8e1cdf6e2561
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88431
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit e8f26dc13b65b1a05d948d9c95110c86315e8f20)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88402
Diffstat (limited to 'vcl')
| -rw-r--r-- | vcl/source/control/edit.cxx | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/vcl/source/control/edit.cxx b/vcl/source/control/edit.cxx index d76b4464ffcf..8cb0a06594fa 100644 --- a/vcl/source/control/edit.cxx +++ b/vcl/source/control/edit.cxx @@ -61,6 +61,7 @@ #include <officecfg/Office/Common.hxx> +#include <algorithm> #include <memory> using namespace ::com::sun::star; @@ -687,10 +688,14 @@ void Edit::ImplDelete( const Selection& rSelection, sal_uInt8 nDirection, sal_uI { i18n::Boundary aBoundary = xBI->getWordBoundary( maText.toString(), aSelection.Min(), GetSettings().GetLanguageTag().getLocale(), i18n::WordType::ANYWORD_IGNOREWHITESPACES, true ); - if ( aBoundary.startPos == aSelection.Min() ) + auto startPos = aBoundary.startPos; + if ( startPos == aSelection.Min() ) + { aBoundary = xBI->previousWord( maText.toString(), aSelection.Min(), GetSettings().GetLanguageTag().getLocale(), i18n::WordType::ANYWORD_IGNOREWHITESPACES ); - aSelection.Min() = aBoundary.startPos; + startPos = std::max(aBoundary.startPos, sal_Int32(0)); + } + aSelection.Min() = startPos; } else if ( nMode == EDIT_DELMODE_RESTOFCONTENT ) { |