November 2009 - Wine-Commits

Rob Shearman : ole32: Fix the length of data written and read for VT_CF types in property storage code .
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: 2226b678e339aa77eb9f5b1f4a6b4ac295b2ad78 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2226b678e339aa77eb9f5b1f4… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Sun Nov 29 10:33:44 2009 +0000 ole32: Fix the length of data written and read for VT_CF types in property storage code. --- dlls/ole32/stg_prop.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/ole32/stg_prop.c b/dlls/ole32/stg_prop.c index bc45cea..643848a 100644 --- a/dlls/ole32/stg_prop.c +++ b/dlls/ole32/stg_prop.c @@ -1143,8 +1143,8 @@ static HRESULT PropertyStorage_ReadProperty(PropertyStorage_impl *This, prop->u.pclipdata = CoTaskMemAlloc(sizeof (CLIPDATA)); prop->u.pclipdata->cbSize = len; prop->u.pclipdata->ulClipFmt = tag; - prop->u.pclipdata->pClipData = CoTaskMemAlloc(len); - memcpy(prop->u.pclipdata->pClipData, data+8, len); + prop->u.pclipdata->pClipData = CoTaskMemAlloc(len - sizeof(prop->u.pclipdata->ulClipFmt)); + memcpy(prop->u.pclipdata->pClipData, data+8, len - sizeof(prop->u.pclipdata->ulClipFmt)); } else hr = STG_E_INVALIDPARAMETER; @@ -1735,7 +1735,8 @@ static HRESULT PropertyStorage_WritePropertyToStream(PropertyStorage_impl *This, hr = IStream_Write(This->stm, cf_hdr, sizeof(cf_hdr), &count); if (FAILED(hr)) goto end; - hr = IStream_Write(This->stm, var->u.pclipdata->pClipData, len, &count); + hr = IStream_Write(This->stm, var->u.pclipdata->pClipData, + len - sizeof(var->u.pclipdata->ulClipFmt), &count); if (FAILED(hr)) goto end; bytesWritten = count + sizeof cf_hdr;

1 0

Rob Shearman : rpcrt4: Fix memory leak in test_conf_complex_struct.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: cd453f453fc1a348ed0d463c0f0fad86c5b7460e URL: http://source.winehq.org/git/wine.git/?a=commit;h=cd453f453fc1a348ed0d463c0… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Sun Nov 29 10:31:29 2009 +0000 rpcrt4: Fix memory leak in test_conf_complex_struct. --- dlls/rpcrt4/tests/ndr_marshall.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/rpcrt4/tests/ndr_marshall.c b/dlls/rpcrt4/tests/ndr_marshall.c index 2dfba12..15ccbef 100644 --- a/dlls/rpcrt4/tests/ndr_marshall.c +++ b/dlls/rpcrt4/tests/ndr_marshall.c @@ -1879,6 +1879,7 @@ static void test_conf_complex_struct(void) StubMsg.pfnFree(mem); HeapFree(GetProcessHeap(), 0, StubMsg.RpcMsg->Buffer); + HeapFree(GetProcessHeap(), 0, memsrc); } static void test_ndr_buffer(void)

1 0

Rob Shearman : ole32: Fix off-by-one error in FileMonikerImpl_DecomposePath .
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: db33e8fdd9b2a24b1b35b86a0e3814fd98356810 URL: http://source.winehq.org/git/wine.git/?a=commit;h=db33e8fdd9b2a24b1b35b86a0… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Sun Nov 29 10:18:01 2009 +0000 ole32: Fix off-by-one error in FileMonikerImpl_DecomposePath. strgtable needs to be an array of the same size as str, including the nul-terminator. --- dlls/ole32/filemoniker.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/filemoniker.c b/dlls/ole32/filemoniker.c index ff543aa..72f5913 100644 --- a/dlls/ole32/filemoniker.c +++ b/dlls/ole32/filemoniker.c @@ -1048,7 +1048,7 @@ int FileMonikerImpl_DecomposePath(LPCOLESTR str, LPOLESTR** stringTable) TRACE("%s, %p\n", debugstr_w(str), *stringTable); - strgtable = CoTaskMemAlloc(len*sizeof(*strgtable)); + strgtable = CoTaskMemAlloc((len + 1)*sizeof(*strgtable)); if (strgtable==NULL) return E_OUTOFMEMORY;

1 0

Paul Vriens : shell32/tests: Only fetch some special folders once.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: d5dac6c43d672b761c30c82dd1fa2274989cf22d URL: http://source.winehq.org/git/wine.git/?a=commit;h=d5dac6c43d672b761c30c82dd… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Thu Nov 26 19:17:25 2009 +0100 shell32/tests: Only fetch some special folders once. --- dlls/shell32/tests/progman_dde.c | 47 +++++++++++++++++--------------------- 1 files changed, 21 insertions(+), 26 deletions(-) diff --git a/dlls/shell32/tests/progman_dde.c b/dlls/shell32/tests/progman_dde.c index 0ea7571..6ba22ed 100644 --- a/dlls/shell32/tests/progman_dde.c +++ b/dlls/shell32/tests/progman_dde.c @@ -76,6 +76,9 @@ static void init_function_pointers(void) pSHGetSpecialFolderPathA = (void*)GetProcAddress(hmod, "SHGetSpecialFolderPathA"); } +static char CommonPrograms[MAX_PATH]; +static char Programs[MAX_PATH]; + static char Group1Title[MAX_PATH] = "Group1"; static char Group2Title[MAX_PATH] = "Group2"; static char Group3Title[MAX_PATH] = "Group3"; @@ -87,31 +90,34 @@ static void init_strings(void) HKEY key; DWORD fullpath = 0; DWORD size = sizeof(DWORD); + char startup[MAX_PATH]; if (!pSHGetSpecialFolderPathA) return; + /* FIXME: On Vista+ we should only use CSIDL_PROGRAMS */ + pSHGetSpecialFolderPathA(NULL, Programs, CSIDL_PROGRAMS, FALSE); + if (!pSHGetSpecialFolderPathA(NULL, CommonPrograms, CSIDL_COMMON_PROGRAMS, FALSE)) + { + /* Win9x */ + lstrcpyA(CommonPrograms, Programs); + } + pSHGetSpecialFolderPathA(NULL, startup, CSIDL_STARTUP, FALSE); + lstrcpyA(Startup, (strrchr(startup, '\\') + 1)); + RegOpenKeyA(HKEY_CURRENT_USER, "Software\\Microsoft\\Windows\\CurrentVersion\\Explorer\\CabinetState", &key); RegQueryValueExA(key, "FullPath", NULL, NULL, (LPBYTE)&fullpath, &size); RegCloseKey(key); if (fullpath == 1) { - BOOL ret; - char commonprog[MAX_PATH], startup[MAX_PATH]; - - ret = pSHGetSpecialFolderPathA(NULL, commonprog, CSIDL_COMMON_PROGRAMS, FALSE); - if (!ret) /* Win9x */ - pSHGetSpecialFolderPathA(NULL, commonprog, CSIDL_PROGRAMS, FALSE); - lstrcpyA(Group1Title, commonprog); + lstrcpyA(Group1Title, CommonPrograms); lstrcatA(Group1Title, "\\Group1"); - lstrcpyA(Group2Title, commonprog); + lstrcpyA(Group2Title, CommonPrograms); lstrcatA(Group2Title, "\\Group2"); - lstrcpyA(Group3Title, commonprog); + lstrcpyA(Group3Title, CommonPrograms); lstrcatA(Group3Title, "\\Group3"); - pSHGetSpecialFolderPathA(NULL, startup, CSIDL_STARTUP, FALSE); lstrcpyA(StartupTitle, startup); - lstrcpyA(Startup, (strrchr(StartupTitle, '\\') + 1)); } } @@ -282,7 +288,6 @@ static void CheckFileExistsInProgramGroups(const char *nameToCheck, int shouldEx const char *groupName, int testParams) { char *path; - int err; DWORD attributes; int len; @@ -292,21 +297,11 @@ static void CheckFileExistsInProgramGroups(const char *nameToCheck, int shouldEx path = HeapAlloc(GetProcessHeap(), 0, MAX_PATH); if (path != NULL) { - int specialFolder; - - err = FALSE; - /* Win 9x doesn't support common */ if (testParams & DDE_TEST_COMMON) - { - specialFolder = CSIDL_COMMON_PROGRAMS; - err = pSHGetSpecialFolderPathA(NULL, path, specialFolder, FALSE); - /* Win 9x fails, use CSIDL_PROGRAMS (err == FALSE) */ - } - if (err == FALSE) - { - specialFolder = CSIDL_PROGRAMS; - err = pSHGetSpecialFolderPathA(NULL, path, specialFolder, FALSE); - } + lstrcpyA(path, CommonPrograms); + else + lstrcpyA(path, Programs); + len = strlen(path) + strlen(nameToCheck)+1; if (groupName != NULL) {

1 0

Paul Vriens : shell32/tests: Run tests on NT4 again.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: 17089d0c2e8d4bb649705e3889d933dee0ac5880 URL: http://source.winehq.org/git/wine.git/?a=commit;h=17089d0c2e8d4bb649705e388… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Tue Nov 24 21:49:54 2009 +0100 shell32/tests: Run tests on NT4 again. --- dlls/shell32/tests/progman_dde.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/shell32/tests/progman_dde.c b/dlls/shell32/tests/progman_dde.c index 09eb791..0ea7571 100644 --- a/dlls/shell32/tests/progman_dde.c +++ b/dlls/shell32/tests/progman_dde.c @@ -99,9 +99,9 @@ static void init_strings(void) BOOL ret; char commonprog[MAX_PATH], startup[MAX_PATH]; - ret = SHGetSpecialFolderPathA(NULL, commonprog, CSIDL_COMMON_PROGRAMS, FALSE); + ret = pSHGetSpecialFolderPathA(NULL, commonprog, CSIDL_COMMON_PROGRAMS, FALSE); if (!ret) /* Win9x */ - SHGetSpecialFolderPathA(NULL, commonprog, CSIDL_PROGRAMS, FALSE); + pSHGetSpecialFolderPathA(NULL, commonprog, CSIDL_PROGRAMS, FALSE); lstrcpyA(Group1Title, commonprog); lstrcatA(Group1Title, "\\Group1"); lstrcpyA(Group2Title, commonprog); @@ -109,7 +109,7 @@ static void init_strings(void) lstrcpyA(Group3Title, commonprog); lstrcatA(Group3Title, "\\Group3"); - SHGetSpecialFolderPathA(NULL, startup, CSIDL_STARTUP, FALSE); + pSHGetSpecialFolderPathA(NULL, startup, CSIDL_STARTUP, FALSE); lstrcpyA(StartupTitle, startup); lstrcpyA(Startup, (strrchr(StartupTitle, '\\') + 1)); }

1 0

Paul Vriens : shell32/tests: Use ReadCabinetState to check title logic.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: 98fef79f2d976658fdf7fde234a5e7d948d98072 URL: http://source.winehq.org/git/wine.git/?a=commit;h=98fef79f2d976658fdf7fde23… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Fri Nov 27 16:26:22 2009 +0100 shell32/tests: Use ReadCabinetState to check title logic. --- dlls/shell32/tests/progman_dde.c | 18 ++++++++++-------- 1 files changed, 10 insertions(+), 8 deletions(-) diff --git a/dlls/shell32/tests/progman_dde.c b/dlls/shell32/tests/progman_dde.c index 8902d90..fd10298 100644 --- a/dlls/shell32/tests/progman_dde.c +++ b/dlls/shell32/tests/progman_dde.c @@ -67,6 +67,7 @@ #define DDE_TEST_NUMMASK 0x0000ffff static BOOL (WINAPI *pSHGetSpecialFolderPathA)(HWND, LPSTR, int, BOOL); +static BOOL (WINAPI *pReadCabinetState)(CABINETSTATE *, int); static void init_function_pointers(void) { @@ -74,6 +75,9 @@ static void init_function_pointers(void) hmod = GetModuleHandleA("shell32.dll"); pSHGetSpecialFolderPathA = (void*)GetProcAddress(hmod, "SHGetSpecialFolderPathA"); + pReadCabinetState = (void*)GetProcAddress(hmod, "ReadCabinetState"); + if (!pReadCabinetState) + pReadCabinetState = (void*)GetProcAddress(hmod, (LPSTR)651); } static char CommonPrograms[MAX_PATH]; @@ -87,10 +91,8 @@ static char StartupTitle[MAX_PATH] = "Startup"; static void init_strings(void) { - HKEY key; - DWORD fullpath = 0; - DWORD size; char startup[MAX_PATH]; + CABINETSTATE cs; if (pSHGetSpecialFolderPathA) { @@ -106,6 +108,8 @@ static void init_strings(void) } else { + HKEY key; + DWORD size; LONG res; /* Older Win9x and NT4 */ @@ -128,11 +132,9 @@ static void init_strings(void) RegCloseKey(key); } - RegOpenKeyA(HKEY_CURRENT_USER, "Software\\Microsoft\\Windows\\CurrentVersion\\Explorer\\CabinetState", &key); - size = sizeof(DWORD); - RegQueryValueExA(key, "FullPath", NULL, NULL, (LPBYTE)&fullpath, &size); - RegCloseKey(key); - if (fullpath == 1) + memset(&cs, 0, sizeof(cs)); + pReadCabinetState(&cs, sizeof(cs)); + if (cs.fFullPathTitle == -1) { lstrcpyA(Group1Title, CommonPrograms); lstrcatA(Group1Title, "\\Group1");

1 0

Eric Pouech : dbghelp: In dwarf parsing, allow 64 bit wide values.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: db077e11adbb66f26d2ce7de8b79990a58d0ad2f URL: http://source.winehq.org/git/wine.git/?a=commit;h=db077e11adbb66f26d2ce7de8… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Nov 28 13:24:01 2009 +0100 dbghelp: In dwarf parsing, allow 64 bit wide values. --- dlls/dbghelp/dwarf.c | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/dlls/dbghelp/dwarf.c b/dlls/dbghelp/dwarf.c index e5617c3..6006202 100644 --- a/dlls/dbghelp/dwarf.c +++ b/dlls/dbghelp/dwarf.c @@ -490,7 +490,9 @@ static void dwarf2_fill_attr(const dwarf2_parse_context_t* ctx, break; case DW_FORM_data8: - FIXME("Unhandled 64bits support\n"); + attr->u.block.size = 8; + attr->u.block.ptr = data; + data += 8; break; case DW_FORM_ref1: @@ -1411,6 +1413,11 @@ static void dwarf2_parse_variable(dwarf2_subprogram_t* subpgm, break; case DW_FORM_data8: + v.n1.n2.vt = VT_I1 | VT_BYREF; + v.n1.n2.n3.byref = pool_alloc(&subpgm->ctx->module->pool, value.u.block.size); + memcpy(v.n1.n2.n3.byref, value.u.block.ptr, value.u.block.size); + break; + default: FIXME("Unsupported form for const value %s (%lx)\n", name.u.string, value.form);

1 0

Paul Vriens : shell32/tests: Change a trace into a skip for skipped tests.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: 404dcbcd65aa07b8a6ddbb9672f598b99a80a9ab URL: http://source.winehq.org/git/wine.git/?a=commit;h=404dcbcd65aa07b8a6ddbb967… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Fri Nov 27 13:27:59 2009 +0100 shell32/tests: Change a trace into a skip for skipped tests. --- dlls/shell32/tests/shlexec.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/shell32/tests/shlexec.c b/dlls/shell32/tests/shlexec.c index 5e97f61..c673f0a 100644 --- a/dlls/shell32/tests/shlexec.c +++ b/dlls/shell32/tests/shlexec.c @@ -1095,7 +1095,7 @@ static void test_find_executable(void) * like '.mpeg', etc. * Also it means we cannot do any other test. */ - trace("FindExecutable() is broken -> skipping 4+ character extension tests\n"); + win_skip("FindExecutable() is broken -> not running 4+ character extension tests\n"); return; }

1 0

Paul Vriens : shell32/tests: Remove an unneeded HeapAlloc.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: e5d83ac6257193123b4c386d490fe625f8ff4e7c URL: http://source.winehq.org/git/wine.git/?a=commit;h=e5d83ac6257193123b4c386d4… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Fri Nov 27 16:28:57 2009 +0100 shell32/tests: Remove an unneeded HeapAlloc. --- dlls/shell32/tests/progman_dde.c | 67 ++++++++++++++++--------------------- 1 files changed, 29 insertions(+), 38 deletions(-) diff --git a/dlls/shell32/tests/progman_dde.c b/dlls/shell32/tests/progman_dde.c index f1d7d31..7bc53d4 100644 --- a/dlls/shell32/tests/progman_dde.c +++ b/dlls/shell32/tests/progman_dde.c @@ -320,56 +320,47 @@ static void CheckWindowCreated(const char *winName, int closeWindow, int testPar static void CheckFileExistsInProgramGroups(const char *nameToCheck, int shouldExist, int isGroup, const char *groupName, int testParams) { - char *path; + char path[MAX_PATH]; DWORD attributes; int len; - path = HeapAlloc(GetProcessHeap(), 0, MAX_PATH); - if (path != NULL) - { - if (testParams & DDE_TEST_COMMON) - lstrcpyA(path, CommonPrograms); - else - lstrcpyA(path, Programs); + if (testParams & DDE_TEST_COMMON) + lstrcpyA(path, CommonPrograms); + else + lstrcpyA(path, Programs); - len = strlen(path) + strlen(nameToCheck)+1; + len = strlen(path) + strlen(nameToCheck)+1; + if (groupName != NULL) + { + len += strlen(groupName)+1; + } + ok (len <= MAX_PATH, "Path Too Long.%s\n", GetStringFromTestParams(testParams)); + if (len <= MAX_PATH) + { if (groupName != NULL) { - len += strlen(groupName)+1; + strcat(path, "\\"); + strcat(path, groupName); } - ok (len <= MAX_PATH, "Path Too Long.%s\n", GetStringFromTestParams(testParams)); - if (len <= MAX_PATH) + strcat(path, "\\"); + strcat(path, nameToCheck); + attributes = GetFileAttributes(path); + if (!shouldExist) { - if (groupName != NULL) + ok (attributes == INVALID_FILE_ATTRIBUTES , "File exists and shouldn't %s.%s\n", + path, GetStringFromTestParams(testParams)); + } else { + if (attributes == INVALID_FILE_ATTRIBUTES) { - strcat(path, "\\"); - strcat(path, groupName); - } - strcat(path, "\\"); - strcat(path, nameToCheck); - attributes = GetFileAttributes(path); - if (!shouldExist) - { - ok (attributes == INVALID_FILE_ATTRIBUTES , "File exists and shouldn't %s.%s\n", - path, GetStringFromTestParams(testParams)); + ok (FALSE, "Created File %s doesn't exist.%s\n", path, GetStringFromTestParams(testParams)); + } else if (isGroup) { + ok (attributes & FILE_ATTRIBUTE_DIRECTORY, "%s is not a folder (attr=%x).%s\n", + path, attributes, GetStringFromTestParams(testParams)); } else { - if (attributes == INVALID_FILE_ATTRIBUTES) - { - ok (FALSE, "Created File %s doesn't exist.%s\n", path, GetStringFromTestParams(testParams)); - } else if (isGroup) { - ok (attributes & FILE_ATTRIBUTE_DIRECTORY, "%s is not a folder (attr=%x).%s\n", - path, attributes, GetStringFromTestParams(testParams)); - } else { - ok (attributes & FILE_ATTRIBUTE_ARCHIVE, "Created File %s has wrong attributes (%x).%s\n", - path, attributes, GetStringFromTestParams(testParams)); - } + ok (attributes & FILE_ATTRIBUTE_ARCHIVE, "Created File %s has wrong attributes (%x).%s\n", + path, attributes, GetStringFromTestParams(testParams)); } } - HeapFree(GetProcessHeap(), 0, path); - } - else - { - ok (FALSE, "Could not Allocate Path Buffer\n"); } }

1 0

Paul Vriens : shell32/tests: Fix test failures on Vista and higher.
by Alexandre Julliard Nov. 30, 2009

Nov. 30, 2009

Module: wine Branch: master Commit: 1c6f813bc7bb53c83997e39b085e3d0c78ff5434 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1c6f813bc7bb53c83997e39b0… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Fri Nov 27 16:27:10 2009 +0100 shell32/tests: Fix test failures on Vista and higher. --- dlls/shell32/tests/progman_dde.c | 53 ++++++++++++++++++++++++++++++++++--- 1 files changed, 48 insertions(+), 5 deletions(-) diff --git a/dlls/shell32/tests/progman_dde.c b/dlls/shell32/tests/progman_dde.c index fd10298..f1d7d31 100644 --- a/dlls/shell32/tests/progman_dde.c +++ b/dlls/shell32/tests/progman_dde.c @@ -96,13 +96,20 @@ static void init_strings(void) if (pSHGetSpecialFolderPathA) { - /* FIXME: On Vista+ we should only use CSIDL_PROGRAMS */ pSHGetSpecialFolderPathA(NULL, Programs, CSIDL_PROGRAMS, FALSE); if (!pSHGetSpecialFolderPathA(NULL, CommonPrograms, CSIDL_COMMON_PROGRAMS, FALSE)) { /* Win9x */ lstrcpyA(CommonPrograms, Programs); } + if (GetProcAddress(GetModuleHandleA("shell32.dll"), "SHGetKnownFolderPath")) + { + /* Vista and higher use CSIDL_PROGRAMS for these tests. + * Wine doesn't have SHGetKnownFolderPath yet but should most likely follow + * this new ProgMan DDE behavior once implemented. + */ + lstrcpyA(CommonPrograms, Programs); + } pSHGetSpecialFolderPathA(NULL, startup, CSIDL_STARTUP, FALSE); lstrcpyA(Startup, (strrchr(startup, '\\') + 1)); } @@ -542,12 +549,25 @@ static void CompoundCommandTest(DWORD instance, HCONV hConv, const char *command } } +static void CreateAddItemText(char *itemtext, const char *cmdline, const char *name) +{ + lstrcpyA(itemtext, "[AddItem("); + lstrcatA(itemtext, cmdline); + lstrcatA(itemtext, ","); + lstrcatA(itemtext, name); + lstrcatA(itemtext, ")]"); +} + /* 1st set of tests */ static int DdeTestProgman(DWORD instance, HCONV hConv) { HDDEDATA hData; UINT error; int testnum; + char temppath[MAX_PATH]; + char f1g1[MAX_PATH], f2g1[MAX_PATH], f3g1[MAX_PATH], f1g3[MAX_PATH], f2g3[MAX_PATH]; + char itemtext[MAX_PATH + 20]; + char comptext[2 * (MAX_PATH + 20) + 21]; testnum = 1; /* Invalid Command */ @@ -555,12 +575,23 @@ static int DdeTestProgman(DWORD instance, HCONV hConv) ok (error == DMLERR_NOTPROCESSED, "InvalidCommand(), expected error %s, received %s.\n", GetStringFromError(DMLERR_NOTPROCESSED), GetStringFromError(error)); + /* On Vista+ the files have to exist when adding a link */ + GetTempPathA(MAX_PATH, temppath); + GetTempFileNameA(temppath, "dde", 0, f1g1); + GetTempFileNameA(temppath, "dde", 0, f2g1); + GetTempFileNameA(temppath, "dde", 0, f3g1); + GetTempFileNameA(temppath, "dde", 0, f1g3); + GetTempFileNameA(temppath, "dde", 0, f2g3); + /* CreateGroup Tests (including AddItem, DeleteItem) */ CreateGroupTest(instance, hConv, "[CreateGroup(Group1)]", DMLERR_NO_ERROR, "Group1", Group1Title, DDE_TEST_COMMON|DDE_TEST_CREATEGROUP|testnum++); - AddItemTest(instance, hConv, "[AddItem(c:\\f1g1,f1g1Name)]", DMLERR_NO_ERROR, "f1g1Name.lnk", "Group1", DDE_TEST_COMMON|DDE_TEST_ADDITEM|testnum++); - AddItemTest(instance, hConv, "[AddItem(c:\\f2g1,f2g1Name)]", DMLERR_NO_ERROR, "f2g1Name.lnk", "Group1", DDE_TEST_COMMON|DDE_TEST_ADDITEM|testnum++); + CreateAddItemText(itemtext, f1g1, "f1g1Name"); + AddItemTest(instance, hConv, itemtext, DMLERR_NO_ERROR, "f1g1Name.lnk", "Group1", DDE_TEST_COMMON|DDE_TEST_ADDITEM|testnum++); + CreateAddItemText(itemtext, f2g1, "f2g1Name"); + AddItemTest(instance, hConv, itemtext, DMLERR_NO_ERROR, "f2g1Name.lnk", "Group1", DDE_TEST_COMMON|DDE_TEST_ADDITEM|testnum++); DeleteItemTest(instance, hConv, "[DeleteItem(f2g1Name)]", DMLERR_NO_ERROR, "f2g1Name.lnk", "Group1", DDE_TEST_COMMON|DDE_TEST_DELETEITEM|testnum++); - AddItemTest(instance, hConv, "[AddItem(c:\\f3g1,f3g1Name)]", DMLERR_NO_ERROR, "f3g1Name.lnk", "Group1", DDE_TEST_COMMON|DDE_TEST_ADDITEM|testnum++); + CreateAddItemText(itemtext, f3g1, "f3g1Name"); + AddItemTest(instance, hConv, itemtext, DMLERR_NO_ERROR, "f3g1Name.lnk", "Group1", DDE_TEST_COMMON|DDE_TEST_ADDITEM|testnum++); CreateGroupTest(instance, hConv, "[CreateGroup(Group2)]", DMLERR_NO_ERROR, "Group2", Group2Title, DDE_TEST_COMMON|DDE_TEST_CREATEGROUP|testnum++); /* Create Group that already exists - same instance */ CreateGroupTest(instance, hConv, "[CreateGroup(Group1)]", DMLERR_NO_ERROR, "Group1", Group1Title, DDE_TEST_COMMON|DDE_TEST_CREATEGROUP|testnum++); @@ -575,12 +606,24 @@ static int DdeTestProgman(DWORD instance, HCONV hConv) DeleteGroupTest(instance, hConv, "[DeleteGroup(Group1)]", DMLERR_NO_ERROR, "Group1", DDE_TEST_DELETEGROUP|testnum++); /* Compound Execute String Command */ - CompoundCommandTest(instance, hConv, "[CreateGroup(Group3)][AddItem(c:\\f1g3,f1g3Name)][AddItem(c:\\f2g3,f2g3Name)]", DMLERR_NO_ERROR, "Group3", Group3Title, "f1g3Name.lnk", "f2g3Name.lnk", DDE_TEST_COMMON|DDE_TEST_COMPOUND|testnum++); + lstrcpyA(comptext, "[CreateGroup(Group3)]"); + CreateAddItemText(itemtext, f1g3, "f1g3Name"); + lstrcatA(comptext, itemtext); + CreateAddItemText(itemtext, f2g3, "f2g3Name"); + lstrcatA(comptext, itemtext); + CompoundCommandTest(instance, hConv, comptext, DMLERR_NO_ERROR, "Group3", Group3Title, "f1g3Name.lnk", "f2g3Name.lnk", DDE_TEST_COMMON|DDE_TEST_COMPOUND|testnum++); + DeleteGroupTest(instance, hConv, "[DeleteGroup(Group3)]", DMLERR_NO_ERROR, "Group3", DDE_TEST_DELETEGROUP|testnum++); /* Full Parameters of Add Item */ /* AddItem(CmdLine[,Name[,IconPath[,IconIndex[,xPos,yPos[,DefDir[,HotKey[,fMinimize[fSeparateSpace]]]]]]]) */ + DeleteFileA(f1g1); + DeleteFileA(f2g1); + DeleteFileA(f3g1); + DeleteFileA(f1g3); + DeleteFileA(f2g3); + return testnum; }

1 0