wine-commits December 2009

wine-commits@winehq.org

3 participants
1233 discussions

Paul Vriens : kernel32/tests: Only set the size ourselves if we are on W2K.
by Alexandre Julliard 15 Dec '09

15 Dec '09

Module: wine Branch: master Commit: 763694a646af16a38804d635dba24851b0530342 URL: http://source.winehq.org/git/wine.git/?a=commit;h=763694a646af16a38804d635d… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon Dec 14 19:43:17 2009 +0100 kernel32/tests: Only set the size ourselves if we are on W2K. --- dlls/kernel32/tests/environ.c | 15 +++++++++++---- 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/dlls/kernel32/tests/environ.c b/dlls/kernel32/tests/environ.c index b352667..19d11b5 100644 --- a/dlls/kernel32/tests/environ.c +++ b/dlls/kernel32/tests/environ.c @@ -395,7 +395,11 @@ static void test_GetComputerNameExA(void) ok(error == ERROR_MORE_DATA, "Expected ERROR_MORE_DATA, got %d\n", error); /* size is not set in win2k */ - size = MAX_COMP_NAME; + if (size == 0) + { + win_skip("Win2k doesn't set the size\n"); + size = MAX_COMP_NAME; + } name = HeapAlloc(GetProcessHeap(), 0, size * sizeof(name[0])); ok(name != NULL, "HeapAlloc failed with error %d\n", GetLastError()); ret = pGetComputerNameExA(ComputerNameDnsDomain, name, &size); @@ -410,7 +414,8 @@ static void test_GetComputerNameExA(void) ok(error == ERROR_MORE_DATA, "Expected ERROR_MORE_DATA, got %d\n", error); /* size is not set in win2k */ - size = MAX_COMP_NAME; + if (size == 0) + size = MAX_COMP_NAME; name = HeapAlloc(GetProcessHeap(), 0, size * sizeof(name[0])); ok(name != NULL, "HeapAlloc failed with error %d\n", GetLastError()); ret = pGetComputerNameExA(ComputerNameDnsFullyQualified, name, &size); @@ -425,7 +430,8 @@ static void test_GetComputerNameExA(void) ok(error == ERROR_MORE_DATA, "Expected ERROR_MORE_DATA, got %d\n", error); /* size is not set in win2k */ - size = MAX_COMP_NAME; + if (size == 0) + size = MAX_COMP_NAME; name = HeapAlloc(GetProcessHeap(), 0, size * sizeof(name[0])); ok(name != NULL, "HeapAlloc failed with error %d\n", GetLastError()); ret = pGetComputerNameExA(ComputerNameDnsHostname, name, &size); @@ -440,7 +446,8 @@ static void test_GetComputerNameExA(void) ok(error == ERROR_MORE_DATA, "Expected ERROR_MORE_DATA, got %d\n", error); /* size is not set in win2k */ - size = MAX_COMP_NAME; + if (size == 0) + size = MAX_COMP_NAME; name = HeapAlloc(GetProcessHeap(), 0, size * sizeof(name[0])); ok(name != NULL, "HeapAlloc failed with error %d\n", GetLastError()); ret = pGetComputerNameExA(ComputerNameNetBIOS, name, &size);

1 0

