quartz: Only query for event interface when there is a filtergraph.

Prevents a null pointer dereference.

quartz: Only query for event interface when there is a filtergraph.
3a9040c1 · Maarten Lankhorst · Alexandre Julliard · e8705402 · 3a9040c1
Commit 3a9040c1 authored Jun 10, 2008 by Maarten Lankhorst Committed by Alexandre Julliard Jun 21, 2008
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 5 deletions

nullrenderer.c dlls/quartz/nullrenderer.c +10 -5

No files found.
--- a/dlls/quartz/nullrenderer.c
+++ b/dlls/quartz/nullrenderer.c
@@ -568,16 +568,21 @@ static HRESULT WINAPI NullRenderer_InputPin_EndOfStream(IPin * iface)
 {
    InputPin* This = (InputPin*)iface;
    IMediaEventSink* pEventSink;
-    HRESULT hr;
+    IFilterGraph *graph;
+    HRESULT hr = S_OK;

    TRACE("(%p/%p)->()\n", This, iface);

    InputPin_EndOfStream(iface);
-    hr = IFilterGraph_QueryInterface(((NullRendererImpl*)This->pin.pinInfo.pFilter)->filterInfo.pGraph, &IID_IMediaEventSink, (LPVOID*)&pEventSink);
-    if (SUCCEEDED(hr))
+    graph = ((NullRendererImpl*)This->pin.pinInfo.pFilter)->filterInfo.pGraph;
+    if (graph)
    {
-        hr = IMediaEventSink_Notify(pEventSink, EC_COMPLETE, S_OK, 0);
-        IMediaEventSink_Release(pEventSink);
+        hr = IFilterGraph_QueryInterface(((NullRendererImpl*)This->pin.pinInfo.pFilter)->filterInfo.pGraph, &IID_IMediaEventSink, (LPVOID*)&pEventSink);
+        if (SUCCEEDED(hr))
+        {
+            hr = IMediaEventSink_Notify(pEventSink, EC_COMPLETE, S_OK, 0);
+            IMediaEventSink_Release(pEventSink);
+        }
    }

    return hr;