diff options
| author | Patrick Jaap <patrick.jaap@tu-dresden.de> | 2020-02-06 18:33:19 +0100 |
|---|---|---|
| committer | Patrick Jaap <patrick.jaap@tu-dresden.de> | 2020-02-11 15:29:27 +0100 |
| commit | 6e5444959ec914f6d0979aa1e6d4d8a462d79891 (patch) | |
| tree | 63d3a6b3859a89407d02a658fc3999d0e2196b34 /filter | |
| parent | 020ee2fcc1a40415dca8e974cf30834667f55036 (diff) | |
tdf#130262 ignore negative wrap distance values in doc prop import
Change-Id: I2c06b12c958dfde09adcc2ae4c59532ffde6b6fe
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88127
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
(cherry picked from commit 8cbd7f59bb99282c4bcb60639da1263c8eb3a5e3)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88391
Reviewed-by: Patrick Jaap <patrick.jaap@tu-dresden.de>
Diffstat (limited to 'filter')
| -rw-r--r-- | filter/source/msfilter/dffpropset.cxx | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/filter/source/msfilter/dffpropset.cxx b/filter/source/msfilter/dffpropset.cxx index 5b0b1194ae06..e5803046b4e4 100644 --- a/filter/source/msfilter/dffpropset.cxx +++ b/filter/source/msfilter/dffpropset.cxx @@ -1217,6 +1217,17 @@ void DffPropSet::ReadPropSet( SvStream& rIn, bool bSetUninitializedOnly ) } if ( bSetProperty ) { + // tdf#130262: ignore negative values for distances (maybe this list needs to be extended) + // LO does not allow negative values but [MS-ODRAW] does not forbid them + if ( nRecType == DFF_Prop_dxWrapDistLeft || nRecType == DFF_Prop_dxWrapDistRight + || nRecType == DFF_Prop_dyWrapDistTop || nRecType == DFF_Prop_dyWrapDistBottom ) + { + if ( static_cast<sal_Int32>(nContent) < 0 ) + { + break; + } + } + mpPropSetEntries[ nRecType ].nContent = nContent; mpPropSetEntries[ nRecType ].aFlags = aPropFlag; } |