Rob Shearman : ntdll: Ignore the hole in the FILE_PIPE_WAIT_FOR_BUFFER structure passed into NtFsControlFile before writing the structure to the server .
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: 052873350df26c5add3a36a010b5910131bd0917 URL: http://source.winehq.org/git/wine.git/?a=commit;h=052873350df26c5add3a36a01… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Mon Dec 14 14:32:50 2009 +0000 ntdll: Ignore the hole in the FILE_PIPE_WAIT_FOR_BUFFER structure passed into NtFsControlFile before writing the structure to the server. --- dlls/ntdll/file.c | 29 +++++++++++++++++++++++++++++ 1 files changed, 29 insertions(+), 0 deletions(-) diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c index 00c54c2..b81d38c 100644 --- a/dlls/ntdll/file.c +++ b/dlls/ntdll/file.c @@ -69,6 +69,9 @@ #ifdef HAVE_SYS_STATFS_H # include <sys/statfs.h> #endif +#ifdef HAVE_VALGRIND_MEMCHECK_H +# include <valgrind/memcheck.h> +#endif #define NONAMELESSUNION #define NONAMELESSSTRUCT @@ -1257,6 +1260,30 @@ static NTSTATUS server_ioctl_file( HANDLE handle, HANDLE event, return status; } +/* Tell Valgrind to ignore any holes in structs we will be passing to the + * server */ +static void ignore_server_ioctl_struct_holes (ULONG code, const void *in_buffer, + ULONG in_size) +{ +#ifdef VALGRIND_MAKE_MEM_DEFINED +# define IGNORE_STRUCT_HOLE(buf, size, t, f1, f2) \ + do { \ + if ((size) >= FIELD_OFFSET(t, f2) && \ + FIELD_OFFSET(t, f1) + sizeof(((t *)0)->f1) < FIELD_OFFSET(t, f2)) \ + VALGRIND_MAKE_MEM_DEFINED( \ + (const char *)(buf) + FIELD_OFFSET(t, f1) + sizeof(((t *)0)->f1), \ + FIELD_OFFSET(t, f2) - FIELD_OFFSET(t, f1) + sizeof(((t *)0)->f1)); \ + } while (0) + + switch (code) + { + case FSCTL_PIPE_WAIT: + IGNORE_STRUCT_HOLE(in_buffer, in_size, FILE_PIPE_WAIT_FOR_BUFFER, TimeoutSpecified, Name); + break; + } +#endif +} + /************************************************************************** * NtDeviceIoControlFile [NTDLL.@] @@ -1356,6 +1383,8 @@ NTSTATUS WINAPI NtFsControlFile(HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc if (!io) return STATUS_INVALID_PARAMETER; + ignore_server_ioctl_struct_holes( code, in_buffer, in_size ); + switch(code) { case FSCTL_DISMOUNT_VOLUME:

1 0

Rob Shearman : cabinet: Fix memory leaks on failure in FDICopy.
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: eae5b071c23cf5a7c6285201f245faa14b73ef37 URL: http://source.winehq.org/git/wine.git/?a=commit;h=eae5b071c23cf5a7c6285201f… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Mon Dec 14 14:13:20 2009 +0000 cabinet: Fix memory leaks on failure in FDICopy. --- dlls/cabinet/fdi.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/dlls/cabinet/fdi.c b/dlls/cabinet/fdi.c index 61dafdb..de64f02 100644 --- a/dlls/cabinet/fdi.c +++ b/dlls/cabinet/fdi.c @@ -2519,6 +2519,7 @@ BOOL __cdecl FDICopy( /* slight overestimation here to save CPU cycles in the developer's brain */ if ((pathlen + filenamelen + 3) > MAX_PATH) { ERR("MAX_PATH exceeded.\n"); + PFDI_FREE(hfdi, decomp_state); PFDI_INT(hfdi)->perf->erfOper = FDIERROR_CABINET_NOT_FOUND; PFDI_INT(hfdi)->perf->erfType = ERROR_FILE_NOT_FOUND; PFDI_INT(hfdi)->perf->fError = TRUE; @@ -2540,6 +2541,7 @@ BOOL __cdecl FDICopy( /* get a handle to the cabfile */ cabhf = PFDI_OPEN(hfdi, fullpath, _O_RDONLY|_O_BINARY, _S_IREAD | _S_IWRITE); if (cabhf == -1) { + PFDI_FREE(hfdi, decomp_state); PFDI_INT(hfdi)->perf->erfOper = FDIERROR_CABINET_NOT_FOUND; PFDI_INT(hfdi)->perf->fError = TRUE; SetLastError(ERROR_FILE_NOT_FOUND); @@ -2548,6 +2550,7 @@ BOOL __cdecl FDICopy( if (cabhf == 0) { ERR("PFDI_OPEN returned zero for %s.\n", fullpath); + PFDI_FREE(hfdi, decomp_state); PFDI_INT(hfdi)->perf->erfOper = FDIERROR_CABINET_NOT_FOUND; PFDI_INT(hfdi)->perf->erfType = ERROR_FILE_NOT_FOUND; PFDI_INT(hfdi)->perf->fError = TRUE; @@ -2558,6 +2561,7 @@ BOOL __cdecl FDICopy( /* check if it's really a cabfile. Note that this doesn't implement the bug */ if (!FDI_read_entries(hfdi, cabhf, &fdici, &(CAB(mii)))) { ERR("FDIIsCabinet failed.\n"); + PFDI_FREE(hfdi, decomp_state); PFDI_CLOSE(hfdi, cabhf); return FALSE; }

