diff options
| author | Justin Luth <justin_luth@sil.org> | 2016-12-24 12:00:21 +0300 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2017-01-02 13:00:17 +0000 |
| commit | 70b3dd697cb248fb56830db691269fe9e78c57fb (patch) | |
| tree | ee64b02b33b240f91d9dea25e99fd1ab99fe73c6 | |
| parent | 6431e2bff67e81b4aff4e3b52e67903e1cd566f3 (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>
| -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 68b017e09b6d..0477795fe412 100644 --- a/sw/source/filter/ww8/ww8par6.cxx +++ b/sw/source/filter/ww8/ww8par6.cxx @@ -2355,8 +2355,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) ); } } } |