Created attachment 180258 [details] Comparison LibreOffice 7.4 master and PP 2016 Steps to reproduce: 1. Open attachment 158365 [details] from bug 131109 -> The image has incorrect height. See comparison Reproduced in Version: 7.4.0.0.alpha1+ / LibreOffice Community Build ID: 22d7cbc0f0c50c59abca7ab5f7db46e4d0e5f86a CPU threads: 8; OS: Linux 5.10; UI render: default; VCL: gtk3 Locale: es-ES (es_ES.UTF-8); UI: en-US Calc: threaded [Bug found by office-interoperability-tools]
Regression introduced by: author offtkp <parisoplop@gmail.com> 2022-04-02 15:49:32 +0300 committer Tomaž Vajngerl <quikee@gmail.com> 2022-05-06 04:02:05 +0200 commit e9c50fbbc3b07ef927d133da9cf2395c55611e0f (patch) tree 7b2dbf92184b5b2707685dc44ee4764d4d58fdad parent e4a7a50d831cb6fca95f2afff2af5298d66b5876 (diff) tdf#103954: Z compressed graphic formats support for EMF/WMF Bisected with: bibisect-linux64-7.4 Adding Cc: to offtkp
Reproduced, I'll look into this
The problem is probably the code removed in GraphicDescriptor::ImpDetectEMF as that one reads the size from the metadata and sets it..
offtkp committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/063d641f023ac90f49116de105e32e38be72a1af tdf#149206 Set size correctly when loading EMF It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/504d61697bac45236da27fba23c7189daf8c171e tdf#149206: sd_import_tests2: Add unittest It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
offtkp committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5f30b778bccbcc417e4ed1da5997650babb942a8 tdf#149206: Add EMF test in GraphicDescriptorTest It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Verified in Version: 7.4.0.0.alpha1+ / LibreOffice Community Build ID: a8df5c815c8b002b7083b8777e3dd8beac573bf3 CPU threads: 8; OS: Linux 5.10; UI render: default; VCL: gtk3 Locale: es-ES (es_ES.UTF-8); UI: en-US Calc: threaded @Paris, thanks for fixing this issue!!