1 0

Rob Shearman : msvcrt: Free memory allocated in TLS slot on module unload as well as thread exit.
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: c20868e0a2f7af909cf8af2877ae8b024fa6d11e URL: http://source.winehq.org/git/wine.git/?a=commit;h=c20868e0a2f7af909cf8af287… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Mon Dec 14 14:13:57 2009 +0000 msvcrt: Free memory allocated in TLS slot on module unload as well as thread exit. --- dlls/msvcrt/main.c | 27 +++++++++++++++------------ 1 files changed, 15 insertions(+), 12 deletions(-) diff --git a/dlls/msvcrt/main.c b/dlls/msvcrt/main.c index b42b64a..e64f0ba 100644 --- a/dlls/msvcrt/main.c +++ b/dlls/msvcrt/main.c @@ -60,13 +60,24 @@ static inline BOOL msvcrt_free_tls(void) return TRUE; } +static inline void msvcrt_free_tls_mem(void) +{ + thread_data_t *tls = TlsGetValue(msvcrt_tls_index); + if (tls) + { + HeapFree(GetProcessHeap(),0,tls->efcvt_buffer); + HeapFree(GetProcessHeap(),0,tls->asctime_buffer); + HeapFree(GetProcessHeap(),0,tls->wasctime_buffer); + HeapFree(GetProcessHeap(),0,tls->strerror_buffer); + } + HeapFree(GetProcessHeap(), 0, tls); +} + /********************************************************************* * Init */ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) { - thread_data_t *tls; - TRACE("(%p, %s, %p) pid(%x), tid(%x), tls(%u)\n", hinstDLL, msvcrt_get_reason(fdwReason), lpvReserved, GetCurrentProcessId(), GetCurrentThreadId(), @@ -94,21 +105,13 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) msvcrt_free_console(); msvcrt_free_args(); msvcrt_free_signals(); + msvcrt_free_tls_mem(); if (!msvcrt_free_tls()) return FALSE; TRACE("finished process free\n"); break; case DLL_THREAD_DETACH: - /* Free TLS */ - tls = TlsGetValue(msvcrt_tls_index); - if (tls) - { - HeapFree(GetProcessHeap(),0,tls->efcvt_buffer); - HeapFree(GetProcessHeap(),0,tls->asctime_buffer); - HeapFree(GetProcessHeap(),0,tls->wasctime_buffer); - HeapFree(GetProcessHeap(),0,tls->strerror_buffer); - } - HeapFree(GetProcessHeap(), 0, tls); + msvcrt_free_tls_mem(); TRACE("finished thread free\n"); break; }

1 0

