Andrey Melnikov : gdi32: Handle NULL filename in GetICMProfileA().

[Alexandre Julliard]

Module: wine
Branch: master
Commit: d4d304f4365134a6aec0fad18989e3c5f1950fe1
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=d4d304f4365134a6aec0fad189...

Author: Andrey Melnikov <temnota.am(a)gmail.com>
Date:   Mon Dec 21 12:22:41 2015 +0300

gdi32: Handle NULL filename in GetICMProfileA().

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>
Signed-off-by: Andrey Melnikov <temnota.am(a)gmail.com>
Signed-off-by: Huw Davies <huw(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 dlls/gdi32/icm.c       | 9 ++++++++-
 dlls/gdi32/tests/icm.c | 1 -
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/dlls/gdi32/icm.c b/dlls/gdi32/icm.c
index d3a345d..9d0e2f6 100644
--- a/dlls/gdi32/icm.c
+++ b/dlls/gdi32/icm.c
@@ -116,11 +116,18 @@ BOOL WINAPI GetICMProfileA(HDC hdc, LPDWORD size, LPSTR filename)
 
     TRACE("%p, %p, %p\n", hdc, size, filename);
 
-    if (!hdc || !size || !filename) return FALSE;
+    if (!hdc || !size) return FALSE;
 
     if (GetICMProfileW(hdc, &buflen, filenameW))
     {
         int len = WideCharToMultiByte(CP_ACP, 0, filenameW, -1, NULL, 0, NULL, NULL);
+
+        if (!filename)
+        {
+            *size = len;
+            return FALSE;
+        }
+
         if (*size >= len)
         {
             WideCharToMultiByte(CP_ACP, 0, filenameW, -1, filename, *size, NULL, NULL);
diff --git a/dlls/gdi32/tests/icm.c b/dlls/gdi32/tests/icm.c
index 764d0d2..6208090 100644
--- a/dlls/gdi32/tests/icm.c
+++ b/dlls/gdi32/tests/icm.c
@@ -55,7 +55,6 @@ static void test_GetICMProfileA( HDC dc )
     size = 0;
     ret = GetICMProfileA( dc, &size, NULL );
     ok( !ret, "GetICMProfileA succeeded\n" );
-todo_wine
     ok( size > 0, "got %u\n", size );
 
     size = MAX_PATH;