diff options
author | Justin Luth <justin_luth@sil.org> | 2016-12-24 12:00:21 +0300 |
---|---|---|
committer | Justin Luth <justin_luth@sil.org> | 2017-01-02 16:21:59 +0000 |
commit | 74ce6ebe51027ecd742b651d095c455417595908 (patch) | |
tree | 7bfb02f9488d38cbd480f628345e31a0340b83dc | |
parent | bed818c5d5e92a0b189f25e18495fc205d949128 (diff) |
tdf#81263 ww8par6: only use valid sal_uInt16 values for ULSpace
VerticalOrient position returns a signed SwTwips number,
but ULSpace uses unsigned int16, so negative numbers
were creating huge margins.
fixes commit 3755c87d0426a499d4755e82d8fbc77caa240764
Change-Id: I23daea9a913ef73efc42a65e0adfc393eaf4e775
Reviewed-on: https://gerrit.libreoffice.org/32400
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Justin Luth <justin_luth@sil.org>
(cherry picked from commit 604cf7024fce29143150ab77b2f14b59a45e61eb)
Reviewed-on: https://gerrit.libreoffice.org/32402
Tested-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
(cherry picked from commit 70b3dd697cb248fb56830db691269fe9e78c57fb)
Reviewed-on: https://gerrit.libreoffice.org/32648
-rw-r--r-- | sw/source/filter/ww8/ww8par6.cxx | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sw/source/filter/ww8/ww8par6.cxx b/sw/source/filter/ww8/ww8par6.cxx index 06d3ebca1808..0a8e60c4ed0b 100644 --- a/sw/source/filter/ww8/ww8par6.cxx +++ b/sw/source/filter/ww8/ww8par6.cxx @@ -2360,8 +2360,9 @@ bool SwWW8ImplReader::StartApo(const ApoTestResults &rApo, if (aFlySet.HasItem(RES_VERT_ORIENT, &pItem)) { const SwFormatVertOrient* pOrient = static_cast<const SwFormatVertOrient*>(pItem); - if (pOrient->GetPos() != 0) - pULSpaceItem->SetUpper(pOrient->GetPos()); + SwTwips nPos = pOrient->GetPos(); + if( 0 < nPos && nPos <= SAL_MAX_UINT16 ) + pULSpaceItem->SetUpper( sal_uInt16(nPos) ); } } } |