Rob Shearman : gdi32: Fix memory leaks on EMF_Create_HENHMETAFILE failure.
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: eb81fa283b5efbf15dd9360bf1a95ac809374b80 URL: http://source.winehq.org/git/wine.git/?a=commit;h=eb81fa283b5efbf15dd9360bf… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Mon Dec 14 14:12:58 2009 +0000 gdi32: Fix memory leaks on EMF_Create_HENHMETAFILE failure. --- dlls/gdi32/enhmetafile.c | 10 +++++++++- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/dlls/gdi32/enhmetafile.c b/dlls/gdi32/enhmetafile.c index 95e5589..adb19a0 100644 --- a/dlls/gdi32/enhmetafile.c +++ b/dlls/gdi32/enhmetafile.c @@ -466,8 +466,12 @@ UINT WINAPI GetEnhMetaFileDescriptionW( HENHMETAFILE WINAPI SetEnhMetaFileBits(UINT bufsize, const BYTE *buf) { ENHMETAHEADER *emh = HeapAlloc( GetProcessHeap(), 0, bufsize ); + HENHMETAFILE hmf; memmove(emh, buf, bufsize); - return EMF_Create_HENHMETAFILE( emh, FALSE ); + hmf = EMF_Create_HENHMETAFILE( emh, FALSE ); + if (!hmf) + HeapFree( GetProcessHeap(), 0, emh ); + return hmf; } /***************************************************************************** @@ -2509,6 +2513,8 @@ HENHMETAFILE WINAPI CopyEnhMetaFileA( emrDst = HeapAlloc( GetProcessHeap(), 0, emrSrc->nBytes ); memcpy( emrDst, emrSrc, emrSrc->nBytes ); hmfDst = EMF_Create_HENHMETAFILE( emrDst, FALSE ); + if (!hmfDst) + HeapFree( GetProcessHeap(), 0, emrDst ); } else { HANDLE hFile; DWORD w; @@ -2549,6 +2555,8 @@ HENHMETAFILE WINAPI CopyEnhMetaFileW( emrDst = HeapAlloc( GetProcessHeap(), 0, emrSrc->nBytes ); memcpy( emrDst, emrSrc, emrSrc->nBytes ); hmfDst = EMF_Create_HENHMETAFILE( emrDst, FALSE ); + if (!hmfDst) + HeapFree( GetProcessHeap(), 0, emrDst ); } else { HANDLE hFile; DWORD w;

1 0

Alexandre Julliard : winex11: Make GetKeyboardLayoutName() use the same default layout as GetKeyboardLayout().
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: d18ad16331d1c68ab72a39f7bba77ca1b9a17cda URL: http://source.winehq.org/git/wine.git/?a=commit;h=d18ad16331d1c68ab72a39f7b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Dec 14 15:10:24 2009 +0100 winex11: Make GetKeyboardLayoutName() use the same default layout as GetKeyboardLayout(). --- dlls/winex11.drv/keyboard.c | 11 +++-------- 1 files changed, 3 insertions(+), 8 deletions(-) diff --git a/dlls/winex11.drv/keyboard.c b/dlls/winex11.drv/keyboard.c index 7cf0fdc..4d486b0 100644 --- a/dlls/winex11.drv/keyboard.c +++ b/dlls/winex11.drv/keyboard.c @@ -1635,16 +1635,11 @@ static HKL get_locale_kbd_layout(void) */ BOOL CDECL X11DRV_GetKeyboardLayoutName(LPWSTR name) { - static const WCHAR formatW[] = {'%','0','8','l','x',0}; + static const WCHAR formatW[] = {'%','0','8','x',0}; DWORD layout; - LANGID langid; - - layout = main_key_tab[kbd_layout].lcid; - /* see comment for get_locale_kbd_layout */ - langid = PRIMARYLANGID(LANGIDFROMLCID(layout)); - if (langid == LANG_CHINESE || langid == LANG_JAPANESE || langid == LANG_KOREAN) - layout |= 0xe001 << 16; /* FIXME */ + layout = HandleToUlong( get_locale_kbd_layout() ); + if (HIWORD(layout) == LOWORD(layout)) layout = LOWORD(layout); sprintfW(name, formatW, layout); TRACE("returning %s\n", debugstr_w(name)); return TRUE;

