wine-commits April 2008

wine-commits@winehq.org

4 participants
1278 discussions

Alexandre Julliard : imm32: Remove stray semicolon. Don' t try to load an invalid file.
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: 01d167be7c863c41e51deeace17ce7df044a9809 URL: http://source.winehq.org/git/wine.git/?a=commit;h=01d167be7c863c41e51deeace… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Apr 1 23:17:21 2008 +0200 imm32: Remove stray semicolon. Don't try to load an invalid file. --- dlls/imm32/imm.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c index 9173d1b..866c3e3 100644 --- a/dlls/imm32/imm.c +++ b/dlls/imm32/imm.c @@ -139,8 +139,7 @@ static ImmHkl *IMM_GetImmHkl(HKL hkl) ptr = HeapAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,sizeof(ImmHkl)); ptr->hkl = hkl; - ImmGetIMEFileNameW(hkl, filename, MAX_PATH); - ptr->hIME = LoadLibraryW(filename); + if (ImmGetIMEFileNameW(hkl, filename, MAX_PATH)) ptr->hIME = LoadLibraryW(filename); if (ptr->hIME) { LOAD_FUNCPTR(ImeInquire); @@ -179,7 +178,7 @@ static void IMM_FreeAllImmHkl(void) { ImmHkl *ptr,*cursor2; - LIST_FOR_EACH_ENTRY_SAFE(ptr, cursor2, &ImmHklList, ImmHkl, entry); + LIST_FOR_EACH_ENTRY_SAFE(ptr, cursor2, &ImmHklList, ImmHkl, entry) { list_remove(&ptr->entry); if (ptr->hIME)

1 0

