xmloff/source/style/XMLClipPropertyHandler.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit 2df2cf1b6e77aecfc4a70af52bc7d19b95034e32 Author: Caolán McNamara <[email protected]> AuthorDate: Sat Oct 11 20:51:06 2025 +0100 Commit: Caolán McNamara <[email protected]> CommitDate: Sun Oct 12 16:47:15 2025 +0200 ofz: attempt to get abs of -2147483648 What is actually reported is Use-of-uninitialized-value but I don't see a problem of an uninitialized value. But I do see that nVal is -2147483648 so presumably that's the actual problem. Change-Id: I9e5bdcb0e8fdfc6274888c1d7f1c94a65a8613d6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/192238 Tested-by: Jenkins Reviewed-by: Caolán McNamara <[email protected]> diff --git a/xmloff/source/style/XMLClipPropertyHandler.cxx b/xmloff/source/style/XMLClipPropertyHandler.cxx index 7a4d01cd99df..440ebdc4d1e4 100644 --- a/xmloff/source/style/XMLClipPropertyHandler.cxx +++ b/xmloff/source/style/XMLClipPropertyHandler.cxx @@ -82,7 +82,7 @@ bool XMLClipPropertyHandler::importXML( const OUString& rStrImpValue, uno::Any& break; // fdo#80009 such nonsense could be written via WW8 import fdo#77454 - if (abs(nVal) > 400000) + if (nVal == SAL_MIN_INT32 || abs(nVal) > 400000) { SAL_INFO("xmloff.style", "ignoring excessive clip " << OUString(aToken)); nVal = 0;