1 0

Alexandre Julliard : gdiplus/tests: Fix parentheses in the expect() macro.
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: 8d8544fa157ae92a61d7bf298a92551fe16f9227 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8d8544fa157ae92a61d7bf298… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Dec 14 14:55:05 2009 +0100 gdiplus/tests: Fix parentheses in the expect() macro. --- dlls/gdiplus/tests/image.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index 66c6410..2b08ef8 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -27,7 +27,7 @@ #include "gdiplus.h" #include "wine/test.h" -#define expect(expected, got) ok(((UINT)got) == ((UINT)expected), "Expected %.8x, got %.8x\n", (UINT)expected, (UINT)got) +#define expect(expected, got) ok((UINT)(got) == (UINT)(expected), "Expected %.8x, got %.8x\n", (UINT)(expected), (UINT)(got)) #define expectf(expected, got) ok(fabs(expected - got) < 0.0001, "Expected %.2f, got %.2f\n", expected, got) static void expect_rawformat(REFGUID expected, GpImage *img, int line, BOOL todo)

1 0

Paul Vriens : kernel32: Add a define for UNICODE_STRING_MAX_CHARS.
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: 4d6b74758f9ce967e87a9a1d0b67079898e8a5bb URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d6b74758f9ce967e87a9a1d0… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon Dec 14 13:58:24 2009 +0100 kernel32: Add a define for UNICODE_STRING_MAX_CHARS. --- dlls/kernel32/environ.c | 4 ++-- include/winnt.h | 2 ++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/environ.c b/dlls/kernel32/environ.c index aaa9d6e..b11f7bf 100644 --- a/dlls/kernel32/environ.c +++ b/dlls/kernel32/environ.c @@ -364,8 +364,8 @@ DWORD WINAPI ExpandEnvironmentStringsW( LPCWSTR src, LPWSTR dst, DWORD len ) RtlInitUnicodeString(&us_src, src); /* make sure we don't overflow the maximum UNICODE_STRING size */ - if (len > 0x7fff) - len = 0x7fff; + if (len > UNICODE_STRING_MAX_CHARS) + len = UNICODE_STRING_MAX_CHARS; us_dst.Length = 0; us_dst.MaximumLength = len * sizeof(WCHAR); diff --git a/include/winnt.h b/include/winnt.h index e2d1d1a..28409af 100644 --- a/include/winnt.h +++ b/include/winnt.h @@ -617,6 +617,8 @@ typedef struct _MEMORY_BASIC_INFORMATION #define MAXWORD 0xffff #define MAXDWORD 0xffffffff +#define UNICODE_STRING_MAX_CHARS 32767 + #define FIELD_OFFSET(type, field) \ ((LONG)(INT_PTR)&(((type *)0)->field))

1 0

