September 2007 - Wine-commits

Juan Lang : setupapi: Implement SetupDiGetClassDevsA on top of SetupDiGetClassDevsExW.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: 1b023ff69d35d1008f83cef79a3f37802c7ff0d8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b023ff69d35d1008f83cef79… Author: Juan Lang <juan.lang(a)gmail.com> Date: Mon Sep 24 13:02:58 2007 -0700 setupapi: Implement SetupDiGetClassDevsA on top of SetupDiGetClassDevsExW. --- dlls/setupapi/devinst.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 5faeaa0..503eaa5 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -1761,7 +1761,8 @@ HDEVINFO WINAPI SetupDiGetClassDevsA( } MultiByteToWideChar(CP_ACP, 0, enumstr, -1, enumstrW, len); } - ret = SetupDiGetClassDevsW(class, enumstrW, parent, flags); + ret = SetupDiGetClassDevsExW(class, enumstrW, parent, flags, NULL, NULL, + NULL); HeapFree(GetProcessHeap(), 0, enumstrW); end:

1 0

Juan Lang : setupapi: Implement SetupDiGetClassDevsExA.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: 06547f129f44fcebc99801c8c5ea90d80d6a16a2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=06547f129f44fcebc99801c8c… Author: Juan Lang <juan.lang(a)gmail.com> Date: Mon Sep 24 13:32:46 2007 -0700 setupapi: Implement SetupDiGetClassDevsExA. --- dlls/setupapi/devinst.c | 47 +++++++++++++++++++++++++++++++++++++++++++++++ dlls/setupapi/stubs.c | 11 ----------- 2 files changed, 47 insertions(+), 11 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 503eaa5..2459ccd 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -1769,6 +1769,53 @@ end: return ret; } +/*********************************************************************** + * SetupDiGetClassDevsExA (SETUPAPI.@) + */ +HDEVINFO WINAPI SetupDiGetClassDevsExA( + const GUID *class, + PCSTR enumstr, + HWND parent, + DWORD flags, + HDEVINFO deviceset, + PCSTR machine, + PVOID reserved) +{ + HDEVINFO ret; + LPWSTR enumstrW = NULL, machineW = NULL; + + if (enumstr) + { + int len = MultiByteToWideChar(CP_ACP, 0, enumstr, -1, NULL, 0); + enumstrW = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!enumstrW) + { + ret = (HDEVINFO)INVALID_HANDLE_VALUE; + goto end; + } + MultiByteToWideChar(CP_ACP, 0, enumstr, -1, enumstrW, len); + } + if (machine) + { + int len = MultiByteToWideChar(CP_ACP, 0, machine, -1, NULL, 0); + machineW = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!machineW) + { + HeapFree(GetProcessHeap(), 0, enumstrW); + ret = (HDEVINFO)INVALID_HANDLE_VALUE; + goto end; + } + MultiByteToWideChar(CP_ACP, 0, machine, -1, machineW, len); + } + ret = SetupDiGetClassDevsExW(class, enumstrW, parent, flags, deviceset, + machineW, reserved); + HeapFree(GetProcessHeap(), 0, enumstrW); + HeapFree(GetProcessHeap(), 0, machineW); + +end: + return ret; +} + static void SETUPDI_AddDeviceInterfaces(SP_DEVINFO_DATA *dev, HKEY key, const GUID *interface) { diff --git a/dlls/setupapi/stubs.c b/dlls/setupapi/stubs.c index 3342a65..de1fec1 100644 --- a/dlls/setupapi/stubs.c +++ b/dlls/setupapi/stubs.c @@ -50,17 +50,6 @@ DWORD WINAPI suErrorToIds16( WORD w1, WORD w2 ) } /*********************************************************************** - * (SETUPAPI.@) - * - * NO WINAPI in description given - */ -HDEVINFO WINAPI SetupDiGetClassDevsExA(const GUID *class, PCSTR filter, HWND parent, DWORD flags, HDEVINFO deviceset, PCSTR machine, PVOID reserved) -{ - FIXME("filter %s machine %s\n",debugstr_a(filter),debugstr_a(machine)); - return FALSE; -} - -/*********************************************************************** * CM_Connect_MachineW (SETUPAPI.@) */ DWORD WINAPI CM_Connect_MachineW(LPCWSTR name, void * machine)

