From 2387c53dba185b45b0e34beae2a8ddaa315bb689 Mon Sep 17 00:00:00 2001 From: Michael Stahl Date: Sat, 28 Jun 2014 00:09:43 +0200 Subject: fdo#80009: xmloff: ignore excessive fo:clip values These cause problems in drawinglayer, and nobody needs a 5km clip. Change-Id: Ic485250413194adbcd0ccf855f12e89e532d7120 (cherry picked from commit 924a28a7b1dc2f89e6940630057557f5f03494df) Reviewed-on: https://gerrit.libreoffice.org/9946 Reviewed-by: Miklos Vajna Tested-by: Miklos Vajna --- xmloff/source/style/XMLClipPropertyHandler.cxx | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/xmloff/source/style/XMLClipPropertyHandler.cxx b/xmloff/source/style/XMLClipPropertyHandler.cxx index af089d7db656..2090462e4460 100644 --- a/xmloff/source/style/XMLClipPropertyHandler.cxx +++ b/xmloff/source/style/XMLClipPropertyHandler.cxx @@ -81,6 +81,13 @@ bool XMLClipPropertyHandler::importXML( const OUString& rStrImpValue, uno::Any& !rUnitConverter.convertMeasureToCore( nVal, aToken ) ) break; + // fdo#80009 such nonsense could be written via WW8 import fdo#77454 + if (abs(nVal) > 400000) + { + SAL_INFO("xmloff.style", "ignoring excessive clip " << aToken); + nVal = 0; + } + switch( nPos ) { case 0: aCrop.Top = nVal; break; -- cgit v1.2.3