February 2010 - Wine-commits

Lei Zhang : userenv: Add a small test for DestroyEnvironmentBlock.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: a96db3c35c24624bac4f73e312f05d76ff0960fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=a96db3c35c24624bac4f73e31… Author: Lei Zhang <thestig(a)google.com> Date: Tue Feb 2 12:45:06 2010 -0800 userenv: Add a small test for DestroyEnvironmentBlock. --- dlls/userenv/tests/userenv.c | 6 ++++++ include/userenv.h | 1 + 2 files changed, 7 insertions(+), 0 deletions(-) diff --git a/dlls/userenv/tests/userenv.c b/dlls/userenv/tests/userenv.c index 39c1bbf..c61958f 100644 --- a/dlls/userenv/tests/userenv.c +++ b/dlls/userenv/tests/userenv.c @@ -197,6 +197,12 @@ static void test_create_env(void) r = get_env(env[3], "WINE_XYZZY", &st); expect(TRUE, r); if (r) HeapFree(GetProcessHeap(), 0, st); + + for (i = 0; i < sizeof(env) / sizeof(env[0]); i++) + { + r = DestroyEnvironmentBlock(env[i]); + expect(TRUE, r); + } } START_TEST(userenv) diff --git a/include/userenv.h b/include/userenv.h index 507715f..de8bbe0 100644 --- a/include/userenv.h +++ b/include/userenv.h @@ -31,6 +31,7 @@ extern "C" { #endif BOOL WINAPI CreateEnvironmentBlock(LPVOID*,HANDLE,BOOL); +BOOL WINAPI DestroyEnvironmentBlock(LPVOID); BOOL WINAPI ExpandEnvironmentStringsForUserA(HANDLE,LPCSTR,LPSTR,DWORD); BOOL WINAPI ExpandEnvironmentStringsForUserW(HANDLE,LPCWSTR,LPWSTR,DWORD); #define ExpandEnvironmentStringsForUser WINELIB_NAME_AW(ExpandEnvironmentStringsForUser)

1 0

Luca Bennati : browseui: Add Italian translation.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 1ecd8f3e7ef3b8a241b40dbb540670cb556698af URL: http://source.winehq.org/git/wine.git/?a=commit;h=1ecd8f3e7ef3b8a241b40dbb5… Author: Luca Bennati <lucak3(a)gmail.com> Date: Mon Feb 1 23:07:23 2010 +0100 browseui: Add Italian translation. --- dlls/browseui/Makefile.in | 1 + dlls/browseui/rsrc_It.rc | 39 +++++++++++++++++++++++++++++++++++++++ 2 files changed, 40 insertions(+), 0 deletions(-) diff --git a/dlls/browseui/Makefile.in b/dlls/browseui/Makefile.in index 777a217..92ac831 100644 --- a/dlls/browseui/Makefile.in +++ b/dlls/browseui/Makefile.in @@ -19,6 +19,7 @@ RC_SRCS = \ rsrc_En.rc \ rsrc_Es.rc \ rsrc_Fr.rc \ + rsrc_It.rc \ rsrc_Ja.rc \ rsrc_Ko.rc \ rsrc_Lt.rc \ diff --git a/dlls/browseui/rsrc_It.rc b/dlls/browseui/rsrc_It.rc new file mode 100644 index 0000000..3da2fa9 --- /dev/null +++ b/dlls/browseui/rsrc_It.rc @@ -0,0 +1,39 @@ +/* + * Copyright 2010 Luca Bennati + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include "resids.h" + +LANGUAGE LANG_ITALIAN, SUBLANG_NEUTRAL + +STRINGTABLE +{ + IDS_CANCELLING "Annullando..." +} + +IDD_PROGRESS_DLG DIALOG 0, 0, 260, 85 +STYLE DS_MODALFRAME | DS_CENTER | WS_POPUP | WS_VISIBLE | WS_CLIPCHILDREN | WS_CAPTION | WS_SYSMENU +CAPTION "" +FONT 8, "MS Shell Dlg" +{ + CONTROL "", IDC_ANIMATION, ANIMATE_CLASSA, 0, 12, 10, 236, 25 /* TODO: style */ + LTEXT "", IDC_TEXT_LINE, 7, 45, 250, 10, SS_LEFT|SS_NOPREFIX + LTEXT "", IDC_TEXT_LINE + 1, 7, 55, 250, 10, SS_LEFT|SS_NOPREFIX + CONTROL "", IDC_PROGRESS_BAR, PROGRESS_CLASSA, WS_BORDER, 7, 65, 190, 8 + PUSHBUTTON "&Annulla", IDCANCEL, 205, 65, 40, 15, WS_GROUP | WS_TABSTOP | WS_VISIBLE + LTEXT "", IDC_TEXT_LINE + 2, 7, 75, 190, 10, SS_LEFT|SS_NOPREFIX +}

1 0

Luca Bennati : winspool.drv: Fix Italian translation.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 4509d9aa1030ed672b75ae4aa59b5d4b436f0f1a URL: http://source.winehq.org/git/wine.git/?a=commit;h=4509d9aa1030ed672b75ae4aa… Author: Luca Bennati <lucak3(a)gmail.com> Date: Tue Feb 2 10:57:29 2010 +0100 winspool.drv: Fix Italian translation. --- dlls/winspool.drv/It.rc | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/winspool.drv/It.rc b/dlls/winspool.drv/It.rc index aff05a4..a661dc7 100644 --- a/dlls/winspool.drv/It.rc +++ b/dlls/winspool.drv/It.rc @@ -20,6 +20,9 @@ #include "wspool.h" +/*UTF-8*/ +#pragma code_page(65001) + LANGUAGE LANG_ITALIAN, SUBLANG_NEUTRAL FILENAME_DIALOG DIALOG LOADONCALL MOVEABLE DISCARDABLE 6, 18, 245, 47 @@ -36,6 +39,6 @@ END STRINGTABLE DISCARDABLE { IDS_CAPTION "Porta Locale" - IDS_FILE_EXISTS "Il file in uscita esiste gi�. Clicca su OK per sovrascriverlo." + IDS_FILE_EXISTS "Il file in uscita esiste già. Clicca su OK per sovrascriverlo." IDS_CANNOT_OPEN "Impossibile creare il file in uscita." }

1 0

Dan Kegel : cmd: %~dp0 should expand to the directory containing the batch file, not the current directory.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 60fe4dae7a8c93c6e9575f29ef7a6c1824b251bb URL: http://source.winehq.org/git/wine.git/?a=commit;h=60fe4dae7a8c93c6e9575f29e… Author: Dan Kegel <dank(a)kegel.com> Date: Tue Feb 2 16:56:38 2010 -0800 cmd: %~dp0 should expand to the directory containing the batch file, not the current directory. --- programs/cmd/batch.c | 6 +++++- programs/cmd/wcmd.h | 1 + 2 files changed, 6 insertions(+), 1 deletions(-) diff --git a/programs/cmd/batch.c b/programs/cmd/batch.c index 21d8694..28744d4 100644 --- a/programs/cmd/batch.c +++ b/programs/cmd/batch.c @@ -91,6 +91,7 @@ void WCMD_batch (WCHAR *file, WCHAR *command, int called, WCHAR *startLabel, HAN prev_context = context; context = LocalAlloc (LMEM_FIXED, sizeof (BATCH_CONTEXT)); context -> h = h; + context->batchfileW = WCMD_strdupW(string); context -> command = command; memset(context -> shift_count, 0x00, sizeof(context -> shift_count)); context -> prev_context = prev_context; @@ -122,6 +123,7 @@ void WCMD_batch (WCHAR *file, WCHAR *command, int called, WCHAR *startLabel, HAN * to the caller's caller. */ + HeapFree(GetProcessHeap(), 0, context->batchfileW); LocalFree (context); if ((prev_context != NULL) && (!called)) { prev_context -> skip_rest = TRUE; @@ -382,7 +384,9 @@ void WCMD_HandleTildaModifiers(WCHAR **start, WCHAR *forVariable, WCHAR *forValu if (lastModifier == firstModifier) return; /* Invalid syntax */ /* Extract the parameter to play with */ - if ((*lastModifier >= '0' && *lastModifier <= '9')) { + if (*lastModifier == '0') { + strcpyW(outputparam, context->batchfileW); + } else if ((*lastModifier >= '1' && *lastModifier <= '9')) { strcpyW(outputparam, WCMD_parameter (context -> command, *lastModifier-'0' + context -> shift_count[*lastModifier-'0'], NULL)); } else { diff --git a/programs/cmd/wcmd.h b/programs/cmd/wcmd.h index 00bada2..2fde678 100644 --- a/programs/cmd/wcmd.h +++ b/programs/cmd/wcmd.h @@ -118,6 +118,7 @@ void WCMD_execute (WCHAR *orig_command, WCHAR *redirects, WCHAR *parameter, typedef struct { WCHAR *command; /* The command which invoked the batch file */ HANDLE h; /* Handle to the open batch file */ + WCHAR *batchfileW; /* Name of same */ int shift_count[10]; /* Offset in terms of shifts for %0 - %9 */ void *prev_context; /* Pointer to the previous context block */ BOOL skip_rest; /* Skip the rest of the batch program and exit */

1 0

Dan Kegel : cmd: Size buffer in create_full_path() correctly.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 15998c2bdba386554b7e5f3e12d535d31bb9170c URL: http://source.winehq.org/git/wine.git/?a=commit;h=15998c2bdba386554b7e5f3e1… Author: Dan Kegel <dank(a)kegel.com> Date: Tue Feb 2 17:42:01 2010 -0800 cmd: Size buffer in create_full_path() correctly. --- programs/cmd/builtins.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c index c63360b..d1019cc 100644 --- a/programs/cmd/builtins.c +++ b/programs/cmd/builtins.c @@ -319,7 +319,7 @@ static BOOL create_full_path(WCHAR* path) WCHAR *new_path; BOOL ret = TRUE; - new_path = HeapAlloc(GetProcessHeap(),0,(strlenW(path) * sizeof(WCHAR))+1); + new_path = HeapAlloc(GetProcessHeap(),0,(strlenW(path)+1) * sizeof(WCHAR)); strcpyW(new_path,path); while ((len = strlenW(new_path)) && new_path[len - 1] == '\\')

1 0

Andrew Nguyen : ws2_32: Fix parameter validation order in getpeername.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 1bcc3aa35ca21b64d98f9777967bb6ccf98af567 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1bcc3aa35ca21b64d98f97779… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Wed Feb 3 01:18:40 2010 -0600 ws2_32: Fix parameter validation order in getpeername. --- dlls/ws2_32/socket.c | 27 ++++++---------- dlls/ws2_32/tests/sock.c | 76 ++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 86 insertions(+), 17 deletions(-) diff --git a/dlls/ws2_32/socket.c b/dlls/ws2_32/socket.c index 9a65cb0..a9e9538 100644 --- a/dlls/ws2_32/socket.c +++ b/dlls/ws2_32/socket.c @@ -149,6 +149,7 @@ #include "iphlpapi.h" #include "wine/server.h" #include "wine/debug.h" +#include "wine/exception.h" #include "wine/unicode.h" #ifdef HAVE_IPX @@ -1796,13 +1797,6 @@ int WINAPI WS_getpeername(SOCKET s, struct WS_sockaddr *name, int *namelen) TRACE("socket: %04lx, ptr %p, len %08x\n", s, name, *namelen); - /* Check if what we've received is valid. Should we use IsBadReadPtr? */ - if( (name == NULL) || (namelen == NULL) ) - { - SetLastError( WSAEFAULT ); - return SOCKET_ERROR; - } - fd = get_sock_fd( s, 0, NULL ); res = SOCKET_ERROR; @@ -1811,19 +1805,18 @@ int WINAPI WS_getpeername(SOCKET s, struct WS_sockaddr *name, int *namelen) union generic_unix_sockaddr uaddr; unsigned int uaddrlen = sizeof(uaddr); - if (getpeername(fd, &uaddr.addr, &uaddrlen) != 0) - { - SetLastError(wsaErrno()); - } - else if (ws_sockaddr_u2ws(&uaddr.addr, name, namelen) != 0) + if (getpeername(fd, &uaddr.addr, &uaddrlen) == 0) { - /* The buffer was too small */ - SetLastError(WSAEFAULT); + if (!name || !namelen) + SetLastError(WSAEFAULT); + else if (ws_sockaddr_u2ws(&uaddr.addr, name, namelen) != 0) + /* The buffer was too small */ + SetLastError(WSAEFAULT); + else + res = 0; } else - { - res=0; - } + SetLastError(wsaErrno()); release_sock_fd( s, fd ); } return res; diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c index fbe55f6..72242c4 100644 --- a/dlls/ws2_32/tests/sock.c +++ b/dlls/ws2_32/tests/sock.c @@ -3257,6 +3257,81 @@ end: closesocket(connector2); } +static void test_getpeername(void) +{ + SOCKET sock; + struct sockaddr_in sa, sa_out; + int sa_len; + const char buf[] = "hello world"; + int ret; + + /* Test the parameter validation order. */ + ret = getpeername(INVALID_SOCKET, NULL, NULL); + ok(ret == SOCKET_ERROR, "Expected getpeername to return SOCKET_ERROR, got %d\n", ret); + ok(WSAGetLastError() == WSAENOTSOCK, + "Expected WSAGetLastError() to return WSAENOTSOCK, got %d\n", WSAGetLastError()); + + sock = socket(PF_INET, SOCK_DGRAM, IPPROTO_IP); + ok(sock != INVALID_SOCKET, "Expected socket to return a valid socket\n"); + if (sock == INVALID_SOCKET) + { + skip("Socket creation failed with %d\n", WSAGetLastError()); + return; + } + + ret = getpeername(sock, NULL, NULL); + ok(ret == SOCKET_ERROR, "Expected getpeername to return SOCKET_ERROR, got %d\n", ret); + ok(WSAGetLastError() == WSAENOTCONN, + "Expected WSAGetLastError() to return WSAENOTCONN, got %d\n", WSAGetLastError()); + + memset(&sa, 0, sizeof(sa)); + sa.sin_family = AF_INET; + sa.sin_port = htons(139); + sa.sin_addr.s_addr = inet_addr("127.0.0.1"); + + /* sendto does not change a socket's connection state. */ + ret = sendto(sock, buf, sizeof(buf), 0, (struct sockaddr*)&sa, sizeof(sa)); + ok(ret != SOCKET_ERROR, + "Expected sendto to succeed, WSAGetLastError() = %d\n", WSAGetLastError()); + + ret = getpeername(sock, NULL, NULL); + ok(ret == SOCKET_ERROR, "Expected getpeername to return SOCKET_ERROR, got %d\n", ret); + ok(WSAGetLastError() == WSAENOTCONN, + "Expected WSAGetLastError() to return WSAENOTCONN, got %d\n", WSAGetLastError()); + + ret = connect(sock, (struct sockaddr*)&sa, sizeof(sa)); + ok(ret == 0, + "Expected connect to succeed, WSAGetLastError() = %d\n", WSAGetLastError()); + + ret = getpeername(sock, NULL, NULL); + ok(ret == SOCKET_ERROR, "Expected getpeername to return SOCKET_ERROR, got %d\n", ret); + ok(WSAGetLastError() == WSAEFAULT, + "Expected WSAGetLastError() to return WSAEFAULT, got %d\n", WSAGetLastError()); + + /* Test crashes on Wine. */ + if (0) + { + ret = getpeername(sock, (void*)0xdeadbeef, (void*)0xcafebabe); + ok(ret == SOCKET_ERROR, "Expected getpeername to return SOCKET_ERROR, got %d\n", ret); + ok(WSAGetLastError() == WSAEFAULT, + "Expected WSAGetLastError() to return WSAEFAULT, got %d\n", WSAGetLastError()); + } + + sa_len = 0; + ret = getpeername(sock, (struct sockaddr*)&sa_out, &sa_len); + ok(ret == SOCKET_ERROR, "Expected getpeername to return 0, got %d\n", ret); + ok(WSAGetLastError() == WSAEFAULT, + "Expected WSAGetLastError() to return WSAEFAULT, got %d\n", WSAGetLastError()); + + sa_len = sizeof(sa_out); + ret = getpeername(sock, (struct sockaddr*)&sa_out, &sa_len); + ok(ret == 0, "Expected getpeername to return 0, got %d\n", ret); + ok(!memcmp(&sa, &sa_out, sizeof(sa)), + "Expected the returned structure to be identical to the connect structure\n"); + + closesocket(sock); +} + /**************** Main program ***************/ START_TEST( sock ) @@ -3294,6 +3369,7 @@ START_TEST( sock ) test_select(); test_accept(); + test_getpeername(); test_getsockname(); test_inet_addr(); test_addr_to_print();

1 0

Andrew Nguyen : msnet32: Add copyright author information.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: f1b30eb49d5ac61afed0d88d4f5bc59e60967eba URL: http://source.winehq.org/git/wine.git/?a=commit;h=f1b30eb49d5ac61afed0d88d4… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Wed Feb 3 01:28:37 2010 -0600 msnet32: Add copyright author information. --- dlls/msnet32/msnet_main.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/dlls/msnet32/msnet_main.c b/dlls/msnet32/msnet_main.c index 7c75900..1ffef50 100644 --- a/dlls/msnet32/msnet_main.c +++ b/dlls/msnet32/msnet_main.c @@ -1,4 +1,9 @@ /* + * Microsoft Network API implementation + * + * Copyright 2003 Rein Klazes + * Copyright 2006 Alexandre Julliard + * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public * License as published by the Free Software Foundation; either

1 0

Christian Costa : d3drm/tests: Use better name for test function and make it static ( spotted by Stefan Dosinger).
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 3d361e64127db24b8188e43bed8681cca5a751ee URL: http://source.winehq.org/git/wine.git/?a=commit;h=3d361e64127db24b8188e43be… Author: Christian Costa <titan.costa(a)wanadoo.fr> Date: Wed Feb 3 08:38:41 2010 +0100 d3drm/tests: Use better name for test function and make it static (spotted by Stefan Dosinger). --- dlls/d3drm/tests/d3drm.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/d3drm/tests/d3drm.c b/dlls/d3drm/tests/d3drm.c index 9e74c12..3a89aa3 100644 --- a/dlls/d3drm/tests/d3drm.c +++ b/dlls/d3drm/tests/d3drm.c @@ -62,7 +62,7 @@ char data_bad[] = "1; 2; 3;\n" "}\n"; -void Test(void) +static void MeshBuilderTest(void) { HRESULT hr; LPDIRECT3DRM pD3DRM; @@ -95,7 +95,7 @@ START_TEST(d3drm) if (!InitFunctionPtrs()) return; - Test(); + MeshBuilderTest(); FreeLibrary(d3drm_handle); }

1 0

Hans Leidekker : msi: Add a stub implementation of the SetODBCFolders standard action.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 552a9c9b85f34ecf4ab32fbe853f791c9f408815 URL: http://source.winehq.org/git/wine.git/?a=commit;h=552a9c9b85f34ecf4ab32fbe8… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Feb 3 09:19:56 2010 +0100 msi: Add a stub implementation of the SetODBCFolders standard action. --- dlls/msi/action.c | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index 462b9ff..4b2d5b4 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -6081,6 +6081,12 @@ static UINT ACTION_RemoveShortcuts( MSIPACKAGE *package ) return msi_unimplemented_action_stub( package, "RemoveShortcuts", table ); } +static UINT ACTION_SetODBCFolders( MSIPACKAGE *package ) +{ + static const WCHAR table[] = { 'D','i','r','e','c','t','o','r','y',0 }; + return msi_unimplemented_action_stub( package, "SetODBCFolders", table ); +} + static UINT ACTION_UnpublishComponents( MSIPACKAGE *package ) { static const WCHAR table[] = { 'P','u','b','l','i','s','h','C','o','m','p','o','n','e','n','t',0 }; @@ -6185,7 +6191,7 @@ StandardActions[] = { szScheduleReboot, NULL }, { szSelfRegModules, ACTION_SelfRegModules }, { szSelfUnregModules, ACTION_SelfUnregModules }, - { szSetODBCFolders, NULL }, + { szSetODBCFolders, ACTION_SetODBCFolders }, { szStartServices, ACTION_StartServices }, { szStopServices, ACTION_StopServices }, { szUnpublishComponents, ACTION_UnpublishComponents },

1 0

Alexandre Julliard : atl: The object map entries for version 1 don' t have the last two fields.
by Alexandre Julliard 03 Feb '10

03 Feb '10

Module: wine Branch: master Commit: 4356e4941192332d4e927efc392cb8fd12948770 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4356e4941192332d4e927efc3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Feb 2 20:13:56 2010 +0100 atl: The object map entries for version 1 don't have the last two fields. --- dlls/atl/atl_main.c | 32 +++++++++++++++++++++----------- dlls/atl/atlbase.h | 22 ++++++++++++++++++++++ 2 files changed, 43 insertions(+), 11 deletions(-) diff --git a/dlls/atl/atl_main.c b/dlls/atl/atl_main.c index 374b0d2..75f9a11 100644 --- a/dlls/atl/atl_main.c +++ b/dlls/atl/atl_main.c @@ -97,6 +97,19 @@ HRESULT WINAPI AtlModuleInit(_ATL_MODULEW* pM, _ATL_OBJMAP_ENTRYW* p, HINSTANCE return S_OK; } +static _ATL_OBJMAP_ENTRYW_V1 *get_objmap_entry( _ATL_MODULEW *mod, unsigned int index ) +{ + _ATL_OBJMAP_ENTRYW_V1 *ret; + + if (mod->cbSize == ATLVer1Size) + ret = (_ATL_OBJMAP_ENTRYW_V1 *)mod->m_pObjMap + index; + else + ret = (_ATL_OBJMAP_ENTRYW_V1 *)(mod->m_pObjMap + index); + + if (!ret->pclsid) ret = NULL; + return ret; +} + HRESULT WINAPI AtlModuleLoadTypeLib(_ATL_MODULEW *pM, LPCOLESTR lpszIndex, BSTR *pbstrPath, ITypeLib **ppTypeLib) { @@ -158,6 +171,7 @@ HRESULT WINAPI AtlModuleAddTermFunc(_ATL_MODULEW *pM, _ATL_TERMFUNC *pFunc, DWOR HRESULT WINAPI AtlModuleRegisterClassObjects(_ATL_MODULEW *pM, DWORD dwClsContext, DWORD dwFlags) { + _ATL_OBJMAP_ENTRYW_V1 *obj; HRESULT hRes = S_OK; int i=0; @@ -166,10 +180,9 @@ HRESULT WINAPI AtlModuleRegisterClassObjects(_ATL_MODULEW *pM, DWORD dwClsContex if (pM == NULL) return E_INVALIDARG; - while(pM->m_pObjMap[i].pclsid != NULL) + while ((obj = get_objmap_entry( pM, i++ ))) { IUnknown* pUnknown; - _ATL_OBJMAP_ENTRYW *obj = &(pM->m_pObjMap[i]); HRESULT rc; TRACE("Registering object %i\n",i); @@ -185,7 +198,6 @@ HRESULT WINAPI AtlModuleRegisterClassObjects(_ATL_MODULEW *pM, DWORD dwClsContex IUnknown_Release(pUnknown); } } - i++; } return hRes; @@ -269,6 +281,7 @@ HRESULT WINAPI AtlInternalQueryInterface(void* this, const _ATL_INTMAP_ENTRY* pE */ HRESULT WINAPI AtlModuleRegisterServer(_ATL_MODULEW* pM, BOOL bRegTypeLib, const CLSID* clsid) { + const _ATL_OBJMAP_ENTRYW_V1 *obj; int i; HRESULT hRes; @@ -277,12 +290,10 @@ HRESULT WINAPI AtlModuleRegisterServer(_ATL_MODULEW* pM, BOOL bRegTypeLib, const if (pM == NULL) return E_INVALIDARG; - for (i = 0; pM->m_pObjMap[i].pclsid != NULL; i++) /* register CLSIDs */ + for (i = 0; (obj = get_objmap_entry( pM, i )) != NULL; i++) /* register CLSIDs */ { - if (!clsid || IsEqualCLSID(pM->m_pObjMap[i].pclsid, clsid)) + if (!clsid || IsEqualCLSID(obj->pclsid, clsid)) { - const _ATL_OBJMAP_ENTRYW *obj = &pM->m_pObjMap[i]; - TRACE("Registering clsid %s\n", debugstr_guid(obj->pclsid)); hRes = obj->pfnUpdateRegistry(TRUE); /* register */ if (FAILED(hRes)) @@ -351,6 +362,7 @@ HRESULT WINAPI AtlUnmarshalPtr(IStream *stm, const IID *iid, IUnknown **ppUnk) HRESULT WINAPI AtlModuleGetClassObject(_ATL_MODULEW *pm, REFCLSID rclsid, REFIID riid, LPVOID *ppv) { + _ATL_OBJMAP_ENTRYW_V1 *obj; int i; HRESULT hres = CLASS_E_CLASSNOTAVAILABLE; @@ -359,12 +371,10 @@ HRESULT WINAPI AtlModuleGetClassObject(_ATL_MODULEW *pm, REFCLSID rclsid, if (pm == NULL) return E_INVALIDARG; - for (i = 0; pm->m_pObjMap[i].pclsid != NULL; i++) + for (i = 0; (obj = get_objmap_entry( pm, i )) != NULL; i++) { - if (IsEqualCLSID(pm->m_pObjMap[i].pclsid, rclsid)) + if (IsEqualCLSID(obj->pclsid, rclsid)) { - _ATL_OBJMAP_ENTRYW *obj = &pm->m_pObjMap[i]; - TRACE("found object %i\n", i); if (obj->pfnGetClassObject) { diff --git a/dlls/atl/atlbase.h b/dlls/atl/atlbase.h index 49c13e4..59cdddf 100644 --- a/dlls/atl/atlbase.h +++ b/dlls/atl/atlbase.h @@ -33,6 +33,28 @@ typedef LPCWSTR (WINAPI _ATL_DESCRIPTIONFUNCW)(void); typedef const struct _ATL_CATMAP_ENTRY* (_ATL_CATMAPFUNC)(void); typedef void (WINAPI _ATL_TERMFUNC)(DWORD dw); +typedef struct _ATL_OBJMAP_ENTRYA_V1_TAG +{ + const CLSID* pclsid; + HRESULT (WINAPI *pfnUpdateRegistry)(BOOL bRegister); + _ATL_CREATORFUNC* pfnGetClassObject; + _ATL_CREATORFUNC* pfnCreateInstance; + IUnknown* pCF; + DWORD dwRegister; + _ATL_DESCRIPTIONFUNCA* pfnGetObjectDescription; +}_ATL_OBJMAP_ENTRYA_V1; + +typedef struct _ATL_OBJMAP_ENTRYW_V1_TAG +{ + const CLSID* pclsid; + HRESULT (WINAPI *pfnUpdateRegistry)(BOOL bRegister); + _ATL_CREATORFUNC* pfnGetClassObject; + _ATL_CREATORFUNC* pfnCreateInstance; + IUnknown* pCF; + DWORD dwRegister; + _ATL_DESCRIPTIONFUNCW* pfnGetObjectDescription; +} _ATL_OBJMAP_ENTRYW_V1; + typedef struct _ATL_OBJMAP_ENTRYA_TAG { const CLSID* pclsid;

1 0