diff options
| author | Michael Stahl <mstahl@redhat.com> | 2014-05-27 13:27:37 +0200 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2014-05-27 13:34:23 +0200 |
| commit | 5f80a1059b19355753ce7950f7f900f51b13c2f0 (patch) | |
| tree | cd1e8310b4f49b002792fcc7784f6c0b9068b7bf | |
| parent | c9a0c51d74d1169e5ce6eef1edd7499dc7e998d0 (diff) | |
fdo#77454: sw: ww8 import: add heuristic to ignore mangled crop values
Older versions of OOo/LO would mangle negative crop values on round-trip
of WW8, which now (since the drawing-layer does not ignore them) causes
the images to be rendered invisible; probably large factors don't make
sense anyway, so ignore the crop if it's 50x the image size, which
happens to work for the bugdoc.
Change-Id: I9f36d37e3be27234554bc91e80bfe719b1ce86af
(cherry picked from commit 7e91dd0a3885ac561aee9cf7907632677b9886c1)
| -rw-r--r-- | sw/source/filter/ww8/ww8graf.cxx | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/sw/source/filter/ww8/ww8graf.cxx b/sw/source/filter/ww8/ww8graf.cxx index ec4bf384228c..e03811fa7d53 100644 --- a/sw/source/filter/ww8/ww8graf.cxx +++ b/sw/source/filter/ww8/ww8graf.cxx @@ -1975,8 +1975,14 @@ void SwWW8ImplReader::MapWrapIntoFlyFmt(SvxMSDffImportRec* pRecord, static sal_Int32 lcl_ConvertCrop(sal_uInt32 const nCrop, sal_Int32 const nSize) { // cast to sal_Int32 to handle negative crop properly - return ((static_cast<sal_Int32>(nCrop) >> 16) * nSize) - + (((nCrop & 0xffff) * nSize) >> 16) ; + sal_Int32 const nIntegral(static_cast<sal_Int32>(nCrop) >> 16); + // fdo#77454: heuristic to detect mangled values written by old OOo/LO + if (abs(nIntegral) >= 50) // FIXME: what's a good cut-off? + { + SAL_INFO("sw.ww8", "ignoring suspiciously large crop: " << nIntegral); + return 0; + } + return (nIntegral * nSize) + (((nCrop & 0xffff) * nSize) >> 16); } void |