[PATCH 0/1] MR2570: winegstreamer: Fix check for non-zero padding in h264 transform.

[Paul Gofman (＠gofman)]

IsRectEmpty() doesn't do the right thing as padding is not a rectangle. E. g., for rect (0, 0, 0, 8) IsRectEmpty() will return TRUE while the padding is apparently not empty.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2570

[PATCH 1/1] winegstreamer: Fix check for non-zero padding in h264 transform.

[Paul Gofman]

From: Paul Gofman <pgofman(a)codeweavers.com>

---
 dlls/winegstreamer/h264_decoder.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/winegstreamer/h264_decoder.c b/dlls/winegstreamer/h264_decoder.c
index 439701098ad..7d9c38837d3 100644
--- a/dlls/winegstreamer/h264_decoder.c
+++ b/dlls/winegstreamer/h264_decoder.c
@@ -183,7 +183,8 @@ static HRESULT fill_output_media_type(struct h264_decoder *decoder, IMFMediaType
     }
 
     if (FAILED(hr = IMFMediaType_GetItem(media_type, &MF_MT_MINIMUM_DISPLAY_APERTURE, NULL))
-            && !IsRectEmpty(&wg_format->u.video.padding))
+            && (wg_format->u.video.padding.left || wg_format->u.video.padding.right || wg_format->u.video.padding.top
+            || wg_format->u.video.padding.bottom))
     {
         MFVideoArea aperture =
         {
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/2570

[PATCH 0/1] MR2570: winegstreamer: Fix check for non-zero padding in h264 transform. - approved

[Rémi Bernon]

This merge request was approved by Rémi Bernon.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2570