April 2010 - Wine-commits - list.winehq.org

Christian Costa : d3dx9_36: Improve FIXMEs and TRACEs for sprite.
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: c1f6857636849f525fc02db2336ae173130411dd URL: http://source.winehq.org/git/wine.git/?a=commit;h=c1f6857636849f525fc02db23… Author: Christian Costa <titan.costa(a)wanadoo.fr> Date: Fri Apr 2 08:54:48 2010 +0200 d3dx9_36: Improve FIXMEs and TRACEs for sprite. --- dlls/d3dx9_36/sprite.c | 34 +++++++++++++++++++++------------- 1 files changed, 21 insertions(+), 13 deletions(-) diff --git a/dlls/d3dx9_36/sprite.c b/dlls/d3dx9_36/sprite.c index ce9938e..1a0038e 100644 --- a/dlls/d3dx9_36/sprite.c +++ b/dlls/d3dx9_36/sprite.c @@ -49,7 +49,7 @@ static ULONG WINAPI ID3DXSpriteImpl_AddRef(LPD3DXSPRITE iface) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; ULONG ref=InterlockedIncrement(&This->ref); - TRACE("(%p): AddRef from %d\n", This, ref-1); + TRACE("(%p)->(): AddRef from %d\n", This, ref-1); return ref; } @@ -57,7 +57,8 @@ static ULONG WINAPI ID3DXSpriteImpl_Release(LPD3DXSPRITE iface) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; ULONG ref=InterlockedDecrement(&This->ref); - TRACE("(%p): ReleaseRef to %d\n", This, ref); + + TRACE("(%p)->(): ReleaseRef to %d\n", This, ref); if(ref==0) { if(This->sprites) { @@ -79,7 +80,8 @@ static ULONG WINAPI ID3DXSpriteImpl_Release(LPD3DXSPRITE iface) static HRESULT WINAPI ID3DXSpriteImpl_GetDevice(LPD3DXSPRITE iface, LPDIRECT3DDEVICE9 *device) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; - TRACE("(%p): relay\n", This); + + TRACE("(%p)->(%p): relay\n", This, device); if(device==NULL) return D3DERR_INVALIDCALL; *device=This->device; @@ -91,7 +93,8 @@ static HRESULT WINAPI ID3DXSpriteImpl_GetDevice(LPD3DXSPRITE iface, LPDIRECT3DDE static HRESULT WINAPI ID3DXSpriteImpl_GetTransform(LPD3DXSPRITE iface, D3DXMATRIX *transform) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; - TRACE("(%p)\n", This); + + TRACE("(%p)->(%p)\n", This, transform); if(transform==NULL) return D3DERR_INVALIDCALL; *transform=This->transform; @@ -102,7 +105,8 @@ static HRESULT WINAPI ID3DXSpriteImpl_GetTransform(LPD3DXSPRITE iface, D3DXMATRI static HRESULT WINAPI ID3DXSpriteImpl_SetTransform(LPD3DXSPRITE iface, CONST D3DXMATRIX *transform) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; - TRACE("(%p)\n", This); + + TRACE("(%p)->(%p)\n", This, transform); if(transform==NULL) return D3DERR_INVALIDCALL; This->transform=*transform; @@ -113,14 +117,14 @@ static HRESULT WINAPI ID3DXSpriteImpl_SetTransform(LPD3DXSPRITE iface, CONST D3D static HRESULT WINAPI ID3DXSpriteImpl_SetWorldViewRH(LPD3DXSPRITE iface, CONST D3DXMATRIX *world, CONST D3DXMATRIX *view) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%p, %p): stub\n", This, world, view); return E_NOTIMPL; } static HRESULT WINAPI ID3DXSpriteImpl_SetWorldViewLH(LPD3DXSPRITE iface, CONST D3DXMATRIX *world, CONST D3DXMATRIX *view) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%p, %p): stub\n", This, world, view); return E_NOTIMPL; } @@ -269,7 +273,8 @@ static HRESULT WINAPI ID3DXSpriteImpl_Draw(LPD3DXSPRITE iface, LPDIRECT3DTEXTURE { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; D3DSURFACE_DESC texdesc; - TRACE("(%p): relay\n", This); + + TRACE("(%p)->(%p, %p, %p, %p, %#x): relay\n", This, texture, rect, center, position, color); if(texture==NULL) return D3DERR_INVALIDCALL; if(!This->ready) return D3DERR_INVALIDCALL; @@ -328,7 +333,7 @@ static HRESULT WINAPI ID3DXSpriteImpl_Flush(LPD3DXSPRITE iface) ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; SPRITEVERTEX *vertices; int i; - TRACE("(%p): relay\n", This); + TRACE("(%p)->(): relay\n", This); if(!This->ready) return D3DERR_INVALIDCALL; if(!This->sprite_count) return D3D_OK; @@ -395,7 +400,8 @@ static HRESULT WINAPI ID3DXSpriteImpl_Flush(LPD3DXSPRITE iface) static HRESULT WINAPI ID3DXSpriteImpl_End(LPD3DXSPRITE iface) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; - TRACE("(%p): relay\n", This); + + TRACE("(%p)->(): relay\n", This); if(!This->ready) return D3DERR_INVALIDCALL; @@ -412,7 +418,8 @@ static HRESULT WINAPI ID3DXSpriteImpl_End(LPD3DXSPRITE iface) static HRESULT WINAPI ID3DXSpriteImpl_OnLostDevice(LPD3DXSPRITE iface) { ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; - TRACE("(%p)\n", This); + + TRACE("(%p)->()\n", This); if(This->stateblock) IDirect3DStateBlock9_Release(This->stateblock); if(This->vdecl) IDirect3DVertexDeclaration9_Release(This->vdecl); @@ -430,7 +437,7 @@ static HRESULT WINAPI ID3DXSpriteImpl_OnResetDevice(LPD3DXSPRITE iface) ID3DXSpriteImpl *This=(ID3DXSpriteImpl*)iface; int i; - TRACE("(%p)\n", This); + TRACE("(%p)->()\n", This); for(i=0;i<This->sprite_count;i++) if(This->sprites[i].texture) @@ -471,7 +478,8 @@ HRESULT WINAPI D3DXCreateSprite(LPDIRECT3DDEVICE9 device, LPD3DXSPRITE *sprite) { ID3DXSpriteImpl *object; D3DCAPS9 caps; - TRACE("(void): relay\n"); + + TRACE("(%p, %p): relay\n", device, sprite); if(device==NULL || sprite==NULL) return D3DERR_INVALIDCALL;