Stefan Dösinger : wined3d: Catch D3DUSAGE_AUTOGENMIPMAP and D3DUSAGE_RENDERTARGET combos.
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: ab86a4e963ac30e56d9b6f21a3cd1b1be92fc7e1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ab86a4e963ac30e56d9b6f21a… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri Mar 28 23:46:38 2008 +0100 wined3d: Catch D3DUSAGE_AUTOGENMIPMAP and D3DUSAGE_RENDERTARGET combos. --- dlls/d3d9/tests/texture.c | 4 ++-- dlls/wined3d/device.c | 13 ++++++++++++- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/dlls/d3d9/tests/texture.c b/dlls/d3d9/tests/texture.c index 0b2e406..72a0561 100644 --- a/dlls/d3d9/tests/texture.c +++ b/dlls/d3d9/tests/texture.c @@ -137,7 +137,7 @@ static void test_cube_texture_mipmap_gen(IDirect3DDevice9 *device_ptr) hr = IDirect3DDevice9_CreateCubeTexture(device_ptr, 64, 0, (D3DUSAGE_RENDERTARGET | D3DUSAGE_AUTOGENMIPMAP), D3DFMT_X8R8G8B8, D3DPOOL_MANAGED, &texture_ptr, 0); - todo_wine ok(hr == D3DERR_INVALIDCALL, "IDirect3DDevice9_CreateTexture returned 0x%08x, expected 0x%08x\n", + ok(hr == D3DERR_INVALIDCALL, "IDirect3DDevice9_CreateTexture returned 0x%08x, expected 0x%08x\n", hr, D3DERR_INVALIDCALL); if (texture_ptr) IDirect3DCubeTexture9_Release(texture_ptr); texture_ptr = NULL; @@ -186,7 +186,7 @@ static void test_mipmap_gen(IDirect3DDevice9 *device) hr = IDirect3DDevice9_CreateTexture(device, 64, 64, 0, (D3DUSAGE_RENDERTARGET | D3DUSAGE_AUTOGENMIPMAP), D3DFMT_X8R8G8B8, D3DPOOL_MANAGED, &texture, 0); - todo_wine ok(hr == D3DERR_INVALIDCALL, "IDirect3DDevice9_CreateTexture returned 0x%08x, expected 0x%08x\n", + ok(hr == D3DERR_INVALIDCALL, "IDirect3DDevice9_CreateTexture returned 0x%08x, expected 0x%08x\n", hr, D3DERR_INVALIDCALL); if (texture) IDirect3DTexture9_Release(texture); texture = NULL; diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 57b6045..1c1da5a 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -768,11 +768,16 @@ static HRESULT WINAPI IWineD3DDeviceImpl_CreateTexture(IWineD3DDevice *iface, U const GlPixelFormatDesc *glDesc; getFormatDescEntry(Format, &GLINFO_LOCATION, &glDesc); - TRACE("(%p) : Width %d, Height %d, Levels %d, Usage %#x\n", This, Width, Height, Levels, Usage); TRACE("Format %#x (%s), Pool %#x, ppTexture %p, pSharedHandle %p, parent %p\n", Format, debug_d3dformat(Format), Pool, ppTexture, pSharedHandle, parent); + if((Usage & (WINED3DUSAGE_AUTOGENMIPMAP | WINED3DUSAGE_RENDERTARGET)) == + (WINED3DUSAGE_AUTOGENMIPMAP | WINED3DUSAGE_RENDERTARGET)) { + WARN("Application requests both D3DUSAGE_AUTOGENMIPMAP and D3DUSAGE_RENDERTARGET, which are mutually exclusive\n"); + return WINED3DERR_INVALIDCALL; + } + /* TODO: It should only be possible to create textures for formats that are reported as supported */ if (WINED3DFMT_UNKNOWN >= Format) { @@ -1039,6 +1044,12 @@ static HRESULT WINAPI IWineD3DDeviceImpl_CreateCubeTexture(IWineD3DDevice *iface const GlPixelFormatDesc *glDesc; getFormatDescEntry(Format, &GLINFO_LOCATION, &glDesc); + if((Usage & (WINED3DUSAGE_AUTOGENMIPMAP | WINED3DUSAGE_RENDERTARGET)) == + (WINED3DUSAGE_AUTOGENMIPMAP | WINED3DUSAGE_RENDERTARGET)) { + WARN("Application requests both D3DUSAGE_AUTOGENMIPMAP and D3DUSAGE_RENDERTARGET, which are mutually exclusive\n"); + return WINED3DERR_INVALIDCALL; + } + /* TODO: It should only be possible to create textures for formats that are reported as supported */ if (WINED3DFMT_UNKNOWN >= Format) {

1 0

Stefan Dösinger : wined3d: Refuse 3D initialization without opengl.
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: a29a22932434c0c3768a5eb595ef50cd0f59b110 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a29a22932434c0c3768a5eb59… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri Mar 28 23:04:52 2008 +0100 wined3d: Refuse 3D initialization without opengl. --- dlls/wined3d/device.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index ee27abf..57b6045 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -2010,6 +2010,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_Init3D(IWineD3DDevice *iface, WINED3DPR TRACE("(%p)->(%p,%p)\n", This, pPresentationParameters, D3DCB_CreateAdditionalSwapChain); if(This->d3d_initialized) return WINED3DERR_INVALIDCALL; + if(!This->adapter->opengl) return WINED3DERR_INVALIDCALL; /* TODO: Test if OpenGL is compiled in and loaded */

1 0

Stefan Dösinger : wined3d: Return a fake pixel format if gl is not loaded.
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: 8673be0bd7579eb6440fc2d0b1c60c93f3067c64 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8673be0bd7579eb6440fc2d0b… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri Mar 28 23:04:34 2008 +0100 wined3d: Return a fake pixel format if gl is not loaded. --- dlls/wined3d/utils.c | 13 +++++++++---- 1 files changed, 9 insertions(+), 4 deletions(-) diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index 95abe20..d0285df 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -437,11 +437,16 @@ const StaticPixelFormatDesc *getFormatDescEntry(WINED3DFORMAT fmt, WineD3D_GL_In idx = getFmtIdx(WINED3DFMT_UNKNOWN); } if(glDesc) { - if(!gl_info) { - ERR("OpenGL pixel format information was requested, but no gl info structure passed\n"); - return NULL; + if(!gl_info->gl_formats) { + /* If we do not have gl format information, provide a dummy NULL format. This is an easy way to make + * all gl caps check return "unsupported" than catching the lack of gl all over the code. ANSI C requires + * static variables to be initialized to 0. + */ + static const GlPixelFormatDesc dummyFmt; + *glDesc = &dummyFmt; + } else { + *glDesc = &gl_info->gl_formats[idx]; } - *glDesc = &gl_info->gl_formats[idx]; } return &formats[idx]; }

1 0

Stefan Dösinger : wined3d: Create a fake non-gl adapter.
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: 9e831a873300e30340e8fb9446060be9ad55d6e7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e831a873300e30340e8fb944… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Fri Mar 28 23:04:17 2008 +0100 wined3d: Create a fake non-gl adapter. --- dlls/wined3d/directx.c | 38 +++++++++++++++++++++++++++----------- dlls/wined3d/wined3d_private.h | 1 + 2 files changed, 28 insertions(+), 11 deletions(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index a30955c..6baf8d3 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -3676,7 +3676,7 @@ BOOL InitAdapters(void) { /* No need to hold any lock. The calling library makes sure only one thread calls * wined3d simultaneously */ - if(numAdapters > 0) return TRUE; + if(numAdapters > 0) return Adapters[0].opengl; TRACE("Initializing adapters\n"); @@ -3686,7 +3686,7 @@ BOOL InitAdapters(void) { mod_gl = LoadLibraryA("opengl32.dll"); if(!mod_gl) { ERR("Can't load opengl32.dll!\n"); - return FALSE; + goto nogl_adapter; } #else #define USE_GL_FUNC(pfn) pfn = (void*)pwglGetProcAddress(#pfn); @@ -3702,7 +3702,7 @@ BOOL InitAdapters(void) { if(!pwglGetProcAddress) { ERR("Unable to load wglGetProcAddress!\n"); - return FALSE; + goto nogl_adapter; } /* Dynamically load all GL core functions */ @@ -3728,32 +3728,28 @@ BOOL InitAdapters(void) { if (!WineD3D_CreateFakeGLContext()) { ERR("Failed to get a gl context for default adapter\n"); - HeapFree(GetProcessHeap(), 0, Adapters); WineD3D_ReleaseFakeGLContext(); - return FALSE; + goto nogl_adapter; } ret = IWineD3DImpl_FillGLCaps(&Adapters[0].gl_info); if(!ret) { ERR("Failed to initialize gl caps for default adapter\n"); - HeapFree(GetProcessHeap(), 0, Adapters); WineD3D_ReleaseFakeGLContext(); - return FALSE; + goto nogl_adapter; } ret = initPixelFormats(&Adapters[0].gl_info); if(!ret) { ERR("Failed to init gl formats\n"); - HeapFree(GetProcessHeap(), 0, Adapters); WineD3D_ReleaseFakeGLContext(); - return FALSE; + goto nogl_adapter; } hdc = pwglGetCurrentDC(); if(!hdc) { ERR("Failed to get gl HDC\n"); - HeapFree(GetProcessHeap(), 0, Adapters); WineD3D_ReleaseFakeGLContext(); - return FALSE; + goto nogl_adapter; } Adapters[0].driver = "Display"; @@ -3826,11 +3822,31 @@ attributes. */ select_shader_max_constants(ps_selected_mode, vs_selected_mode, &Adapters[0].gl_info); fillGLAttribFuncs(&Adapters[0].gl_info); init_type_lookup(&Adapters[0].gl_info); + Adapters[0].opengl = TRUE; } numAdapters = 1; TRACE("%d adapters successfully initialized\n", numAdapters); return TRUE; + +nogl_adapter: + /* Initialize an adapter for ddraw-only memory counting */ + memset(Adapters, 0, sizeof(Adapters)); + Adapters[0].num = 0; + Adapters[0].opengl = FALSE; + Adapters[0].monitorPoint.x = -1; + Adapters[0].monitorPoint.y = -1; + + Adapters[0].driver = "Display"; + Adapters[0].description = "WineD3D DirectDraw Emulation"; + if(wined3d_settings.emulated_textureram) { + Adapters[0].TextureRam = wined3d_settings.emulated_textureram; + } else { + Adapters[0].TextureRam = 8 * 1024 * 1024; /* This is plenty for a DDraw-only card */ + } + + numAdapters = 1; + return FALSE; } #undef PUSH1 #undef GLINFO_LOCATION diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 0abeb1d..c3f4910 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -682,6 +682,7 @@ typedef struct GLPixelFormatDesc GLPixelFormatDesc; struct WineD3DAdapter { UINT num; + BOOL opengl; POINT monitorPoint; WineD3D_GL_Info gl_info; const char *driver;

1 0

Rob Shearman : advapi32: Remove some unused services functions.
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: 9d834fc180df2d126334a98bc1f4ea245434ebae URL: http://source.winehq.org/git/wine.git/?a=commit;h=9d834fc180df2d126334a98bc… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 1 12:48:34 2008 +0100 advapi32: Remove some unused services functions. --- dlls/advapi32/service.c | 79 ----------------------------------------------- 1 files changed, 0 insertions(+), 79 deletions(-) diff --git a/dlls/advapi32/service.c b/dlls/advapi32/service.c index 0d13442..c95fcd9 100644 --- a/dlls/advapi32/service.c +++ b/dlls/advapi32/service.c @@ -340,85 +340,6 @@ void __RPC_USER MACHINE_HANDLEW_unbind(MACHINE_HANDLEW MachineName, handle_t h) } /****************************************************************************** - * registry access functions and data - */ -static const WCHAR szDisplayName[] = { - 'D','i','s','p','l','a','y','N','a','m','e', 0 }; -static const WCHAR szType[] = {'T','y','p','e',0}; -static const WCHAR szStart[] = {'S','t','a','r','t',0}; -static const WCHAR szError[] = { - 'E','r','r','o','r','C','o','n','t','r','o','l', 0}; -static const WCHAR szImagePath[] = {'I','m','a','g','e','P','a','t','h',0}; -static const WCHAR szGroup[] = {'G','r','o','u','p',0}; -static const WCHAR szDependencies[] = { - 'D','e','p','e','n','d','e','n','c','i','e','s',0}; -static const WCHAR szDependOnService[] = { - 'D','e','p','e','n','d','O','n','S','e','r','v','i','c','e',0}; -static const WCHAR szObjectName[] = { - 'O','b','j','e','c','t','N','a','m','e',0}; -static const WCHAR szTag[] = { - 'T','a','g',0}; - -struct reg_value { - DWORD type; - DWORD size; - LPCWSTR name; - LPCVOID data; -}; - -static inline void service_set_value( struct reg_value *val, - DWORD type, LPCWSTR name, LPCVOID data, DWORD size ) -{ - val->name = name; - val->type = type; - val->data = data; - val->size = size; -} - -static inline void service_set_dword( struct reg_value *val, - LPCWSTR name, const DWORD *data ) -{ - service_set_value( val, REG_DWORD, name, data, sizeof (DWORD)); -} - -static inline void service_set_string( struct reg_value *val, - LPCWSTR name, LPCWSTR string ) -{ - DWORD len = (lstrlenW(string)+1) * sizeof (WCHAR); - service_set_value( val, REG_SZ, name, string, len ); -} - -static inline void service_set_multi_string( struct reg_value *val, - LPCWSTR name, LPCWSTR string ) -{ - DWORD len = 0; - - /* determine the length of a double null terminated multi string */ - do { - len += (lstrlenW( &string[ len ] )+1); - } while ( string[ len++ ] ); - - len *= sizeof (WCHAR); - service_set_value( val, REG_MULTI_SZ, name, string, len ); -} - -static inline LONG service_write_values( HKEY hKey, - const struct reg_value *val, int n ) -{ - LONG r = ERROR_SUCCESS; - int i; - - for( i=0; i<n; i++ ) - { - r = RegSetValueExW(hKey, val[i].name, 0, val[i].type, - (const BYTE*)val[i].data, val[i].size ); - if( r != ERROR_SUCCESS ) - break; - } - return r; -} - -/****************************************************************************** * Service IPC functions */ static LPWSTR service_get_pipe_name(void)

1 0

Rob Shearman : services: Fix memory leak of some data in the service record .
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: 0b7a37acdcb883eaaf83073aef01834261d14e1c URL: http://source.winehq.org/git/wine.git/?a=commit;h=0b7a37acdcb883eaaf83073ae… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 1 12:48:28 2008 +0100 services: Fix memory leak of some data in the service record. --- programs/services/services.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/programs/services/services.c b/programs/services/services.c index 80a0a2d..0177076 100644 --- a/programs/services/services.c +++ b/programs/services/services.c @@ -83,6 +83,8 @@ void free_service_entry(struct service_entry *entry) HeapFree(GetProcessHeap(), 0, entry->config.lpServiceStartName); HeapFree(GetProcessHeap(), 0, entry->config.lpDisplayName); HeapFree(GetProcessHeap(), 0, entry->description); + HeapFree(GetProcessHeap(), 0, entry->dependOnServices); + HeapFree(GetProcessHeap(), 0, entry->dependOnGroups); CloseHandle(entry->control_mutex); CloseHandle(entry->control_pipe); CloseHandle(entry->status_changed_event);

1 0

Rob Shearman : rpcrt4: Add a reference to the binding object in I_RpcNegotiateTransferSyntax and release the reference in I_RpcFreeBuffer .
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: 8aeb2858e44627c2002b9a20483a2a9c78a2aa3f URL: http://source.winehq.org/git/wine.git/?a=commit;h=8aeb2858e44627c2002b9a204… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 1 12:48:15 2008 +0100 rpcrt4: Add a reference to the binding object in I_RpcNegotiateTransferSyntax and release the reference in I_RpcFreeBuffer. This is needed because a context binding handle could be released on unmarshall, but it still needs to stay valid until the binding handle is no longer being used. Re-use the previously unused RPCRT4_ExportBinding function as RPCRT4_AddRefBinding and rename RPCRT4_DestroyBinding to RPCRT4_ReleaseBinding to show that it's purpose is to release a reference count and destroy if necessary, not always destroy. --- dlls/rpcrt4/rpc_binding.c | 14 ++++++-------- dlls/rpcrt4/rpc_binding.h | 4 ++-- dlls/rpcrt4/rpc_message.c | 4 ++++ dlls/rpcrt4/rpc_transport.c | 2 +- 4 files changed, 13 insertions(+), 11 deletions(-) diff --git a/dlls/rpcrt4/rpc_binding.c b/dlls/rpcrt4/rpc_binding.c index 7293032..d670a88 100644 --- a/dlls/rpcrt4/rpc_binding.c +++ b/dlls/rpcrt4/rpc_binding.c @@ -249,14 +249,12 @@ RPC_STATUS RPCRT4_MakeBinding(RpcBinding** Binding, RpcConnection* Connection) return RPC_S_OK; } -RPC_STATUS RPCRT4_ExportBinding(RpcBinding** Binding, RpcBinding* OldBinding) +void RPCRT4_AddRefBinding(RpcBinding* Binding) { - InterlockedIncrement(&OldBinding->refs); - *Binding = OldBinding; - return RPC_S_OK; + InterlockedIncrement(&Binding->refs); } -RPC_STATUS RPCRT4_DestroyBinding(RpcBinding* Binding) +RPC_STATUS RPCRT4_ReleaseBinding(RpcBinding* Binding) { if (InterlockedDecrement(&Binding->refs)) return RPC_S_OK; @@ -661,7 +659,7 @@ RPC_STATUS WINAPI RpcBindingFree( RPC_BINDING_HANDLE* Binding ) { RPC_STATUS status; TRACE("(%p) = %p\n", Binding, *Binding); - status = RPCRT4_DestroyBinding(*Binding); + status = RPCRT4_ReleaseBinding(*Binding); if (status == RPC_S_OK) *Binding = 0; return status; } @@ -741,7 +739,7 @@ RPC_STATUS WINAPI RpcBindingFromStringBindingA( RPC_CSTR StringBinding, RPC_BIND if (ret == RPC_S_OK) *Binding = (RPC_BINDING_HANDLE)bind; else - RPCRT4_DestroyBinding(bind); + RPCRT4_ReleaseBinding(bind); return ret; } @@ -780,7 +778,7 @@ RPC_STATUS WINAPI RpcBindingFromStringBindingW( RPC_WSTR StringBinding, RPC_BIND if (ret == RPC_S_OK) *Binding = (RPC_BINDING_HANDLE)bind; else - RPCRT4_DestroyBinding(bind); + RPCRT4_ReleaseBinding(bind); return ret; } diff --git a/dlls/rpcrt4/rpc_binding.h b/dlls/rpcrt4/rpc_binding.h index e169a65..3251df2 100644 --- a/dlls/rpcrt4/rpc_binding.h +++ b/dlls/rpcrt4/rpc_binding.h @@ -145,8 +145,8 @@ RPC_STATUS RPCRT4_SpawnConnection(RpcConnection** Connection, RpcConnection* Old RPC_STATUS RPCRT4_ResolveBinding(RpcBinding* Binding, LPCSTR Endpoint); RPC_STATUS RPCRT4_SetBindingObject(RpcBinding* Binding, const UUID* ObjectUuid); RPC_STATUS RPCRT4_MakeBinding(RpcBinding** Binding, RpcConnection* Connection); -RPC_STATUS RPCRT4_ExportBinding(RpcBinding** Binding, RpcBinding* OldBinding); -RPC_STATUS RPCRT4_DestroyBinding(RpcBinding* Binding); +void RPCRT4_AddRefBinding(RpcBinding* Binding); +RPC_STATUS RPCRT4_ReleaseBinding(RpcBinding* Binding); RPC_STATUS RPCRT4_OpenBinding(RpcBinding* Binding, RpcConnection** Connection, const RPC_SYNTAX_IDENTIFIER *TransferSyntax, const RPC_SYNTAX_IDENTIFIER *InterfaceId); RPC_STATUS RPCRT4_CloseBinding(RpcBinding* Binding, RpcConnection* Connection); diff --git a/dlls/rpcrt4/rpc_message.c b/dlls/rpcrt4/rpc_message.c index bc33c87..3b1e68d 100644 --- a/dlls/rpcrt4/rpc_message.c +++ b/dlls/rpcrt4/rpc_message.c @@ -1017,7 +1017,10 @@ RPC_STATUS WINAPI I_RpcNegotiateTransferSyntax(PRPC_MESSAGE pMsg) &cif->InterfaceId); if (status == RPC_S_OK) + { pMsg->ReservedForRuntime = conn; + RPCRT4_AddRefBinding(bind); + } } return status; @@ -1114,6 +1117,7 @@ RPC_STATUS WINAPI I_RpcFreeBuffer(PRPC_MESSAGE pMsg) { RpcConnection *conn = pMsg->ReservedForRuntime; RPCRT4_CloseBinding(bind, conn); + RPCRT4_ReleaseBinding(bind); pMsg->ReservedForRuntime = NULL; } I_RpcFree(pMsg->Buffer); diff --git a/dlls/rpcrt4/rpc_transport.c b/dlls/rpcrt4/rpc_transport.c index 822807b..8bca0fe 100644 --- a/dlls/rpcrt4/rpc_transport.c +++ b/dlls/rpcrt4/rpc_transport.c @@ -1551,7 +1551,7 @@ RPC_STATUS RPCRT4_DestroyConnection(RpcConnection* Connection) if (Connection->QOS) RpcQualityOfService_Release(Connection->QOS); /* server-only */ - if (Connection->server_binding) RPCRT4_DestroyBinding(Connection->server_binding); + if (Connection->server_binding) RPCRT4_ReleaseBinding(Connection->server_binding); HeapFree(GetProcessHeap(), 0, Connection); return RPC_S_OK;

1 0

Rob Shearman : rpcrt4: Improve trace in RPCRT4_find_interface to print the interface identifier data as well as the object guid .
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: c7f9b9347fdf7a073329d9b3fada993c2ea74717 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c7f9b9347fdf7a073329d9b3f… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 1 12:48:06 2008 +0100 rpcrt4: Improve trace in RPCRT4_find_interface to print the interface identifier data as well as the object guid. --- dlls/rpcrt4/rpc_server.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/rpcrt4/rpc_server.c b/dlls/rpcrt4/rpc_server.c index 40e9e7e..f93d264 100644 --- a/dlls/rpcrt4/rpc_server.c +++ b/dlls/rpcrt4/rpc_server.c @@ -138,7 +138,9 @@ static RpcServerInterface* RPCRT4_find_interface(UUID* object, } LeaveCriticalSection(&server_cs); if (&cif->entry == &server_interfaces) cif = NULL; - TRACE("returning %p for %s\n", cif, debugstr_guid(object)); + TRACE("returning %p for object %s, if_id { %d.%d %s }\n", cif, + debugstr_guid(object), if_id->SyntaxVersion.MajorVersion, + if_id->SyntaxVersion.MinorVersion, debugstr_guid(&if_id->SyntaxGUID)); return cif; }

1 0

Rob Shearman : widl: Be more strict about where semicolons can appear in IDL files.
by Alexandre Julliard 01 Apr '08

01 Apr '08

Module: wine Branch: master Commit: 772f3af9f0ac5a28fde31641c497fb02dbab110c URL: http://source.winehq.org/git/wine.git/?a=commit;h=772f3af9f0ac5a28fde31641c… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Apr 1 12:47:56 2008 +0100 widl: Be more strict about where semicolons can appear in IDL files. In particular, don't allow them after cpp_quote and without a statement. Update the rules for importlib, library definition, dispinterface definition, interface definition, coclass definition and module definition to optionally allow a semicolon postfix. Call pop_import from the parser instead of the lexer to ensure that pop_import is only called after the last statement in the imported file has been parsed. --- tools/widl/parser.h | 1 + tools/widl/parser.l | 8 ++------ tools/widl/parser.y | 37 +++++++++++++++++++++++++------------ 3 files changed, 28 insertions(+), 18 deletions(-) diff --git a/tools/widl/parser.h b/tools/widl/parser.h index c46364a..71f08dc 100644 --- a/tools/widl/parser.h +++ b/tools/widl/parser.h @@ -39,6 +39,7 @@ int parser_lex(void); extern int import_stack_ptr; int do_import(char *fname); void abort_import(void); +void pop_import(void); #define parse_only import_stack_ptr diff --git a/tools/widl/parser.l b/tools/widl/parser.l index 94eec0d..3bcdc26 100644 --- a/tools/widl/parser.l +++ b/tools/widl/parser.l @@ -79,8 +79,6 @@ struct { } import_stack[MAX_IMPORT_DEPTH]; int import_stack_ptr = 0; -static void pop_import(void); - UUID *parse_uuid(const char *u) { UUID* uuid = xmalloc(sizeof(UUID)); @@ -166,10 +164,8 @@ SAFEARRAY{ws}*/\( return tSAFEARRAY; <INITIAL,ATTR>\>\> return SHR; <INITIAL,ATTR>. return yytext[0]; <<EOF>> { - if (import_stack_ptr) { - pop_import(); + if (import_stack_ptr) return aEOF; - } else yyterminate(); } %% @@ -367,7 +363,7 @@ static char *get_buffered_cstring(void) return xstrdup(cbuffer); } -static void pop_import(void) +void pop_import(void) { int ptr = import_stack_ptr-1; diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 7ddae74..dd02ab6 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -267,6 +267,7 @@ static void check_all_user_types(ifref_list_t *ifaces); %type <num> pointer_type version %type <str> libraryhdr %type <uuid> uuid_string +%type <num> import_start %left ',' %right '?' ':' @@ -328,8 +329,11 @@ int_statements: { $$ = NULL; } | int_statements statement { $$ = $1; } ; -statement: ';' {} - | constdef ';' { if (!parse_only && do_header) { write_constdef($1); } } +semicolon_opt: + | ';' + ; + +statement: constdef ';' { if (!parse_only && do_header) { write_constdef($1); } } | cppquote {} | enumdef ';' { if (!parse_only && do_header) { write_type_def_or_decl(header, $1, FALSE, NULL); @@ -354,12 +358,17 @@ statement: ';' {} cppquote: tCPPQUOTE '(' aSTRING ')' { if (!parse_only && do_header) fprintf(header, "%s\n", $3); } ; import_start: tIMPORT aSTRING ';' { assert(yychar == YYEMPTY); - if (!do_import($2)) yychar = aEOF; } + $$ = do_import($2); + if (!$$) yychar = aEOF; + } ; -import: import_start imp_statements aEOF {} + +import: import_start imp_statements aEOF + { if ($1) pop_import(); } ; -importlib: tIMPORTLIB '(' aSTRING ')' { if(!parse_only) add_importlib($3); } +importlib: tIMPORTLIB '(' aSTRING ')' + semicolon_opt { if(!parse_only) add_importlib($3); } ; libraryhdr: tLIBRARY aIDENTIFIER { $$ = $2; } @@ -369,7 +378,8 @@ library_start: attributes libraryhdr '{' { if (!parse_only) start_typelib($2, $1 if (!parse_only && do_idfile) write_libid($2, $1); } ; -librarydef: library_start imp_statements '}' { if (!parse_only) end_typelib(); } +librarydef: library_start imp_statements '}' + semicolon_opt { if (!parse_only) end_typelib(); } ; m_args: { $$ = NULL; } @@ -726,7 +736,8 @@ coclasshdr: attributes coclass { $$ = $2; } ; -coclassdef: coclasshdr '{' coclass_ints '}' { $$ = $1; +coclassdef: coclasshdr '{' coclass_ints '}' semicolon_opt + { $$ = $1; $$->ifaces = $3; $$->defined = TRUE; } @@ -774,7 +785,7 @@ dispinterfacedef: dispinterfacehdr '{' if (!parse_only && do_idfile) write_diid($$); } | dispinterfacehdr - '{' interface ';' '}' { $$ = $1; + '{' interface ';' '}' { $$ = $1; $$->fields = $3->fields; $$->funcs = $3->funcs; if (!parse_only && do_header) write_dispinterface($$); @@ -802,7 +813,7 @@ interfacehdr: attributes interface { $$.interface = $2; ; interfacedef: interfacehdr inherit - '{' int_statements '}' { $$ = $1.interface; + '{' int_statements '}' semicolon_opt { $$ = $1.interface; $$->ref = $2; $$->funcs = $4; compute_method_indexes($$); @@ -814,7 +825,8 @@ interfacedef: interfacehdr inherit /* MIDL is able to import the definition of a base class from inside the * definition of a derived class, I'll try to support it with this rule */ | interfacehdr ':' aIDENTIFIER - '{' import int_statements '}' { $$ = $1.interface; + '{' import int_statements '}' + semicolon_opt { $$ = $1.interface; $$->ref = find_type2($3, 0); if (!$$->ref) error_loc("base class '%s' not found in import\n", $3); $$->funcs = $6; @@ -824,7 +836,7 @@ interfacedef: interfacehdr inherit if (!parse_only && do_idfile) write_iid($$); pointer_default = $1.old_pointer_default; } - | dispinterfacedef { $$ = $1; } + | dispinterfacedef semicolon_opt { $$ = $1; } ; interfacedec: @@ -841,7 +853,8 @@ modulehdr: attributes module { $$ = $2; } ; -moduledef: modulehdr '{' int_statements '}' { $$ = $1; +moduledef: modulehdr '{' int_statements '}' + semicolon_opt { $$ = $1; $$->funcs = $3; /* FIXME: if (!parse_only && do_header) write_module($$); */ }

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 April 2008