diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2021-10-02 20:34:56 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2021-10-03 15:35:46 +0200 |
| commit | e69d4963c3a7b71b4dc42e2f30b5cbdc4155d594 (patch) | |
| tree | 84b01dfd53d9524e8920118489cb74ad4c64836c /emfio | |
| parent | e7a7cfa029f8a6979179c5cf394d7eea80982ca8 (diff) | |
ofz#39528 Integer-overflow
Change-Id: If50af8538e31e4fb1ee1486ec2b91963c094e7e6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123007
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'emfio')
| -rw-r--r-- | emfio/source/reader/emfreader.cxx | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/emfio/source/reader/emfreader.cxx b/emfio/source/reader/emfreader.cxx index b75d16c5759f..af79b261d1e7 100644 --- a/emfio/source/reader/emfreader.cxx +++ b/emfio/source/reader/emfreader.cxx @@ -1967,26 +1967,26 @@ namespace emfio } } - sal_Int32 nDx = 0, nDy = 0; + pDXAry[i] = 0; + if (nOptions & ETO_PDY) + { + pDYAry[i] = 0; + } + while (nDxCount--) { sal_Int32 nDxTmp = 0; mpInputStream->ReadInt32(nDxTmp); - nDx += nDxTmp; + pDXAry[i] += nDxTmp; if (nOptions & ETO_PDY) { sal_Int32 nDyTmp = 0; mpInputStream->ReadInt32(nDyTmp); - nDy += nDyTmp; + pDYAry[i] += nDyTmp; } } - SAL_INFO("emfio", "\t\t\tSpacing " << i << ": " << nDx); - pDXAry[i] = nDx; - if (nOptions & ETO_PDY) - { - pDYAry[i] = nDy; - } + SAL_INFO("emfio", "\t\t\tSpacing " << i << ": " << pDXAry[i]); } } if ( nOptions & ETO_CLIPPED ) |