1 0

Christian Costa : d3dx9_36: Improve FIXMEs and TRACEs for font.
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: b3ad91f699fc334bb4f6ced1de9b2e354ba183c4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b3ad91f699fc334bb4f6ced1d… Author: Christian Costa <titan.costa(a)wanadoo.fr> Date: Fri Apr 2 08:54:54 2010 +0200 d3dx9_36: Improve FIXMEs and TRACEs for font. --- dlls/d3dx9_36/font.c | 41 +++++++++++++++++++++++------------------ 1 files changed, 23 insertions(+), 18 deletions(-) diff --git a/dlls/d3dx9_36/font.c b/dlls/d3dx9_36/font.c index 98b4fe9..1113060 100644 --- a/dlls/d3dx9_36/font.c +++ b/dlls/d3dx9_36/font.c @@ -44,7 +44,7 @@ static ULONG WINAPI ID3DXFontImpl_AddRef(LPD3DXFONT iface) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; ULONG ref=InterlockedIncrement(&This->ref); - TRACE("(%p): AddRef from %d\n", This, ref-1); + TRACE("(%p)->(): AddRef from %d\n", This, ref-1); return ref; } @@ -52,7 +52,8 @@ static ULONG WINAPI ID3DXFontImpl_Release(LPD3DXFONT iface) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; ULONG ref=InterlockedDecrement(&This->ref); - TRACE("(%p): ReleaseRef to %d\n", This, ref); + + TRACE("(%p)->(): ReleaseRef to %d\n", This, ref); if(ref==0) { DeleteObject(This->hfont); @@ -66,7 +67,8 @@ static ULONG WINAPI ID3DXFontImpl_Release(LPD3DXFONT iface) static HRESULT WINAPI ID3DXFontImpl_GetDevice(LPD3DXFONT iface, LPDIRECT3DDEVICE9 *device) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - TRACE("(%p)\n", This); + + TRACE("(%p)->(%p)\n", This, device); if( !device ) return D3DERR_INVALIDCALL; *device = This->device; @@ -78,7 +80,8 @@ static HRESULT WINAPI ID3DXFontImpl_GetDevice(LPD3DXFONT iface, LPDIRECT3DDEVICE static HRESULT WINAPI ID3DXFontImpl_GetDescA(LPD3DXFONT iface, D3DXFONT_DESCA *desc) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - TRACE("(%p)\n", This); + + TRACE("(%p)->(%p)\n", This, desc); if( !desc ) return D3DERR_INVALIDCALL; memcpy(desc, &This->desc, FIELD_OFFSET(D3DXFONT_DESCA, FaceName)); @@ -90,7 +93,8 @@ static HRESULT WINAPI ID3DXFontImpl_GetDescA(LPD3DXFONT iface, D3DXFONT_DESCA *d static HRESULT WINAPI ID3DXFontImpl_GetDescW(LPD3DXFONT iface, D3DXFONT_DESCW *desc) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - TRACE("(%p)\n", This); + + TRACE("(%p)->(%p)\n", This, desc); if( !desc ) return D3DERR_INVALIDCALL; *desc = This->desc; @@ -101,84 +105,84 @@ static HRESULT WINAPI ID3DXFontImpl_GetDescW(LPD3DXFONT iface, D3DXFONT_DESCW *d static BOOL WINAPI ID3DXFontImpl_GetTextMetricsA(LPD3DXFONT iface, TEXTMETRICA *metrics) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - TRACE("(%p)\n", This); + TRACE("(%p)->(%p)\n", This, metrics); return GetTextMetricsA(This->hdc, metrics); } static BOOL WINAPI ID3DXFontImpl_GetTextMetricsW(LPD3DXFONT iface, TEXTMETRICW *metrics) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - TRACE("(%p)\n", This); + TRACE("(%p)->(%p)\n", This, metrics); return GetTextMetricsW(This->hdc, metrics); } static HDC WINAPI ID3DXFontImpl_GetDC(LPD3DXFONT iface) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - TRACE("(%p)\n", This); + TRACE("(%p)->()\n", This); return This->hdc; } static HRESULT WINAPI ID3DXFontImpl_GetGlyphData(LPD3DXFONT iface, UINT glyph, LPDIRECT3DTEXTURE9 *texture, RECT *blackbox, POINT *cellinc) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%u, %p, %p, %p): stub\n", This, glyph, texture, blackbox, cellinc); return D3D_OK; } static HRESULT WINAPI ID3DXFontImpl_PreloadCharacters(LPD3DXFONT iface, UINT first, UINT last) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%u, %u): stub\n", This, first, last); return D3D_OK; } static HRESULT WINAPI ID3DXFontImpl_PreloadGlyphs(LPD3DXFONT iface, UINT first, UINT last) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%u, %u): stub\n", This, first, last); return D3D_OK; } static HRESULT WINAPI ID3DXFontImpl_PreloadTextA(LPD3DXFONT iface, LPCSTR string, INT count) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%s, %d): stub\n", This, string, count); return D3D_OK; } static HRESULT WINAPI ID3DXFontImpl_PreloadTextW(LPD3DXFONT iface, LPCWSTR string, INT count) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%s, %d): stub\n", This, debugstr_w(string), count); return D3D_OK; } static INT WINAPI ID3DXFontImpl_DrawTextA(LPD3DXFONT iface, LPD3DXSPRITE sprite, LPCSTR string, INT count, LPRECT rect, DWORD format, D3DCOLOR color) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%p, %s, %d, %p, %d, %#x): stub\n", This, sprite, string, count, rect, format, color); return 1; } static INT WINAPI ID3DXFontImpl_DrawTextW(LPD3DXFONT iface, LPD3DXSPRITE sprite, LPCWSTR string, INT count, LPRECT rect, DWORD format, D3DCOLOR color) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(%p, %s, %d, %p, %d, %#x): stub\n", This, sprite, debugstr_w(string), count, rect, format, color); return 1; } static HRESULT WINAPI ID3DXFontImpl_OnLostDevice(LPD3DXFONT iface) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(): stub\n", This); return D3D_OK; } static HRESULT WINAPI ID3DXFontImpl_OnResetDevice(LPD3DXFONT iface) { ID3DXFontImpl *This=(ID3DXFontImpl*)iface; - FIXME("(%p): stub\n", This); + FIXME("(%p)->(): stub\n", This); return D3D_OK; } @@ -277,7 +281,8 @@ HRESULT WINAPI D3DXCreateFontIndirectW(LPDIRECT3DDEVICE9 device, CONST D3DXFONT_ ID3DXFontImpl *object; IDirect3D9 *d3d; HRESULT hr; - FIXME("stub\n"); + + FIXME("(%p, %p, %p): stub\n", device, desc, font); if( !device || !desc || !font ) return D3DERR_INVALIDCALL;

1 0

Alexandre Julliard : server: Update the registry key and value name length limits to the now documented values .
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: 935cc7987df61352a2cc51b83e839551e9720fb2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=935cc7987df61352a2cc51b83… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 2 11:48:14 2010 +0200 server: Update the registry key and value name length limits to the now documented values. --- dlls/ntdll/reg.c | 12 +++++++----- server/registry.c | 4 ++-- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/dlls/ntdll/reg.c b/dlls/ntdll/reg.c index 3bee588..d68f154 100644 --- a/dlls/ntdll/reg.c +++ b/dlls/ntdll/reg.c @@ -42,8 +42,10 @@ WINE_DEFAULT_DEBUG_CHANNEL(reg); -/* maximum length of a key/value name in bytes (without terminating null) */ -#define MAX_NAME_LENGTH ((MAX_PATH-1) * sizeof(WCHAR)) +/* maximum length of a key name in bytes (without terminating null) */ +#define MAX_NAME_LENGTH (255 * sizeof(WCHAR)) +/* maximum length of a value name in bytes (without terminating null) */ +#define MAX_VALUE_LENGTH (16383 * sizeof(WCHAR)) /****************************************************************************** * NtCreateKey [NTDLL.@] @@ -188,7 +190,7 @@ NTSTATUS WINAPI NtDeleteValueKey( HANDLE hkey, const UNICODE_STRING *name ) NTSTATUS ret; TRACE( "(%p,%s)\n", hkey, debugstr_us(name) ); - if (name->Length > MAX_NAME_LENGTH) return STATUS_BUFFER_OVERFLOW; + if (name->Length > MAX_VALUE_LENGTH) return STATUS_BUFFER_OVERFLOW; SERVER_START_REQ( delete_key_value ) { @@ -481,7 +483,7 @@ NTSTATUS WINAPI NtQueryValueKey( HANDLE handle, const UNICODE_STRING *name, TRACE( "(%p,%s,%d,%p,%d)\n", handle, debugstr_us(name), info_class, info, length ); - if (name->Length > MAX_NAME_LENGTH) return STATUS_BUFFER_OVERFLOW; + if (name->Length > MAX_VALUE_LENGTH) return STATUS_BUFFER_OVERFLOW; /* compute the length we want to retrieve */ switch(info_class) @@ -769,7 +771,7 @@ NTSTATUS WINAPI NtSetValueKey( HANDLE hkey, const UNICODE_STRING *name, ULONG Ti TRACE( "(%p,%s,%d,%p,%d)\n", hkey, debugstr_us(name), type, data, count ); - if (name->Length > MAX_NAME_LENGTH) return STATUS_BUFFER_OVERFLOW; + if (name->Length > MAX_VALUE_LENGTH) return STATUS_BUFFER_OVERFLOW; SERVER_START_REQ( set_key_value ) { diff --git a/server/registry.c b/server/registry.c index 93d42c9..574f2e4 100644 --- a/server/registry.c +++ b/server/registry.c @@ -100,8 +100,8 @@ struct key_value #define MIN_SUBKEYS 8 /* min. number of allocated subkeys per key */ #define MIN_VALUES 8 /* min. number of allocated values per key */ -#define MAX_NAME_LEN MAX_PATH /* max. length of a key name */ -#define MAX_VALUE_LEN MAX_PATH /* max. length of a value name */ +#define MAX_NAME_LEN 255 /* max. length of a key name */ +#define MAX_VALUE_LEN 16383 /* max. length of a value name */ /* the root of the registry tree */ static struct key *root_key;

1 0

Alexandre Julliard : advapi32: Don' t use the static Unicode buffer for value names since they can be larger than MAX_PATH .
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: acc41b50d123d693041b7feaa763ed4c227f9b94 URL: http://source.winehq.org/git/wine.git/?a=commit;h=acc41b50d123d693041b7feaa… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 2 11:50:12 2010 +0200 advapi32: Don't use the static Unicode buffer for value names since they can be larger than MAX_PATH. --- dlls/advapi32/registry.c | 31 ++++++++++++++++++------------- dlls/advapi32/tests/registry.c | 3 ++- 2 files changed, 20 insertions(+), 14 deletions(-) diff --git a/dlls/advapi32/registry.c b/dlls/advapi32/registry.c index dd3f516..9266d3f 100644 --- a/dlls/advapi32/registry.c +++ b/dlls/advapi32/registry.c @@ -1103,6 +1103,7 @@ LSTATUS WINAPI RegSetValueExA( HKEY hkey, LPCSTR name, DWORD reserved, DWORD typ CONST BYTE *data, DWORD count ) { ANSI_STRING nameA; + UNICODE_STRING nameW; WCHAR *dataW = NULL; NTSTATUS status; @@ -1133,10 +1134,10 @@ LSTATUS WINAPI RegSetValueExA( HKEY hkey, LPCSTR name, DWORD reserved, DWORD typ } RtlInitAnsiString( &nameA, name ); - if (!(status = RtlAnsiStringToUnicodeString( &NtCurrentTeb()->StaticUnicodeString, - &nameA, FALSE ))) + if (!(status = RtlAnsiStringToUnicodeString( &nameW, &nameA, TRUE ))) { - status = NtSetValueKey( hkey, &NtCurrentTeb()->StaticUnicodeString, 0, type, data, count ); + status = NtSetValueKey( hkey, &nameW, 0, type, data, count ); + RtlFreeUnicodeString( &nameW ); } HeapFree( GetProcessHeap(), 0, dataW ); return RtlNtStatusToDosError( status ); @@ -1305,6 +1306,7 @@ LSTATUS WINAPI RegQueryValueExA( HKEY hkey, LPCSTR name, LPDWORD reserved, LPDWO { NTSTATUS status; ANSI_STRING nameA; + UNICODE_STRING nameW; DWORD total_size, datalen = 0; char buffer[256], *buf_ptr = buffer; KEY_VALUE_PARTIAL_INFORMATION *info = (KEY_VALUE_PARTIAL_INFORMATION *)buffer; @@ -1323,12 +1325,11 @@ LSTATUS WINAPI RegQueryValueExA( HKEY hkey, LPCSTR name, LPDWORD reserved, LPDWO if (type) *type = REG_NONE; RtlInitAnsiString( &nameA, name ); - if ((status = RtlAnsiStringToUnicodeString( &NtCurrentTeb()->StaticUnicodeString, - &nameA, FALSE ))) + if ((status = RtlAnsiStringToUnicodeString( &nameW, &nameA, TRUE ))) return RtlNtStatusToDosError(status); - status = NtQueryValueKey( hkey, &NtCurrentTeb()->StaticUnicodeString, - KeyValuePartialInformation, buffer, sizeof(buffer), &total_size ); + status = NtQueryValueKey( hkey, &nameW, KeyValuePartialInformation, + buffer, sizeof(buffer), &total_size ); if (status && status != STATUS_BUFFER_OVERFLOW) goto done; /* we need to fetch the contents for a string type even if not requested, @@ -1345,8 +1346,8 @@ LSTATUS WINAPI RegQueryValueExA( HKEY hkey, LPCSTR name, LPDWORD reserved, LPDWO goto done; } info = (KEY_VALUE_PARTIAL_INFORMATION *)buf_ptr; - status = NtQueryValueKey( hkey, &NtCurrentTeb()->StaticUnicodeString, - KeyValuePartialInformation, buf_ptr, total_size, &total_size ); + status = NtQueryValueKey( hkey, &nameW, KeyValuePartialInformation, + buf_ptr, total_size, &total_size ); } if (status) goto done; @@ -1384,6 +1385,7 @@ LSTATUS WINAPI RegQueryValueExA( HKEY hkey, LPCSTR name, LPDWORD reserved, LPDWO done: if (buf_ptr != buffer) HeapFree( GetProcessHeap(), 0, buf_ptr ); + RtlFreeUnicodeString( &nameW ); return RtlNtStatusToDosError(status); } @@ -1955,15 +1957,18 @@ LSTATUS WINAPI RegDeleteValueW( HKEY hkey, LPCWSTR name ) */ LSTATUS WINAPI RegDeleteValueA( HKEY hkey, LPCSTR name ) { - STRING nameA; + ANSI_STRING nameA; + UNICODE_STRING nameW; NTSTATUS status; if (!(hkey = get_special_root_hkey( hkey ))) return ERROR_INVALID_HANDLE; RtlInitAnsiString( &nameA, name ); - if (!(status = RtlAnsiStringToUnicodeString( &NtCurrentTeb()->StaticUnicodeString, - &nameA, FALSE ))) - status = NtDeleteValueKey( hkey, &NtCurrentTeb()->StaticUnicodeString ); + if (!(status = RtlAnsiStringToUnicodeString( &nameW, &nameA, TRUE ))) + { + status = NtDeleteValueKey( hkey, &nameW ); + RtlFreeUnicodeString( &nameW ); + } return RtlNtStatusToDosError( status ); } diff --git a/dlls/advapi32/tests/registry.c b/dlls/advapi32/tests/registry.c index d0ef19e..63ed984 100644 --- a/dlls/advapi32/tests/registry.c +++ b/dlls/advapi32/tests/registry.c @@ -1968,7 +1968,8 @@ static void test_delete_value(void) memset(longname, 'a', 400); longname[400] = 0; res = RegDeleteValueA( hkey_main, longname ); - todo_wine ok(res == ERROR_FILE_NOT_FOUND, "expect ERROR_FILE_NOT_FOUND, got %i\n", res); + ok(res == ERROR_FILE_NOT_FOUND || broken(res == ERROR_MORE_DATA), /* nt4, win2k */ + "expect ERROR_FILE_NOT_FOUND, got %i\n", res); } START_TEST(registry)

1 0

Alexandre Julliard : advapi32: Add support for the KEY_WOW64_32KEY flag in RegCreateKey on 64-bit.
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: 68a5c34731253e5965615b0b1cf920b63938e035 URL: http://source.winehq.org/git/wine.git/?a=commit;h=68a5c34731253e5965615b0b1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 2 11:51:15 2010 +0200 advapi32: Add support for the KEY_WOW64_32KEY flag in RegCreateKey on 64-bit. --- dlls/advapi32/registry.c | 71 ++++++++++++++++++++++++++++++++++------ dlls/advapi32/tests/registry.c | 4 ++- 2 files changed, 64 insertions(+), 11 deletions(-) diff --git a/dlls/advapi32/registry.c b/dlls/advapi32/registry.c index 9266d3f..ba9d9a7 100644 --- a/dlls/advapi32/registry.c +++ b/dlls/advapi32/registry.c @@ -78,6 +78,7 @@ static const WCHAR * const root_key_names[NB_SPECIAL_ROOT_KEYS] = name_DYN_DATA }; +static const int is_win64 = (sizeof(void *) > sizeof(int)); /* check if value type needs string conversion (Ansi<->Unicode) */ static inline int is_string( DWORD type ) @@ -91,34 +92,84 @@ static inline int is_version_nt(void) return !(GetVersion() & 0x80000000); } +static BOOL is_wow6432node( const UNICODE_STRING *name ) +{ + static const WCHAR wow6432nodeW[] = {'W','o','w','6','4','3','2','N','o','d','e'}; + + return (name->Length == sizeof(wow6432nodeW) && + !memicmpW( name->Buffer, wow6432nodeW, sizeof(wow6432nodeW)/sizeof(WCHAR) )); +} + +/* open the Wow6432Node subkey of the specified key */ +static HANDLE open_wow6432node( HANDLE key, const UNICODE_STRING *name ) +{ + static const WCHAR wow6432nodeW[] = {'W','o','w','6','4','3','2','N','o','d','e',0}; + OBJECT_ATTRIBUTES attr; + UNICODE_STRING nameW; + HANDLE ret; + + attr.Length = sizeof(attr); + attr.RootDirectory = key; + attr.ObjectName = &nameW; + attr.Attributes = 0; + attr.SecurityDescriptor = NULL; + attr.SecurityQualityOfService = NULL; + RtlInitUnicodeString( &nameW, wow6432nodeW ); + if (NtOpenKey( &ret, MAXIMUM_ALLOWED, &attr )) ret = 0; + return ret; +} + /* wrapper for NtCreateKey that creates the key recursively if necessary */ static NTSTATUS create_key( HKEY *retkey, ACCESS_MASK access, OBJECT_ATTRIBUTES *attr, const UNICODE_STRING *class, ULONG options, PULONG dispos ) { - NTSTATUS status = NtCreateKey( (HANDLE *)retkey, access, attr, 0, class, options, dispos ); + BOOL force_wow32 = is_win64 && (access & KEY_WOW64_32KEY); + NTSTATUS status = STATUS_OBJECT_NAME_NOT_FOUND; + + if (!force_wow32) status = NtCreateKey( (HANDLE *)retkey, access, attr, 0, class, options, dispos ); if (status == STATUS_OBJECT_NAME_NOT_FOUND) { - DWORD attrs, i = 0, len = attr->ObjectName->Length / sizeof(WCHAR); + HANDLE subkey, root = attr->RootDirectory; + WCHAR *buffer = attr->ObjectName->Buffer; + DWORD attrs, pos = 0, i = 0, len = attr->ObjectName->Length / sizeof(WCHAR); + UNICODE_STRING str; + + while (i < len && buffer[i] != '\\') i++; + if (i == len && !force_wow32) return status; - while (i < len && attr->ObjectName->Buffer[i] != '\\') i++; - if (i == len) return status; attrs = attr->Attributes; attr->Attributes &= ~OBJ_OPENLINK; + attr->ObjectName = &str; while (i < len) { - attr->ObjectName->Length = i * sizeof(WCHAR); - status = NtCreateKey( (HANDLE *)retkey, access, attr, 0, class, + str.Buffer = buffer + pos; + str.Length = (i - pos) * sizeof(WCHAR); + if (force_wow32 && pos) + { + if (is_wow6432node( &str )) force_wow32 = FALSE; + else if ((subkey = open_wow6432node( attr->RootDirectory, &str ))) + { + if (attr->RootDirectory != root) NtClose( attr->RootDirectory ); + attr->RootDirectory = subkey; + force_wow32 = FALSE; + } + } + status = NtCreateKey( &subkey, access, attr, 0, class, options & ~REG_OPTION_CREATE_LINK, dispos ); + if (attr->RootDirectory != root) NtClose( attr->RootDirectory ); if (status) return status; - NtClose( *retkey ); - while (i < len && attr->ObjectName->Buffer[i] == '\\') i++; - while (i < len && attr->ObjectName->Buffer[i] != '\\') i++; + attr->RootDirectory = subkey; + while (i < len && buffer[i] == '\\') i++; + pos = i; + while (i < len && buffer[i] != '\\') i++; } + str.Buffer = buffer + pos; + str.Length = (i - pos) * sizeof(WCHAR); attr->Attributes = attrs; - attr->ObjectName->Length = len * sizeof(WCHAR); status = NtCreateKey( (PHANDLE)retkey, access, attr, 0, class, options, dispos ); + if (attr->RootDirectory != root) NtClose( attr->RootDirectory ); } return status; } diff --git a/dlls/advapi32/tests/registry.c b/dlls/advapi32/tests/registry.c index 63ed984..ea036e8 100644 --- a/dlls/advapi32/tests/registry.c +++ b/dlls/advapi32/tests/registry.c @@ -1862,7 +1862,9 @@ static void test_redirection(void) ok( err == ERROR_SUCCESS, "RegCreateKeyExA failed: %u\n", err ); check_key_value( key, "Winetest", 0, ptr_size ); check_key_value( key, "Winetest", KEY_WOW64_64KEY, is_vista ? 64 : ptr_size ); - check_key_value( key, "Winetest", KEY_WOW64_32KEY, 32 ); + dw = get_key_value( key, "Winetest", KEY_WOW64_32KEY ); + if (ptr_size == 32) ok( dw == 32, "wrong value %u\n", dw ); + else todo_wine ok( dw == 32, "wrong value %u\n", dw ); RegCloseKey( key ); if (ptr_size == 32)

1 0

Alexandre Julliard : advapi32: Add support for the KEY_WOW64_32KEY flag in RegOpenKey on 64-bit.
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: 21da080dbd71ed5b5d00ed9dcd4370c771edf2a4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=21da080dbd71ed5b5d00ed9dc… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 2 12:14:21 2010 +0200 advapi32: Add support for the KEY_WOW64_32KEY flag in RegOpenKey on 64-bit. --- dlls/advapi32/registry.c | 53 ++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 51 insertions(+), 2 deletions(-) diff --git a/dlls/advapi32/registry.c b/dlls/advapi32/registry.c index ba9d9a7..abaf09d 100644 --- a/dlls/advapi32/registry.c +++ b/dlls/advapi32/registry.c @@ -174,6 +174,55 @@ static NTSTATUS create_key( HKEY *retkey, ACCESS_MASK access, OBJECT_ATTRIBUTES return status; } +/* wrapper for NtOpenKey to handle Wow6432 nodes */ +static NTSTATUS open_key( HKEY *retkey, ACCESS_MASK access, OBJECT_ATTRIBUTES *attr ) +{ + NTSTATUS status; + BOOL force_wow32 = is_win64 && (access & KEY_WOW64_32KEY); + HANDLE subkey, root = attr->RootDirectory; + WCHAR *buffer = attr->ObjectName->Buffer; + DWORD attrs, pos = 0, i = 0, len = attr->ObjectName->Length / sizeof(WCHAR); + UNICODE_STRING str; + + if (!force_wow32) return NtOpenKey( (HANDLE *)retkey, access, attr ); + + if (len && buffer[0] == '\\') return STATUS_OBJECT_PATH_INVALID; + while (i < len && buffer[i] != '\\') i++; + attrs = attr->Attributes; + attr->Attributes &= ~OBJ_OPENLINK; + attr->ObjectName = &str; + + while (i < len) + { + str.Buffer = buffer + pos; + str.Length = (i - pos) * sizeof(WCHAR); + if (force_wow32 && pos) + { + if (is_wow6432node( &str )) force_wow32 = FALSE; + else if ((subkey = open_wow6432node( attr->RootDirectory, &str ))) + { + if (attr->RootDirectory != root) NtClose( attr->RootDirectory ); + attr->RootDirectory = subkey; + force_wow32 = FALSE; + } + } + status = NtOpenKey( &subkey, access, attr ); + if (attr->RootDirectory != root) NtClose( attr->RootDirectory ); + if (status) return status; + attr->RootDirectory = subkey; + while (i < len && buffer[i] == '\\') i++; + pos = i; + while (i < len && buffer[i] != '\\') i++; + } + str.Buffer = buffer + pos; + str.Length = (i - pos) * sizeof(WCHAR); + attr->Attributes = attrs; + status = NtOpenKey( (PHANDLE)retkey, access, attr ); + if (attr->RootDirectory != root) NtClose( attr->RootDirectory ); + return status; + +} + /* create one of the HKEY_* special root keys */ static HKEY create_special_root_hkey( HKEY hkey, DWORD access ) { @@ -404,7 +453,7 @@ LSTATUS WINAPI RegOpenKeyExW( HKEY hkey, LPCWSTR name, DWORD options, REGSAM acc attr.SecurityQualityOfService = NULL; if (options & REG_OPTION_OPEN_LINK) attr.Attributes |= OBJ_OPENLINK; RtlInitUnicodeString( &nameW, name ); - return RtlNtStatusToDosError( NtOpenKey( (PHANDLE)retkey, access, &attr ) ); + return RtlNtStatusToDosError( open_key( retkey, access, &attr ) ); } @@ -455,7 +504,7 @@ LSTATUS WINAPI RegOpenKeyExA( HKEY hkey, LPCSTR name, DWORD options, REGSAM acce if (!(status = RtlAnsiStringToUnicodeString( &NtCurrentTeb()->StaticUnicodeString, &nameA, FALSE ))) { - status = NtOpenKey( (PHANDLE)retkey, access, &attr ); + status = open_key( retkey, access, &attr ); } return RtlNtStatusToDosError( status ); }

1 0

Alexandre Julliard : server: Disallow all operations on deleted registry keys.
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: 6ebc627996292fe598e5173be4bc774f9001720b URL: http://source.winehq.org/git/wine.git/?a=commit;h=6ebc627996292fe598e5173be… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 2 12:43:31 2010 +0200 server: Disallow all operations on deleted registry keys. --- dlls/advapi32/tests/registry.c | 19 +++++++++++++++---- server/registry.c | 33 +++++++++++++-------------------- 2 files changed, 28 insertions(+), 24 deletions(-) diff --git a/dlls/advapi32/tests/registry.c b/dlls/advapi32/tests/registry.c index ea036e8..30dda7e 100644 --- a/dlls/advapi32/tests/registry.c +++ b/dlls/advapi32/tests/registry.c @@ -1924,16 +1924,24 @@ static void test_deleted_key(void) delete_key( hkey_main ); - val_count = 20; + val_count = sizeof(value); type = 0; res = RegEnumValueA( hkey, 0, value, &val_count, NULL, &type, 0, 0 ); - todo_wine ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); + ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); + + res = RegEnumKeyA( hkey, 0, value, sizeof(value) ); + ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); + + val_count = sizeof(value); + type = 0; + res = RegQueryValueExA( hkey, "test", NULL, &type, (BYTE *)value, &val_count ); + ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); res = RegSetValueExA( hkey, "test", 0, REG_SZ, (const BYTE*)"value", 6); - todo_wine ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); + ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); res = RegOpenKeyA( hkey, "test", &hkey2 ); - todo_wine ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); + ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); if (res == 0) RegCloseKey( hkey2 ); @@ -1942,6 +1950,9 @@ static void test_deleted_key(void) if (res == 0) RegCloseKey( hkey2 ); + res = RegFlushKey( hkey ); + ok(res == ERROR_KEY_DELETED, "expect ERROR_KEY_DELETED, got %i\n", res); + RegCloseKey( hkey ); setup_main_key(); diff --git a/server/registry.c b/server/registry.c index 574f2e4..f4dbf8e 100644 --- a/server/registry.c +++ b/server/registry.c @@ -712,12 +712,6 @@ static struct key *create_key( struct key *key, const struct unicode_str *name, int index; struct unicode_str token, next; - if (key->flags & KEY_DELETED) /* we cannot create a subkey under a deleted key */ - { - set_error( STATUS_KEY_DELETED ); - return NULL; - } - *created = 0; if (!(key = open_key_prefix( key, name, access, &token, &index ))) return NULL; @@ -902,7 +896,7 @@ static void enum_key( const struct key *key, int index, int info_class, static int delete_key( struct key *key, int recurse ) { int index; - struct key *parent; + struct key *parent = key->parent; /* must find parent and index */ if (key == root_key) @@ -910,11 +904,7 @@ static int delete_key( struct key *key, int recurse ) set_error( STATUS_ACCESS_DENIED ); return -1; } - if (!(parent = key->parent) || (key->flags & KEY_DELETED)) - { - set_error( STATUS_KEY_DELETED ); - return -1; - } + assert( parent ); while (recurse && (key->last_subkey>=0)) if (0 > delete_key(key->subkeys[key->last_subkey], 1)) @@ -1165,16 +1155,24 @@ static void delete_value( struct key *key, const struct unicode_str *name ) } /* get the registry key corresponding to an hkey handle */ -static inline struct key *get_hkey_obj( obj_handle_t hkey, unsigned int access ) +static struct key *get_hkey_obj( obj_handle_t hkey, unsigned int access ) { - return (struct key *)get_handle_obj( current->process, hkey, access, &key_ops ); + struct key *key = (struct key *)get_handle_obj( current->process, hkey, access, &key_ops ); + + if (key && key->flags & KEY_DELETED) + { + set_error( STATUS_KEY_DELETED ); + release_object( key ); + key = NULL; + } + return key; } /* get the registry key corresponding to a parent key handle */ static inline struct key *get_parent_hkey_obj( obj_handle_t hkey ) { if (!hkey) return (struct key *)grab_object( root_key ); - return (struct key *)get_handle_obj( current->process, hkey, 0, &key_ops ); + return get_hkey_obj( hkey, 0 ); } /* read a line from the input file */ @@ -1698,11 +1696,6 @@ static void save_registry( struct key *key, obj_handle_t handle ) struct file *file; int fd; - if (key->flags & KEY_DELETED) - { - set_error( STATUS_KEY_DELETED ); - return; - } if (!(file = get_file_obj( current->process, handle, FILE_WRITE_DATA ))) return; fd = dup( get_file_unix_fd( file ) ); release_object( file );

1 0

Hans Leidekker : msi: Implement the UnregisterClassInfo standard action.
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: 843968c056555109f44567bea784683cb3e5c5a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=843968c056555109f44567bea… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Apr 2 10:37:50 2010 +0200 msi: Implement the UnregisterClassInfo standard action. --- dlls/msi/action.c | 8 --- dlls/msi/classes.c | 167 ++++++++++++++++++++++++++++++++++++---------------- dlls/msi/msipriv.h | 9 +++ 3 files changed, 124 insertions(+), 60 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=843968c056555109f4456…

1 0

Hans Leidekker : msi: Set the action taken in the RegisterExtensionInfo and RegisterClassInfo actions .
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: cf75d40dadaf5fb717f5627d5f05ea2e0ea373fd URL: http://source.winehq.org/git/wine.git/?a=commit;h=cf75d40dadaf5fb717f5627d5… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Apr 2 10:38:12 2010 +0200 msi: Set the action taken in the RegisterExtensionInfo and RegisterClassInfo actions. --- dlls/msi/classes.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/msi/classes.c b/dlls/msi/classes.c index 53ed70f..7d4f1a0 100644 --- a/dlls/msi/classes.c +++ b/dlls/msi/classes.c @@ -827,6 +827,7 @@ UINT ACTION_RegisterClassInfo(MSIPACKAGE *package) debugstr_w(feature->Feature), debugstr_w(cls->clsid)); continue; } + feature->Action = feature->ActionRequest; file = get_loaded_file( package, comp->KeyPath ); if (!file) @@ -1220,6 +1221,7 @@ UINT ACTION_RegisterExtensionInfo(MSIPACKAGE *package) debugstr_w(feature->Feature), debugstr_w(ext->Extension)); continue; } + feature->Action = feature->ActionRequest; TRACE("Registering extension %s (%p)\n", debugstr_w(ext->Extension), ext);

1 0

Hans Leidekker : msi: Add tests for registering and unregistering class info.
by Alexandre Julliard 02 Apr '10

02 Apr '10

Module: wine Branch: master Commit: 90158af2dc4f257bab50b0c4603dec9ea8c9b7fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=90158af2dc4f257bab50b0c46… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Apr 2 10:38:35 2010 +0200 msi: Add tests for registering and unregistering class info. --- dlls/msi/tests/install.c | 108 ++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 108 insertions(+), 0 deletions(-) diff --git a/dlls/msi/tests/install.c b/dlls/msi/tests/install.c index 6672de2..35cd64a 100644 --- a/dlls/msi/tests/install.c +++ b/dlls/msi/tests/install.c @@ -1696,6 +1696,55 @@ static const CHAR res_install_exec_seq_dat[] = "Action\tCondition\tSequence\n" "PublishProduct\t\t5200\n" "InstallFinalize\t\t6000\n"; +static const CHAR rci_file_dat[] = "File\tComponent_\tFileName\tFileSize\tVersion\tLanguage\tAttributes\tSequence\n" + "s72\ts72\tl255\ti4\tS72\tS20\tI2\ti2\n" + "File\tFile\n" + "class.txt\tclass\tclass.txt\t1000\t\t\t8192\t1\n"; + +static const CHAR rci_feature_dat[] = "Feature\tFeature_Parent\tTitle\tDescription\tDisplay\tLevel\tDirectory_\tAttributes\n" + "s38\tS38\tL64\tL255\tI2\ti2\tS72\ti2\n" + "Feature\tFeature\n" + "class\t\t\tclass feature\t1\t2\tMSITESTDIR\t0\n"; + +static const CHAR rci_feature_comp_dat[] = "Feature_\tComponent_\n" + "s38\ts72\n" + "FeatureComponents\tFeature_\tComponent_\n" + "class\tclass\n"; + +static const CHAR rci_component_dat[] = "Component\tComponentId\tDirectory_\tAttributes\tCondition\tKeyPath\n" + "s72\tS38\ts72\ti2\tS255\tS72\n" + "Component\tComponent\n" + "class\t{89A98345-F8A1-422E-A48B-0250B5809F2D}\tMSITESTDIR\t0\t\tclass.txt\n"; + +static const CHAR rci_appid_dat[] = "AppId\tRemoteServerName\tLocalService\tServiceParameters\tDllSurrogate\tActivateAtStorage\tRunAsInteractiveUser\n" + "s38\tS255\tS255\tS255\tS255\tI2\tI2\n" + "AppId\tAppId\n" + "{CFCC3B38-E683-497D-9AB4-CB40AAFE307F}\t\t\t\t\t\t\n"; + +static const CHAR rci_class_dat[] = "CLSID\tContext\tComponent_\tProgId_Default\tDescription\tAppId_\tFileTypeMask\tIcon_\tIconIndex\tDefInprocHandler\tArgument\tFeature_\tAttributes\n" + "s38\ts32\ts72\tS255\tL255\tS38\tS255\tS72\tI2\tS32\tS255\ts38\tI2\n" + "Class\tCLSID\tContext\tComponent_\n" + "{110913E7-86D1-4BF3-9922-BA103FCDDDFA}\tLocalServer\tclass\t\tdescription\t{CFCC3B38-E683-497D-9AB4-CB40AAFE307F}\tmask1;mask2\t\t\t2\t\tclass\t\n"; + +static const CHAR rci_install_exec_seq_dat[] = "Action\tCondition\tSequence\n" + "s72\tS255\tI2\n" + "InstallExecuteSequence\tAction\n" + "LaunchConditions\t\t100\n" + "CostInitialize\t\t800\n" + "FileCost\t\t900\n" + "CostFinalize\t\t1000\n" + "InstallValidate\t\t1400\n" + "InstallInitialize\t\t1500\n" + "ProcessComponents\t\t1600\n" + "RemoveFiles\t\t1700\n" + "InstallFiles\t\t2000\n" + "UnregisterClassInfo\t\t3000\n" + "RegisterClassInfo\t\t4000\n" + "RegisterProduct\t\t5000\n" + "PublishFeatures\t\t5100\n" + "PublishProduct\t\t5200\n" + "InstallFinalize\t\t6000\n"; + typedef struct _msi_table { const CHAR *filename; @@ -2614,6 +2663,20 @@ static const msi_table res_tables[] = ADD_TABLE(property) }; +static const msi_table rci_tables[] = +{ + ADD_TABLE(directory), + ADD_TABLE(rci_component), + ADD_TABLE(rci_feature), + ADD_TABLE(rci_feature_comp), + ADD_TABLE(rci_file), + ADD_TABLE(rci_appid), + ADD_TABLE(rci_class), + ADD_TABLE(rci_install_exec_seq), + ADD_TABLE(media), + ADD_TABLE(property) +}; + /* cabinet definitions */ /* make the max size large so there is only one cab file */ @@ -8931,6 +8994,50 @@ static void test_remove_env_strings(void) delete_test_files(); } +static void test_register_class_info(void) +{ + UINT r; + LONG res; + HKEY hkey; + + create_test_files(); + create_file("msitest\\class.txt", 1000); + create_database(msifile, rci_tables, sizeof(rci_tables) / sizeof(msi_table)); + + MsiSetInternalUI(INSTALLUILEVEL_NONE, NULL); + + r = MsiInstallProductA(msifile, NULL); + ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %u\n", r); + + res = RegOpenKeyA(HKEY_CLASSES_ROOT, "CLSID\\{110913E7-86D1-4BF3-9922-BA103FCDDDFA}", &hkey); + ok(res == ERROR_SUCCESS, "key not created\n"); + RegCloseKey(hkey); + + res = RegOpenKeyA(HKEY_CLASSES_ROOT, "FileType\\{110913E7-86D1-4BF3-9922-BA103FCDDDFA}", &hkey); + ok(res == ERROR_SUCCESS, "key not created\n"); + RegCloseKey(hkey); + + res = RegOpenKeyA(HKEY_CLASSES_ROOT, "AppID\\{CFCC3B38-E683-497D-9AB4-CB40AAFE307F}", &hkey); + ok(res == ERROR_SUCCESS, "key not created\n"); + RegCloseKey(hkey); + + r = MsiInstallProductA(msifile, "REMOVE=ALL"); + ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %u\n", r); + + res = RegOpenKeyA(HKEY_CLASSES_ROOT, "CLSID\\{110913E7-86D1-4BF3-9922-BA103FCDDDFA}", &hkey); + ok(res == ERROR_FILE_NOT_FOUND, "key not removed\n"); + + res = RegOpenKeyA(HKEY_CLASSES_ROOT, "FileType\\{110913E7-86D1-4BF3-9922-BA103FCDDDFA}", &hkey); + ok(res == ERROR_FILE_NOT_FOUND, "key not removed\n"); + + res = RegOpenKeyA(HKEY_CLASSES_ROOT, "AppID\\{CFCC3B38-E683-497D-9AB4-CB40AAFE307F}", &hkey); + ok(res == ERROR_FILE_NOT_FOUND, "key not removed\n"); + + ok(!delete_pf("msitest\\class.txt", TRUE), "file not removed\n"); + ok(!delete_pf("msitest", FALSE), "directory not removed\n"); + delete_test_files(); +} + START_TEST(install) { DWORD len; @@ -9038,6 +9145,7 @@ START_TEST(install) test_find_related_products(); test_remove_ini_values(); test_remove_env_strings(); + test_register_class_info(); DeleteFileA(log_file);

1 0