diff options
| author | Jan-Marek Glogowski <glogow@fbihome.de> | 2021-06-30 11:01:06 +0000 |
|---|---|---|
| committer | Jan-Marek Glogowski <glogow@fbihome.de> | 2021-06-30 14:37:02 +0200 |
| commit | a876055b6f56e838114cce041d4c49aed43693f9 (patch) | |
| tree | c1cea4e0f72858abd7e506dab467197b68bb3762 | |
| parent | 05992ce5d03aeb2db8d4fc7a68053ebd9a9aa511 (diff) | |
Fix potential (but unlikely) overflow
warning: assuming signed overflow does not occur when assuming
that (X + c) >= X is always true [-Wstrict-overflow]
while( nPos < nLen )
~~~~~^~~~~~
Change-Id: I95e3bb4a131eaf4e82fbc74cc0bea92930b6afe9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118159
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
| -rw-r--r-- | xmloff/source/transform/TransformerBase.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xmloff/source/transform/TransformerBase.cxx b/xmloff/source/transform/TransformerBase.cxx index c937e00ca2c4..8814d19f26af 100644 --- a/xmloff/source/transform/TransformerBase.cxx +++ b/xmloff/source/transform/TransformerBase.cxx @@ -1315,12 +1315,12 @@ bool XMLTransformerBase::ConvertURIToOOo( OUString& rURI, { case '/': // a relative path segment within the package - nPos = nLen; // leave loop + nPos = nLen - 1; // leave loop break; case ':': // a schema bPackage = false; - nPos = nLen; // leave loop + nPos = nLen - 1; // leave loop break; default: // we don't care about any other characters |