[2/3] quartz: Add check for NULL pointer in FileSource_Load.

Sebastian Lackner sebastian at fds-team.de
Thu Apr 28 00:59:24 CDT 2016


Signed-off-by: Sebastian Lackner <sebastian at fds-team.de>
---
 dlls/quartz/filesource.c        |    3 +++
 dlls/quartz/tests/avisplitter.c |    1 -
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/dlls/quartz/filesource.c b/dlls/quartz/filesource.c
index 1a2af44..f0f85fd 100644
--- a/dlls/quartz/filesource.c
+++ b/dlls/quartz/filesource.c
@@ -613,6 +613,9 @@ static HRESULT WINAPI FileSource_Load(IFileSourceFilter * iface, LPCOLESTR pszFi
 
     TRACE("(%s, %p)\n", debugstr_w(pszFileName), pmt);
 
+    if (!pszFileName)
+        return E_POINTER;
+
     /* open file */
     /* FIXME: check the sharing values that native uses */
     hFile = CreateFileW(pszFileName, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_FLAG_OVERLAPPED, NULL);
diff --git a/dlls/quartz/tests/avisplitter.c b/dlls/quartz/tests/avisplitter.c
index daccedd..4ed4142 100644
--- a/dlls/quartz/tests/avisplitter.c
+++ b/dlls/quartz/tests/avisplitter.c
@@ -221,7 +221,6 @@ static void test_filesourcefilter(void)
         ok(olepath == NULL, "expected NULL, got %p\n", olepath);
 
         hr = IFileSourceFilter_Load(filesource, NULL, NULL);
-        todo_wine
         ok(hr == E_POINTER, "expected E_POINTER, got %08x\n", hr);
 
         file = CreateFileW(path, GENERIC_READ | GENERIC_WRITE, 0, NULL,
-- 
2.8.0



More information about the wine-patches mailing list