1 0

Stefan Dösinger : wined3d: Put V8U8 back into the UNKNOWN format group.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: bf9cc30aacf84798422b2ba2862a889d2128f403 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bf9cc30aacf84798422b2ba28… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon Sep 24 16:16:31 2007 +0200 wined3d: Put V8U8 back into the UNKNOWN format group. --- dlls/wined3d/arb_program_shader.c | 7 ++++++- dlls/wined3d/utils.c | 33 ++++++++++++++++++++++++--------- 2 files changed, 30 insertions(+), 10 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index 4d404d6..3ceffe7 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -598,10 +598,15 @@ static void shader_arb_color_correction(SHADER_OPCODE_ARG* arg) { case WINED3DFMT_V16U16: if(GL_SUPPORT(NV_TEXTURE_SHADER) || (GL_SUPPORT(ATI_ENVMAP_BUMPMAP) && fmt == WINED3DFMT_V8U8)) { - /* The 3rd channel returns 1.0 in d3d, but 0.0 in gl. Fix this while we're at it :-) */ +#if 0 + /* The 3rd channel returns 1.0 in d3d, but 0.0 in gl. Fix this while we're at it :-) + * disabled until an application that needs it is found because it causes unneeded + * shader recompilation in some game + */ if(strlen(writemask) >= 4) { shader_addline(arg->buffer, "MOV %s.%c, one.z;\n", reg, writemask[3]); } +#endif } else { /* Correct the sign, but leave the blue as it is - it was loaded correctly already * ARB shaders are a bit picky wrt writemasks and swizzles. If we're free to scale diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index 4765ae5..bd1d7d3 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -230,18 +230,33 @@ BOOL initPixelFormats(WineD3D_GL_Info *gl_info) gl_info->gl_formats[dst].conversion_group= WINED3DFMT_UNKNOWN; } - /* V8U8 and V16U16 are always tidied up in the pixel shader - blue is set to 1.0. - * They can't be switched with other formats, but they can be switched with each other, - * except if GL_ATI_envmap_bumpmap is supported. In this case, V8U8 uses the gl native format, - * but V16U16 is converted. + /* V8U8 is supported natively by GL_ATI_envmap_bumpmap and GL_NV_texture_shader. + * V16U16 is only supported by GL_NV_texture_shader. The formats need fixup if + * their extensions are not available. + * + * In theory, V8U8 and V16U16 need a fixup of the undefined blue channel. OpenGL + * returns 0.0 when sampling from it, DirectX 1.0. This is disabled until we find + * an application that needs this because it causes performance problems due to + * shader recompiling in some games. */ - dst = getFmtIdx(WINED3DFMT_V8U8); - gl_info->gl_formats[dst].conversion_group = WINED3DFMT_V8U8; - dst = getFmtIdx(WINED3DFMT_V16U16); - if(!GL_SUPPORT(ATI_ENVMAP_BUMPMAP)) { + if(!GL_SUPPORT(ATI_ENVMAP_BUMPMAP) && !GL_SUPPORT(NV_TEXTURE_SHADER2)) { + /* signed -> unsigned fixup */ + dst = getFmtIdx(WINED3DFMT_V8U8); gl_info->gl_formats[dst].conversion_group = WINED3DFMT_V8U8; - } else { + dst = getFmtIdx(WINED3DFMT_V16U16); + gl_info->gl_formats[dst].conversion_group = WINED3DFMT_V8U8; + } else if(GL_SUPPORT(ATI_ENVMAP_BUMPMAP)) { + /* signed -> unsigned fixup */ + dst = getFmtIdx(WINED3DFMT_V16U16); gl_info->gl_formats[dst].conversion_group = WINED3DFMT_V16U16; + } else { + /* Blue = 1.0 fixup, disabled for now */ +#if 0 + dst = getFmtIdx(WINED3DFMT_V8U8); + gl_info->gl_formats[dst].conversion_group = WINED3DFMT_V8U8; + dst = getFmtIdx(WINED3DFMT_V16U16); + gl_info->gl_formats[dst].conversion_group = WINED3DFMT_V8U8; +#endif } if(!GL_SUPPORT(NV_TEXTURE_SHADER)) {

1 0

Juan Lang : setupapi: Implement SetupDiGetDeviceInfoListDetailA/W.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: 9ac11e38647c9844adfaa575f968881b2c8169c4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9ac11e38647c9844adfaa575f… Author: Juan Lang <juan.lang(a)gmail.com> Date: Mon Sep 24 12:45:42 2007 -0700 setupapi: Implement SetupDiGetDeviceInfoListDetailA/W. --- dlls/setupapi/devinst.c | 66 +++++++++++++++++++++++++++++++++++++++++++++++ dlls/setupapi/stubs.c | 18 ------------- 2 files changed, 66 insertions(+), 18 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index a5356e5..85ef0ac 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -2103,6 +2103,72 @@ HDEVINFO WINAPI SetupDiGetClassDevsExW( } /*********************************************************************** + * SetupDiGetDeviceInfoListDetailA (SETUPAPI.@) + */ +BOOL WINAPI SetupDiGetDeviceInfoListDetailA( + HDEVINFO DeviceInfoSet, + PSP_DEVINFO_LIST_DETAIL_DATA_A DevInfoData ) +{ + struct DeviceInfoSet *set = (struct DeviceInfoSet *)DeviceInfoSet; + + TRACE("%p %p\n", DeviceInfoSet, DevInfoData); + + if (!DeviceInfoSet || DeviceInfoSet == (HDEVINFO)INVALID_HANDLE_VALUE) + { + SetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + if (set->magic != SETUP_DEVICE_INFO_SET_MAGIC) + { + SetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + if (!DevInfoData || + DevInfoData->cbSize != sizeof(SP_DEVINFO_LIST_DETAIL_DATA_A)) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } + memcpy(&DevInfoData->ClassGuid, &set->ClassGuid, sizeof(GUID)); + DevInfoData->RemoteMachineHandle = NULL; + DevInfoData->RemoteMachineName[0] = '\0'; + return TRUE; +} + +/*********************************************************************** + * SetupDiGetDeviceInfoListDetailW (SETUPAPI.@) + */ +BOOL WINAPI SetupDiGetDeviceInfoListDetailW( + HDEVINFO DeviceInfoSet, + PSP_DEVINFO_LIST_DETAIL_DATA_W DevInfoData ) +{ + struct DeviceInfoSet *set = (struct DeviceInfoSet *)DeviceInfoSet; + + TRACE("%p %p\n", DeviceInfoSet, DevInfoData); + + if (!DeviceInfoSet || DeviceInfoSet == (HDEVINFO)INVALID_HANDLE_VALUE) + { + SetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + if (set->magic != SETUP_DEVICE_INFO_SET_MAGIC) + { + SetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + if (!DevInfoData || + DevInfoData->cbSize != sizeof(SP_DEVINFO_LIST_DETAIL_DATA_W)) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } + memcpy(&DevInfoData->ClassGuid, &set->ClassGuid, sizeof(GUID)); + DevInfoData->RemoteMachineHandle = NULL; + DevInfoData->RemoteMachineName[0] = '\0'; + return TRUE; +} + +/*********************************************************************** * SetupDiCreateDeviceInterfaceA (SETUPAPI.@) */ BOOL WINAPI SetupDiCreateDeviceInterfaceA( diff --git a/dlls/setupapi/stubs.c b/dlls/setupapi/stubs.c index bf3fa90..3342a65 100644 --- a/dlls/setupapi/stubs.c +++ b/dlls/setupapi/stubs.c @@ -50,24 +50,6 @@ DWORD WINAPI suErrorToIds16( WORD w1, WORD w2 ) } /*********************************************************************** - * SetupDiGetDeviceInfoListDetailA (SETUPAPI.@) - */ -BOOL WINAPI SetupDiGetDeviceInfoListDetailA(HDEVINFO devinfo, PSP_DEVINFO_LIST_DETAIL_DATA_A devinfo_data ) -{ - FIXME("\n"); - return FALSE; -} - -/*********************************************************************** - * SetupDiGetDeviceInfoListDetailW (SETUPAPI.@) - */ -BOOL WINAPI SetupDiGetDeviceInfoListDetailW(HDEVINFO devinfo, PSP_DEVINFO_LIST_DETAIL_DATA_W devinfo_data ) -{ - FIXME("\n"); - return FALSE; -} - -/*********************************************************************** * (SETUPAPI.@) * * NO WINAPI in description given

1 0

Juan Lang : setupapi: Partially implement SetupDiGetClassDevsExW.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: 4e06874117497e28006d6d91861054d9e952b9a3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e06874117497e28006d6d918… Author: Juan Lang <juan.lang(a)gmail.com> Date: Mon Sep 24 13:00:42 2007 -0700 setupapi: Partially implement SetupDiGetClassDevsExW. --- dlls/setupapi/devinst.c | 32 ++++++++++++++++++++++++++++++-- 1 files changed, 30 insertions(+), 2 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 85ef0ac..0df3b70 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -2098,8 +2098,36 @@ HDEVINFO WINAPI SetupDiGetClassDevsExW( PCWSTR machine, PVOID reserved) { - FIXME("stub\n"); - return NULL; + static const DWORD unsupportedFlags = DIGCF_DEFAULT | DIGCF_PRESENT | + DIGCF_PROFILE; + HDEVINFO set; + + TRACE("%s %s %p 0x%08x %p %s %p\n", debugstr_guid(class), + debugstr_w(enumstr), parent, flags, deviceset, debugstr_w(machine), + reserved); + + if (!(flags & DIGCF_ALLCLASSES) && !class) + { + SetLastError(ERROR_INVALID_PARAMETER); + return NULL; + } + if (flags & unsupportedFlags) + WARN("unsupported flags %08x\n", flags & unsupportedFlags); + if (deviceset) + set = deviceset; + else + set = SetupDiCreateDeviceInfoListExW(class, parent, machine, reserved); + if (set) + { + if (machine) + FIXME("%s: unimplemented for remote machines\n", + debugstr_w(machine)); + else if (flags & DIGCF_DEVICEINTERFACE) + SETUPDI_EnumerateInterfaces(set, class, enumstr, flags); + else + SETUPDI_EnumerateDevices(set, class, enumstr, flags); + } + return set; } /***********************************************************************

1 0

Stefan Leichter : ddrawex: Remove 'recursive registry key delete' function.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: eb77d12b2af92c82b8fa367352cefc665bec329d URL: http://source.winehq.org/git/wine.git/?a=commit;h=eb77d12b2af92c82b8fa36735… Author: Stefan Leichter <Stefan.Leichter(a)camline.com> Date: Tue Sep 25 09:56:47 2007 +0200 ddrawex: Remove 'recursive registry key delete' function. --- dlls/ddrawex/regsvr.c | 73 +++---------------------------------------------- 1 files changed, 4 insertions(+), 69 deletions(-) diff --git a/dlls/ddrawex/regsvr.c b/dlls/ddrawex/regsvr.c index 0e73abc..5467f6b 100644 --- a/dlls/ddrawex/regsvr.c +++ b/dlls/ddrawex/regsvr.c @@ -94,9 +94,6 @@ static LONG register_key_defvalueW(HKEY base, WCHAR const *name, WCHAR const *value); static LONG register_key_defvalueA(HKEY base, WCHAR const *name, char const *value); -static LONG recursive_delete_key(HKEY key); -static LONG recursive_delete_keyA(HKEY base, char const *name); -static LONG recursive_delete_keyW(HKEY base, WCHAR const *name); /*********************************************************************** * register_coclasses @@ -200,11 +197,13 @@ static HRESULT unregister_coclasses(struct regsvr_coclass const *list) WCHAR buf[39]; StringFromGUID2(list->clsid, buf, 39); - res = recursive_delete_keyW(coclass_key, buf); + res = RegDeleteTreeW(coclass_key, buf); + if (res == ERROR_FILE_NOT_FOUND) res = ERROR_SUCCESS; if (res != ERROR_SUCCESS) goto error_close_coclass_key; if (list->progid) { - res = recursive_delete_keyA(HKEY_CLASSES_ROOT, list->progid); + res = RegDeleteTreeA(HKEY_CLASSES_ROOT, list->progid); + if (res == ERROR_FILE_NOT_FOUND) res = ERROR_SUCCESS; if (res != ERROR_SUCCESS) goto error_close_coclass_key; } } @@ -256,70 +255,6 @@ static LONG register_key_defvalueA( } /*********************************************************************** - * recursive_delete_key - */ -static LONG recursive_delete_key(HKEY key) -{ - LONG res; - WCHAR subkey_name[MAX_PATH]; - DWORD cName; - HKEY subkey; - - for (;;) { - cName = sizeof(subkey_name) / sizeof(WCHAR); - res = RegEnumKeyExW(key, 0, subkey_name, &cName, - NULL, NULL, NULL, NULL); - if (res != ERROR_SUCCESS && res != ERROR_MORE_DATA) { - res = ERROR_SUCCESS; /* presumably we're done enumerating */ - break; - } - res = RegOpenKeyExW(key, subkey_name, 0, - KEY_READ | KEY_WRITE, &subkey); - if (res == ERROR_FILE_NOT_FOUND) continue; - if (res != ERROR_SUCCESS) break; - - res = recursive_delete_key(subkey); - RegCloseKey(subkey); - if (res != ERROR_SUCCESS) break; - } - - if (res == ERROR_SUCCESS) res = RegDeleteKeyW(key, 0); - return res; -} - -/*********************************************************************** - * recursive_delete_keyA - */ -static LONG recursive_delete_keyA(HKEY base, char const *name) -{ - LONG res; - HKEY key; - - res = RegOpenKeyExA(base, name, 0, KEY_READ | KEY_WRITE, &key); - if (res == ERROR_FILE_NOT_FOUND) return ERROR_SUCCESS; - if (res != ERROR_SUCCESS) return res; - res = recursive_delete_key(key); - RegCloseKey(key); - return res; -} - -/*********************************************************************** - * recursive_delete_keyW - */ -static LONG recursive_delete_keyW(HKEY base, WCHAR const *name) -{ - LONG res; - HKEY key; - - res = RegOpenKeyExW(base, name, 0, KEY_READ | KEY_WRITE, &key); - if (res == ERROR_FILE_NOT_FOUND) return ERROR_SUCCESS; - if (res != ERROR_SUCCESS) return res; - res = recursive_delete_key(key); - RegCloseKey(key); - return res; -} - -/*********************************************************************** * coclass list */

1 0

Stefan Leichter : ddraw: Remove 'recursive registry key delete' function.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: b78736a4bbf8fbef7db4afaff5435fa4a9578cfb URL: http://source.winehq.org/git/wine.git/?a=commit;h=b78736a4bbf8fbef7db4afaff… Author: Stefan Leichter <Stefan.Leichter(a)camline.com> Date: Tue Sep 25 10:07:49 2007 +0200 ddraw: Remove 'recursive registry key delete' function. --- dlls/ddraw/regsvr.c | 76 ++++----------------------------------------------- 1 files changed, 6 insertions(+), 70 deletions(-) diff --git a/dlls/ddraw/regsvr.c b/dlls/ddraw/regsvr.c index 877c8a2..4371e62 100644 --- a/dlls/ddraw/regsvr.c +++ b/dlls/ddraw/regsvr.c @@ -105,9 +105,6 @@ static LONG register_key_defvalueW(HKEY base, WCHAR const *name, WCHAR const *value); static LONG register_key_defvalueA(HKEY base, WCHAR const *name, char const *value); -static LONG recursive_delete_key(HKEY key); -static LONG recursive_delete_keyA(HKEY base, char const *name); -static LONG recursive_delete_keyW(HKEY base, WCHAR const *name); /*********************************************************************** * register_interfaces @@ -196,7 +193,8 @@ static HRESULT unregister_interfaces(struct regsvr_interface const *list) WCHAR buf[39]; StringFromGUID2(list->iid, buf, 39); - res = recursive_delete_keyW(interface_key, buf); + res = RegDeleteTreeW(interface_key, buf); + if (res == ERROR_FILE_NOT_FOUND) res = ERROR_SUCCESS; } RegCloseKey(interface_key); @@ -306,11 +304,13 @@ static HRESULT unregister_coclasses(struct regsvr_coclass const *list) WCHAR buf[39]; StringFromGUID2(list->clsid, buf, 39); - res = recursive_delete_keyW(coclass_key, buf); + res = RegDeleteTreeW(coclass_key, buf); + if (res == ERROR_FILE_NOT_FOUND) res = ERROR_SUCCESS; if (res != ERROR_SUCCESS) goto error_close_coclass_key; if (list->progid) { - res = recursive_delete_keyA(HKEY_CLASSES_ROOT, list->progid); + res = RegDeleteTreeA(HKEY_CLASSES_ROOT, list->progid); + if (res == ERROR_FILE_NOT_FOUND) res = ERROR_SUCCESS; if (res != ERROR_SUCCESS) goto error_close_coclass_key; } } @@ -373,70 +373,6 @@ static LONG register_key_defvalueA( } /*********************************************************************** - * recursive_delete_key - */ -static LONG recursive_delete_key(HKEY key) -{ - LONG res; - WCHAR subkey_name[MAX_PATH]; - DWORD cName; - HKEY subkey; - - for (;;) { - cName = sizeof(subkey_name) / sizeof(WCHAR); - res = RegEnumKeyExW(key, 0, subkey_name, &cName, - NULL, NULL, NULL, NULL); - if (res != ERROR_SUCCESS && res != ERROR_MORE_DATA) { - res = ERROR_SUCCESS; /* presumably we're done enumerating */ - break; - } - res = RegOpenKeyExW(key, subkey_name, 0, - KEY_READ | KEY_WRITE, &subkey); - if (res == ERROR_FILE_NOT_FOUND) continue; - if (res != ERROR_SUCCESS) break; - - res = recursive_delete_key(subkey); - RegCloseKey(subkey); - if (res != ERROR_SUCCESS) break; - } - - if (res == ERROR_SUCCESS) res = RegDeleteKeyW(key, 0); - return res; -} - -/*********************************************************************** - * recursive_delete_keyA - */ -static LONG recursive_delete_keyA(HKEY base, char const *name) -{ - LONG res; - HKEY key; - - res = RegOpenKeyExA(base, name, 0, KEY_READ | KEY_WRITE, &key); - if (res == ERROR_FILE_NOT_FOUND) return ERROR_SUCCESS; - if (res != ERROR_SUCCESS) return res; - res = recursive_delete_key(key); - RegCloseKey(key); - return res; -} - -/*********************************************************************** - * recursive_delete_keyW - */ -static LONG recursive_delete_keyW(HKEY base, WCHAR const *name) -{ - LONG res; - HKEY key; - - res = RegOpenKeyExW(base, name, 0, KEY_READ | KEY_WRITE, &key); - if (res == ERROR_FILE_NOT_FOUND) return ERROR_SUCCESS; - if (res != ERROR_SUCCESS) return res; - res = recursive_delete_key(key); - RegCloseKey(key); - return res; -} - -/*********************************************************************** * coclass list */

1 0

Andrew Talbot : gdi32: Fix a memory leak.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: ce0e1706fe8aada6be3444b6de3457cb4be1dbe2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ce0e1706fe8aada6be3444b6d… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Mon Sep 24 21:13:57 2007 +0100 gdi32: Fix a memory leak. --- dlls/gdi32/painting.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdi32/painting.c b/dlls/gdi32/painting.c index 72d8183..77d05c2 100644 --- a/dlls/gdi32/painting.c +++ b/dlls/gdi32/painting.c @@ -970,7 +970,7 @@ BOOL WINAPI PolyDraw(HDC hdc, const POINT *lppt, const BYTE *lpbTypes, Polyline(hdc, line_pts, num_pts); MoveToEx(hdc, line_pts[num_pts - 1].x, line_pts[num_pts - 1].y, NULL); - + HeapFree(GetProcessHeap(), 0, line_pts); result = TRUE; }

1 0

Michael Stefaniuc : ole32: Free memory on error path (found by Smatch).
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: dbfa8eedf79cd7fa54917a4ad6f5a4520b97c87d URL: http://source.winehq.org/git/wine.git/?a=commit;h=dbfa8eedf79cd7fa54917a4ad… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Sep 25 01:15:07 2007 +0200 ole32: Free memory on error path (found by Smatch). --- dlls/ole32/oleproxy.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/oleproxy.c b/dlls/ole32/oleproxy.c index e807941..42dc8d1 100644 --- a/dlls/ole32/oleproxy.c +++ b/dlls/ole32/oleproxy.c @@ -402,9 +402,10 @@ static HRESULT WINAPI CFProxy_CreateInstance( hGlobal = GlobalAlloc(GMEM_MOVEABLE|GMEM_NODISCARD|GMEM_SHARE,msg.cbBuffer); memcpy(GlobalLock(hGlobal),msg.Buffer,msg.cbBuffer); hres = CreateStreamOnHGlobal(hGlobal,TRUE,&pStream); - if (hres) { + if (hres != S_OK) { FIXME("CreateStreamOnHGlobal failed with %x\n",hres); IRpcChannelBuffer_FreeBuffer(This->chanbuf,&msg); + GlobalFree(hGlobal); return hres; } hres = IStream_Read(pStream, ppv, sizeof(*ppv), NULL);

1 0

Mikolaj Zalewski : advapi32: Fix a test.
by Alexandre Julliard 25 Sep '07

25 Sep '07

Module: wine Branch: master Commit: 1122dc130a447c190fc02508432772ab75db3bad URL: http://source.winehq.org/git/wine.git/?a=commit;h=1122dc130a447c190fc025084… Author: Mikolaj Zalewski <mikolajz(a)google.com> Date: Mon Sep 24 16:56:32 2007 -0700 advapi32: Fix a test. --- dlls/advapi32/tests/service.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/advapi32/tests/service.c b/dlls/advapi32/tests/service.c index 7aea906..bfb535d 100644 --- a/dlls/advapi32/tests/service.c +++ b/dlls/advapi32/tests/service.c @@ -727,7 +727,7 @@ static void test_sequence(void) static const CHAR servicename [] = "Winetest"; static const CHAR displayname [] = "Winetest dummy service"; static const CHAR pathname [] = "we_dont_care.exe"; - static const CHAR dependencies[] = "Master1\0Master2\0+MasterGroup1\0\0"; + static const CHAR dependencies[] = "Master1\0Master2\0+MasterGroup1\0"; static const CHAR password [] = ""; static const CHAR empty [] = ""; static const CHAR localsystem [] = "LocalSystem";

1 0