[PATCH 0/1] MR9439: mf: Return E_NOINTERFACE if service is missing

[Aric Stewart (＠aricstewart)]

Signed-off-by: Aric Stewart

[PATCH 1/1] mf: Return E_NOINTERFACE if service is missing

[Aric Stewart]

From: Aric Stewart

Signed-off-by: Aric Stewart 
---
 dlls/mf/session.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/dlls/mf/session.c b/dlls/mf/session.c
index 30bfdd62cba..8d2ca079634 100644
--- a/dlls/mf/session.c
+++ b/dlls/mf/session.c
@@ -2732,7 +2732,10 @@ static HRESULT WINAPI session_get_service_GetService(IMFGetService *iface, REFGU
         hr = session_get_audio_render_service(session, service, riid, obj);
     }
     else
+    {
         FIXME("Unsupported service %s.\n", debugstr_guid(service));
+        hr = E_NOINTERFACE;
+    }

LeaveCriticalSection(&session->cs);

[Nikolay Sivov (＠nsivov)]

Could you please add a test? The change looks correct.

[Nikolay Sivov (＠nsivov)]

If you have access to application that triggered this, could you check which service GUID was queried? Maybe it's another thing we should fix.

[Aric Stewart (＠aricstewart)]

Powerpoint 2016 playing a wav file is the trigger.

It is trying to query the MF_WORKQUEUE_SERVICES GUID,  which looks like a lot of work.  I just found that this fix worked while triaging the issue.

I am building windows VM to run and verify my test.