diff options
author | László Németh <nemeth@numbertext.org> | 2020-04-09 17:20:18 +0200 |
---|---|---|
committer | László Németh <nemeth@numbertext.org> | 2020-04-10 15:38:52 +0200 |
commit | 1350832be533ce6627607b1aaabd2b3565e6e7b3 (patch) | |
tree | b702b8f454964028de6f3e10d4bc58df31d78d83 /writerfilter | |
parent | 894c0806328627aadd1e92c8601a5858a3ea627f (diff) |
tdf#132001 DOCX import: fix crash: redline in floating table
Copying redlines to frame text was failed in tables inside
frames. Skip these redlines temporarily.
Regression from commit e8bae67b3dbcc90ace8264b6b1aefaf0ce459aba
(tdf#125894: DOCX: import tracked changes in frames).
Change-Id: I4f3ca2e95fb2e7637f8cf8dca1088a7727bcf98d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91985
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
Diffstat (limited to 'writerfilter')
-rw-r--r-- | writerfilter/source/dmapper/DomainMapper_Impl.cxx | 20 |
1 files changed, 16 insertions, 4 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx index a5ba85300ea6..4a7c8f7e4995 100644 --- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx +++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx @@ -6439,10 +6439,19 @@ void DomainMapper_Impl::ExecuteFrameConversion() uno::Reference< text::XTextRange > xRange; aFramedRedlines[i] >>= xRange; uno::Reference<text::XTextCursor> xRangeCursor = GetTopTextAppend()->createTextCursorByRange( xRange ); - sal_Int32 nLen = xRange->getString().getLength(); - redLen.push_back(nLen); - xRangeCursor->gotoRange(m_xFrameStartRange, true); - redPos.push_back(xRangeCursor->getString().getLength() - nLen); + if (xRangeCursor.is()) + { + sal_Int32 nLen = xRange->getString().getLength(); + redLen.push_back(nLen); + xRangeCursor->gotoRange(m_xFrameStartRange, true); + redPos.push_back(xRangeCursor->getString().getLength() - nLen); + } + else + { + // failed createTextCursorByRange(), for example, table inside the frame + redLen.push_back(-1); + redPos.push_back(-1); + } } const uno::Reference< text::XTextContent >& xTextContent = xTextAppendAndConvert->convertToTextFrame( @@ -6455,6 +6464,9 @@ void DomainMapper_Impl::ExecuteFrameConversion() { OUString sType; beans::PropertyValues aRedlineProperties( 3 ); + // skip failed createTextCursorByRange() + if (redPos[i/3] == -1) + continue; aFramedRedlines[i+1] >>= sType; aFramedRedlines[i+2] >>= aRedlineProperties; uno::Reference< text::XTextFrame > xFrame( xTextContent, uno::UNO_QUERY_THROW ); |