[PATCH] dmime/tests: Break out of loop if we fail to get segment pointer

Alistair Leslie-Hughes leslie_alistair at hotmail.com
Fri Oct 4 00:34:11 CDT 2019


Signed-off-by: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
---
This stop the crash which can be seen. Strange that it worked yesterday.
https://source.winehq.org/patches/data/171018

 dlls/dmime/tests/performance.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/dlls/dmime/tests/performance.c b/dlls/dmime/tests/performance.c
index e4e5dc1b58..962a632875 100644
--- a/dlls/dmime/tests/performance.c
+++ b/dlls/dmime/tests/performance.c
@@ -454,6 +454,11 @@ static void test_notification_type(void)
 
                 hr = IDirectMusicSegmentState8_GetSegment(segmentstate, &segment);
                 ok(hr == S_OK, "Failed: %08x\n", hr);
+                if (FAILED(hr))
+                {
+                    IDirectMusicSegmentState8_Release(segmentstate);
+                    break;
+                }
 
                 hr = IDirectMusicSegment_QueryInterface(segment, &IID_IDirectMusicSegment8, (void**)&segment8);
                 ok(hr == S_OK, "Failed: %08x\n", hr);
-- 
2.17.1




More information about the wine-devel mailing list