Stefan Leichter : setupapi: Implement SetupGetInfFileListW.
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: 9db754fe10469ee558e32598a43c01589f1199c2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9db754fe10469ee558e32598a… Author: Stefan Leichter <Stefan.Leichter(a)camline.com> Date: Thu Dec 10 20:55:09 2009 +0100 setupapi: Implement SetupGetInfFileListW. --- dlls/setupapi/install.c | 131 ++++++++++++++++++++++++++++++++++++++++- dlls/setupapi/tests/install.c | 13 +---- 2 files changed, 129 insertions(+), 15 deletions(-) diff --git a/dlls/setupapi/install.c b/dlls/setupapi/install.c index 2064315..54eff1d 100644 --- a/dlls/setupapi/install.c +++ b/dlls/setupapi/install.c @@ -1454,8 +1454,133 @@ BOOL WINAPI SetupInstallServicesFromInfSectionA( HINF Inf, PCSTR SectionName, DW BOOL WINAPI SetupGetInfFileListW(PCWSTR dir, DWORD style, PWSTR buffer, DWORD insize, PDWORD outsize) { - FIXME("(%s %d %p %d %p) stub\n", debugstr_w(dir), style, buffer, insize, outsize); - if(buffer) buffer[0] = 0; - if(outsize) *outsize = 1; + static WCHAR inf[] = {'\\','*','.','i','n','f',0 }; + WCHAR *filter, *fullname = NULL, *ptr = buffer; + DWORD dir_len, name_len = 20, size ; + WIN32_FIND_DATAW finddata; + HANDLE hdl; + if (style & ~( INF_STYLE_OLDNT | INF_STYLE_WIN4 | + INF_STYLE_CACHE_ENABLE | INF_STYLE_CACHE_DISABLE )) + { + FIXME( "unknown inf_style(s) 0x%x\n", + style & ~( INF_STYLE_OLDNT | INF_STYLE_WIN4 | + INF_STYLE_CACHE_ENABLE | INF_STYLE_CACHE_DISABLE )); + if( outsize ) *outsize = 1; + return TRUE; + } + if ((style & ( INF_STYLE_OLDNT | INF_STYLE_WIN4 )) == INF_STYLE_NONE) + { + FIXME( "inf_style INF_STYLE_NONE not handled\n" ); + if( outsize ) *outsize = 1; + return TRUE; + } + if (style & ( INF_STYLE_CACHE_ENABLE | INF_STYLE_CACHE_DISABLE )) + FIXME("ignored inf_style(s) %s %s\n", + ( style & INF_STYLE_CACHE_ENABLE ) ? "INF_STYLE_CACHE_ENABLE" : "", + ( style & INF_STYLE_CACHE_DISABLE ) ? "INF_STYLE_CACHE_DISABLE" : ""); + if( dir ) + { + DWORD att; + DWORD msize; + dir_len = strlenW( dir ); + if ( !dir_len ) return FALSE; + msize = ( 7 + dir_len ) * sizeof( WCHAR ); /* \\*.inf\0 */ + filter = HeapAlloc( GetProcessHeap(), 0, msize ); + if( !filter ) + { + SetLastError( ERROR_NOT_ENOUGH_MEMORY ); + return FALSE; + } + strcpyW( filter, dir ); + if ( '\\' == filter[dir_len - 1] ) + filter[--dir_len] = 0; + + att = GetFileAttributesW( filter ); + if (att != INVALID_FILE_ATTRIBUTES && !(att & FILE_ATTRIBUTE_DIRECTORY)) + { + HeapFree( GetProcessHeap(), 0, filter ); + SetLastError( ERROR_DIRECTORY ); + return FALSE; + } + } + else + { + WCHAR infdir[] = {'\\','i','n','f',0 }; + DWORD msize; + dir_len = GetWindowsDirectoryW( NULL, 0 ); + msize = ( 7 + 4 + dir_len ) * sizeof( WCHAR ); + filter = HeapAlloc( GetProcessHeap(), 0, msize ); + if( !filter ) + { + SetLastError( ERROR_NOT_ENOUGH_MEMORY ); + return FALSE; + } + GetWindowsDirectoryW( filter, msize ); + strcatW( filter, infdir ); + } + strcatW( filter, inf ); + + hdl = FindFirstFileW( filter , &finddata ); + if ( hdl == INVALID_HANDLE_VALUE ) + { + if( outsize ) *outsize = 1; + HeapFree( GetProcessHeap(), 0, filter ); + return TRUE; + } + size = 1; + do + { + static const WCHAR key[] = + {'S','i','g','n','a','t','u','r','e',0 }; + static const WCHAR section[] = + {'V','e','r','s','i','o','n',0 }; + static const WCHAR sig_win4_1[] = + {'$','C','h','i','c','a','g','o','$',0 }; + static const WCHAR sig_win4_2[] = + {'$','W','I','N','D','O','W','S',' ','N','T','$',0 }; + WCHAR signature[ MAX_PATH ]; + BOOL valid = FALSE; + DWORD len = strlenW( finddata.cFileName ); + if (!fullname || ( name_len < len )) + { + name_len = ( name_len < len ) ? len : name_len; + HeapFree( GetProcessHeap(), 0, fullname ); + fullname = HeapAlloc( GetProcessHeap(), 0, + ( 2 + dir_len + name_len) * sizeof( WCHAR )); + if( !fullname ) + { + HeapFree( GetProcessHeap(), 0, filter ); + SetLastError( ERROR_NOT_ENOUGH_MEMORY ); + return FALSE; + } + strcpyW( fullname, filter ); + } + fullname[ dir_len + 1] = 0; /* keep '\\' */ + strcatW( fullname, finddata.cFileName ); + if (!GetPrivateProfileStringW( section, key, NULL, signature, MAX_PATH, fullname )) + signature[0] = 0; + if( INF_STYLE_OLDNT & style ) + valid = strcmpiW( sig_win4_1, signature ) && + strcmpiW( sig_win4_2, signature ); + if( INF_STYLE_WIN4 & style ) + valid = valid || !strcmpiW( sig_win4_1, signature ) || + !strcmpiW( sig_win4_2, signature ); + if( valid ) + { + size += 1 + strlenW( finddata.cFileName ); + if( ptr && insize >= size ) + { + strcpyW( ptr, finddata.cFileName ); + ptr += 1 + strlenW( finddata.cFileName ); + *ptr = 0; + } + } + } + while( FindNextFileW( hdl, &finddata )); + FindClose( hdl ); + + HeapFree( GetProcessHeap(), 0, fullname ); + HeapFree( GetProcessHeap(), 0, filter ); + if( outsize ) *outsize = size; return TRUE; } diff --git a/dlls/setupapi/tests/install.c b/dlls/setupapi/tests/install.c index 9aed163..668b2c2 100644 --- a/dlls/setupapi/tests/install.c +++ b/dlls/setupapi/tests/install.c @@ -515,7 +515,6 @@ static void test_inffilelist(void) outsize = 0; SetLastError(0xdeadbeef); ret = pSetupGetInfFileListW(dir, INF_STYLE_WIN4, NULL, 0, &outsize); - todo_wine ok(!ret, "expected SetupGetInfFileListW to fail!\n"); /* create a private directory, the temp directory may contain some @@ -528,7 +527,7 @@ static void test_inffilelist(void) } if (!CreateDirectoryA(dirA, NULL )) { - win_skip("CreateDirectoryA failed with error %d\n", GetLastError()); + win_skip("CreateDirectoryA(%s) failed with error %d\n", dirA, GetLastError()); return; } if (!SetCurrentDirectoryA(dirA)) @@ -548,7 +547,6 @@ static void test_inffilelist(void) ret = pSetupGetInfFileListW(dir, INF_STYLE_WIN4, NULL, 0, &outsize); ok(ret, "expected SetupGetInfFileListW to succeed!\n"); ok(outsize == 1, "expected required buffersize to be 1, got %d\n", outsize); - todo_wine ok(ERROR_PATH_NOT_FOUND == GetLastError(), "expected error ERROR_PATH_NOT_FOUND, got %d\n", GetLastError()); @@ -563,9 +561,7 @@ static void test_inffilelist(void) outsize = 0xffffffff; SetLastError(0xdeadbeef); ret = pSetupGetInfFileListW(dir, INF_STYLE_WIN4, NULL, 0, &outsize); - todo_wine ok(!ret, "expected SetupGetInfFileListW to fail!\n"); - todo_wine ok(ERROR_DIRECTORY == GetLastError(), "expected error ERROR_DIRECTORY, got %d\n", GetLastError()); @@ -575,9 +571,7 @@ static void test_inffilelist(void) dir[lstrlenW(dir)] = '\\'; SetLastError(0xdeadbeef); ret = pSetupGetInfFileListW(dir, INF_STYLE_WIN4, NULL, 0, &outsize); - todo_wine ok(!ret, "expected SetupGetInfFileListW to fail!\n"); - todo_wine ok(ERROR_DIRECTORY == GetLastError(), "expected error ERROR_DIRECTORY, got %d\n", GetLastError()); @@ -587,7 +581,6 @@ static void test_inffilelist(void) expected = 3 + strlen(inffile) + strlen(inffile2); ret = pSetupGetInfFileListW(dir, INF_STYLE_WIN4, buffer, MAX_PATH, &outsize); ok(ret, "expected SetupGetInfFileListW to succeed!\n"); - todo_wine ok(expected == outsize, "expected required buffersize to be %d, got %d\n", expected, outsize); for(p = buffer; lstrlenW(p) && (outsize > (p - buffer)); p+=lstrlenW(p) + 1) @@ -599,7 +592,6 @@ static void test_inffilelist(void) create_inf_file(inffile2, inf2); ret = pSetupGetInfFileListW(dir, INF_STYLE_WIN4, buffer, MAX_PATH, &outsize); ok(ret, "expected SetupGetInfFileListW to succeed!\n"); - todo_wine ok(expected == outsize, "expected required buffersize to be %d, got %d\n", expected, outsize); for(p = buffer; lstrlenW(p) && (outsize > (p - buffer)); p+=lstrlenW(p) + 1) @@ -612,7 +604,6 @@ static void test_inffilelist(void) expected = 3 + strlen(inffile) + strlen(inffile2); ret = pSetupGetInfFileListW(dir, INF_STYLE_WIN4, buffer, MAX_PATH, &outsize); ok(ret, "expected SetupGetInfFileListW to succeed!\n"); - todo_wine ok(expected == outsize, "expected required buffersize to be %d, got %d\n", expected, outsize); for(p = buffer; lstrlenW(p) && (outsize > (p - buffer)); p+=lstrlenW(p) + 1) @@ -624,7 +615,6 @@ static void test_inffilelist(void) expected = 2 + strlen(invalid_inf); ret = pSetupGetInfFileListW(dir, INF_STYLE_OLDNT, buffer, MAX_PATH, &outsize); ok(ret, "expected SetupGetInfFileListW to succeed!\n"); - todo_wine ok(expected == outsize, "expected required buffersize to be %d, got %d\n", expected, outsize); for(p = buffer; lstrlenW(p) && (outsize > (p - buffer)); p+=lstrlenW(p) + 1) @@ -636,7 +626,6 @@ static void test_inffilelist(void) ret = pSetupGetInfFileListW(dir, INF_STYLE_OLDNT | INF_STYLE_WIN4, buffer, MAX_PATH, &outsize); ok(ret, "expected SetupGetInfFileListW to succeed!\n"); - todo_wine ok(expected == outsize, "expected required buffersize to be %d, got %d\n", expected, outsize); for(p = buffer; lstrlenW(p) && (outsize > (p - buffer)); p+=lstrlenW(p) + 1)

1 0

Hans Leidekker : rpcrt4: Add tests for RPC calls with authentication.
by Alexandre Julliard 14 Dec '09

14 Dec '09

Module: wine Branch: master Commit: fddabccaa1daae17f5fde0060d02d36b80acb080 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fddabccaa1daae17f5fde0060… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon Dec 14 11:54:36 2009 +0100 rpcrt4: Add tests for RPC calls with authentication. --- dlls/rpcrt4/tests/Makefile.in | 2 +- dlls/rpcrt4/tests/server.c | 159 ++++++++++++++++++++++++++++++++++++----- dlls/rpcrt4/tests/server.idl | 2 + 3 files changed, 145 insertions(+), 18 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=fddabccaa1daae17f5fde…

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits December 2009