diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2017-11-04 21:01:39 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2017-11-05 17:55:21 +0100 |
| commit | b97d210bce5f05c23330b7a6fb0602f8f74da5a0 (patch) | |
| tree | 063672c14b18b1832d9fd6ce268d9ed26e7c962f /xmloff | |
| parent | 92ca905bc73a829106975dac0d1c53111667ca2a (diff) | |
ofz#4057 Integer-overflow
Change-Id: I6a7ef3ca4c26ce4965deeea27ae47a03169bcf12
Reviewed-on: https://gerrit.libreoffice.org/44320
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'xmloff')
| -rw-r--r-- | xmloff/source/style/shadwhdl.cxx | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/xmloff/source/style/shadwhdl.cxx b/xmloff/source/style/shadwhdl.cxx index 20d12f7571f4..0e14ed87fbd1 100644 --- a/xmloff/source/style/shadwhdl.cxx +++ b/xmloff/source/style/shadwhdl.cxx @@ -96,8 +96,10 @@ bool XMLShadowPropHdl::importXML( const OUString& rStrImpValue, uno::Any& rValue aShadow.Location = table::ShadowLocation_BOTTOM_RIGHT; } - if( nX < 0 ) nX *= -1; - if( nY < 0 ) nY *= -1; + if (nX < 0) + nX = o3tl::saturating_toggle_sign(nX); + if (nY < 0) + nY = o3tl::saturating_toggle_sign(nY); sal_Int32 nWidth; bRet = !o3tl::checked_add(nX, nY, nWidth); |