Alexandre Julliard : mscms/tests: Allocate the profile buffer dynamically to make sure it's large enough.

[Alexandre Julliard]

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

Author: Alexandre Julliard 
Date:   Fri Aug 17 11:52:29 2007 +0200

mscms/tests: Allocate the profile buffer dynamically to make sure it's large enough.

---

dlls/mscms/tests/profile.c |   30 +++++++++++++++++++++---------
 1 files changed, 21 insertions(+), 9 deletions(-)

diff --git a/dlls/mscms/tests/profile.c b/dlls/mscms/tests/profile.c
index e5a7e81..1055faa 100644
--- a/dlls/mscms/tests/profile.c
+++ b/dlls/mscms/tests/profile.c
@@ -760,13 +760,12 @@ static void test_GetStandardColorSpaceProfileW(void)
 static void test_EnumColorProfilesA(void)
 {
     BOOL ret;
-    DWORD size, number;
+    DWORD total, size, number;
     ENUMTYPEA record;
-    BYTE buffer[MAX_PATH];
+    BYTE *buffer;

/* Parameter checks */

-    size = sizeof(buffer);
     memset( &record, 0, sizeof(ENUMTYPEA) );

record.dwSize = sizeof(ENUMTYPEA);
@@ -774,6 +773,12 @@ static void test_EnumColorProfilesA(void)
     record.dwFields |= ET_DATACOLORSPACE;
     record.dwDataColorSpace = SPACE_RGB;

+    total = 0;
+    ret = pEnumColorProfilesA( NULL, &record, NULL, &total, &number );
+    ok( !ret, "EnumColorProfilesA() failed (%d)\n", GetLastError() );
+    buffer = HeapAlloc( GetProcessHeap(), 0, total );
+
+    size = total;
     ret = pEnumColorProfilesA( machine, &record, buffer, &size, &number );
     ok( !ret, "EnumColorProfilesA() succeeded (%d)\n", GetLastError() );

@@ -798,23 +803,23 @@ static void test_EnumColorProfilesA(void)

if (standardprofile)
     {
-        size = sizeof(buffer);
+        size = total;

ret = pEnumColorProfilesA( NULL, &record, buffer, &size, &number );
         ok( ret, "EnumColorProfilesA() failed (%d)\n", GetLastError() );
     }
+    HeapFree( GetProcessHeap(), 0, buffer );
 }

static void test_EnumColorProfilesW(void)
 {
     BOOL ret;
-    DWORD size, number;
+    DWORD total, size, number;
     ENUMTYPEW record;
-    BYTE buffer[MAX_PATH * sizeof(WCHAR)];
+    BYTE *buffer;

/* Parameter checks */

-    size = sizeof(buffer);
     memset( &record, 0, sizeof(ENUMTYPEW) );

record.dwSize = sizeof(ENUMTYPEW);
@@ -822,6 +827,12 @@ static void test_EnumColorProfilesW(void)
     record.dwFields |= ET_DATACOLORSPACE;
     record.dwDataColorSpace = SPACE_RGB;

+    total = 0;
+    ret = pEnumColorProfilesW( NULL, &record, NULL, &total, &number );
+    ok( !ret, "EnumColorProfilesW() failed (%d)\n", GetLastError() );
+    buffer = HeapAlloc( GetProcessHeap(), 0, total * sizeof(WCHAR) );
+
+    size = total;
     ret = pEnumColorProfilesW( machineW, &record, buffer, &size, &number );
     ok( !ret, "EnumColorProfilesW() succeeded (%d)\n", GetLastError() );

@@ -833,7 +844,7 @@ static void test_EnumColorProfilesW(void)

if (standardprofileW)
     {
-        ret = pEnumColorProfilesW( NULL, &record, buffer, &size, NULL );
+        ret = pEnumColorProfilesW( NULL, &record, buffer, &size, &number );
         ok( ret, "EnumColorProfilesW() failed (%d)\n", GetLastError() );
     }

@@ -846,11 +857,12 @@ static void test_EnumColorProfilesW(void)

if (standardprofileW)
     {
-        size = sizeof(buffer);
+        size = total;

ret = pEnumColorProfilesW( NULL, &record, buffer, &size, &number );
         ok( ret, "EnumColorProfilesW() failed (%d)\n", GetLastError() );
     }
+    HeapFree( GetProcessHeap(), 0, buffer );
 }

static void test_InstallColorProfileA(void)