wine-commits December 2007

wine-commits@winehq.org

3 participants
824 discussions

Rob Shearman : rpcrt4: Add tests for NdrServerInitializeNew.
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: 07622b0f05f18fdcbfdd635cc33c9e4fe4ca18ac URL: http://source.winehq.org/git/wine.git/?a=commit;h=07622b0f05f18fdcbfdd635cc… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Dec 30 16:46:15 2007 +0000 rpcrt4: Add tests for NdrServerInitializeNew. --- dlls/rpcrt4/tests/ndr_marshall.c | 104 ++++++++++++++++++++++++++++++++++++-- 1 files changed, 100 insertions(+), 4 deletions(-) diff --git a/dlls/rpcrt4/tests/ndr_marshall.c b/dlls/rpcrt4/tests/ndr_marshall.c index 73e33a4..eadb9c7 100644 --- a/dlls/rpcrt4/tests/ndr_marshall.c +++ b/dlls/rpcrt4/tests/ndr_marshall.c @@ -866,14 +866,15 @@ static void test_client_init(void) MIDL_STUB_MESSAGE stubMsg; RPC_MESSAGE rpcMsg; + memset(&rpcMsg, 0, sizeof(rpcMsg)); memset(&stubMsg, 0xcc, sizeof(stubMsg)); NdrClientInitializeNew(&rpcMsg, &stubMsg, &Object_StubDesc, 1); -#define TEST_ZERO(field, fmt) ok(stubMsg.field == 0, #field " should have be set to zero instead of " fmt "\n", stubMsg.field) -#define TEST_POINTER_UNSET(field) ok(stubMsg.field == (void *)0xcccccccc, #field " should have be unset instead of %p\n", stubMsg.field) -#define TEST_ULONG_UNSET(field) ok(stubMsg.field == 0xcccccccc, #field " should have be unset instead of 0x%x\n", stubMsg.field) -#define TEST_ULONG_PTR_UNSET(field) ok(stubMsg.field == 0xcccccccc, #field " should have be unset instead of 0x%lx\n", stubMsg.field) +#define TEST_ZERO(field, fmt) ok(stubMsg.field == 0, #field " should have been set to zero instead of " fmt "\n", stubMsg.field) +#define TEST_POINTER_UNSET(field) ok(stubMsg.field == (void *)0xcccccccc, #field " should have been unset instead of %p\n", stubMsg.field) +#define TEST_ULONG_UNSET(field) ok(stubMsg.field == 0xcccccccc, #field " should have been unset instead of 0x%x\n", stubMsg.field) +#define TEST_ULONG_PTR_UNSET(field) ok(stubMsg.field == 0xcccccccc, #field " should have been unset instead of 0x%lx\n", stubMsg.field) ok(stubMsg.RpcMsg == &rpcMsg, "stubMsg.RpcMsg should have been %p instead of %p\n", &rpcMsg, stubMsg.RpcMsg); TEST_POINTER_UNSET(Buffer); @@ -946,6 +947,100 @@ static void test_client_init(void) } +static void test_server_init(void) +{ + MIDL_STUB_MESSAGE stubMsg; + RPC_MESSAGE rpcMsg; + unsigned char *ret; + unsigned char buffer[256]; + + memset(&rpcMsg, 0, sizeof(rpcMsg)); + rpcMsg.Buffer = buffer; + rpcMsg.BufferLength = sizeof(buffer); + rpcMsg.RpcFlags = RPC_BUFFER_COMPLETE; + + memset(&stubMsg, 0xcc, sizeof(stubMsg)); + + ret = NdrServerInitializeNew(&rpcMsg, &stubMsg, &Object_StubDesc); + ok(ret == NULL, "NdrServerInitializeNew should have returned NULL instead of %p\n", ret); + +#define TEST_ZERO(field, fmt) ok(stubMsg.field == 0, #field " should have been set to zero instead of " fmt "\n", stubMsg.field) +#define TEST_POINTER_UNSET(field) ok(stubMsg.field == (void *)0xcccccccc, #field " should have been unset instead of %p\n", stubMsg.field) +#define TEST_ULONG_UNSET(field) ok(stubMsg.field == 0xcccccccc, #field " should have been unset instead of 0x%x\n", stubMsg.field) +#define TEST_ULONG_PTR_UNSET(field) ok(stubMsg.field == 0xcccccccc, #field " should have been unset instead of 0x%lx\n", stubMsg.field) + + ok(stubMsg.RpcMsg == &rpcMsg, "stubMsg.RpcMsg should have been %p instead of %p\n", &rpcMsg, stubMsg.RpcMsg); + ok(stubMsg.Buffer == buffer, "stubMsg.Buffer should have been %p instead of %p\n", buffer, stubMsg.Buffer); + ok(stubMsg.BufferStart == buffer, "stubMsg.BufferStart should have been %p instead of %p\n", buffer, stubMsg.BufferStart); + ok(stubMsg.BufferEnd == buffer + sizeof(buffer), "stubMsg.BufferEnd should have been %p instead of %p\n", buffer + sizeof(buffer), stubMsg.BufferEnd); + TEST_POINTER_UNSET(BufferMark); +todo_wine + TEST_ZERO(BufferLength, "%d"); + TEST_ULONG_UNSET(MemorySize); + TEST_POINTER_UNSET(Memory); + ok(stubMsg.IsClient == 0, "stubMsg.IsClient should have been 0 instead of %u\n", stubMsg.IsClient); + TEST_ZERO(ReuseBuffer, "%d"); + TEST_ZERO(pAllocAllNodesContext, "%p"); + TEST_ZERO(pPointerQueueState, "%p"); + TEST_ZERO(IgnoreEmbeddedPointers, "%d"); + TEST_ZERO(PointerBufferMark, "%p"); + ok(stubMsg.fBufferValid == 0xcc, "fBufferValid should have been unset instead of 0x%x\n", stubMsg.fBufferValid); + TEST_ZERO(uFlags, "%d"); + /* FIXME: UniquePtrCount */ + TEST_ULONG_PTR_UNSET(MaxCount); + TEST_ULONG_UNSET(Offset); + TEST_ULONG_UNSET(ActualCount); + ok(stubMsg.pfnAllocate == my_alloc, "stubMsg.pfnAllocate should have been %p instead of %p\n", my_alloc, stubMsg.pfnAllocate); + ok(stubMsg.pfnFree == my_free, "stubMsg.pfnFree should have been %p instead of %p\n", my_free, stubMsg.pfnFree); + TEST_ZERO(StackTop, "%p"); + TEST_POINTER_UNSET(pPresentedType); + TEST_POINTER_UNSET(pTransmitType); + TEST_POINTER_UNSET(SavedHandle); + ok(stubMsg.StubDesc == &Object_StubDesc, "stubMsg.StubDesc should have been %p instead of %p\n", &Object_StubDesc, stubMsg.StubDesc); + TEST_ZERO(FullPtrXlatTables, "%p"); + TEST_ZERO(FullPtrRefId, "%d"); + TEST_ZERO(PointerLength, "%d"); + TEST_ZERO(fInDontFree, "%d"); + TEST_ZERO(fDontCallFreeInst, "%d"); + TEST_ZERO(fInOnlyParam, "%d"); + TEST_ZERO(fHasReturn, "%d"); + TEST_ZERO(fHasExtensions, "%d"); + TEST_ZERO(fHasNewCorrDesc, "%d"); + TEST_ZERO(fUnused, "%d"); + ok(stubMsg.fUnused2 == 0xffffcccc, "stubMsg.fUnused2 should have been 0xcccc instead of 0x%x\n", stubMsg.fUnused2); + ok(stubMsg.dwDestContext == MSHCTX_DIFFERENTMACHINE, "stubMsg.dwDestContext should have been MSHCTX_DIFFERENTMACHINE instead of %d\n", stubMsg.dwDestContext); + TEST_ZERO(pvDestContext, "%p"); + TEST_POINTER_UNSET(SavedContextHandles); + TEST_ULONG_UNSET(ParamNumber); + TEST_ZERO(pRpcChannelBuffer, "%p"); + TEST_ZERO(pArrayInfo, "%p"); + TEST_POINTER_UNSET(SizePtrCountArray); + TEST_POINTER_UNSET(SizePtrOffsetArray); + TEST_POINTER_UNSET(SizePtrLengthArray); + TEST_POINTER_UNSET(pArgQueue); + TEST_ZERO(dwStubPhase, "%d"); + /* FIXME: where does this value come from? */ + trace("LowStackMark is %p\n", stubMsg.LowStackMark); + TEST_ZERO(pAsyncMsg, "%p"); + TEST_ZERO(pCorrInfo, "%p"); + TEST_ZERO(pCorrMemory, "%p"); + TEST_ZERO(pMemoryList, "%p"); + TEST_POINTER_UNSET(pCSInfo); + TEST_POINTER_UNSET(ConformanceMark); + TEST_POINTER_UNSET(VarianceMark); + ok(stubMsg.Unused == 0xcccccccc, "Unused should have be unset instead of 0x%lx\n", stubMsg.Unused); + TEST_POINTER_UNSET(pContext); + TEST_POINTER_UNSET(ContextHandleHash); + TEST_POINTER_UNSET(pUserMarshalList); + TEST_ULONG_PTR_UNSET(Reserved51_3); + TEST_ULONG_PTR_UNSET(Reserved51_4); + TEST_ULONG_PTR_UNSET(Reserved51_5); +#undef TEST_ULONG_UNSET +#undef TEST_POINTER_UNSET +#undef TEST_ZERO + +} + static void test_ndr_allocate(void) { RPC_MESSAGE RpcMessage; @@ -1421,6 +1516,7 @@ START_TEST( ndr_marshall ) test_simple_struct(); test_fullpointer_xlat(); test_client_init(); + test_server_init(); test_ndr_allocate(); test_conformant_array(); test_conformant_string();

1 0

Rob Shearman : rpcrt4: Fix the MIDL_STUB_MESSAGE fields set by NdrServerInitializeNew.
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: e745f93c79404b166f45eefcb883ff4d761f90fe URL: http://source.winehq.org/git/wine.git/?a=commit;h=e745f93c79404b166f45eefcb… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Dec 30 16:46:05 2007 +0000 rpcrt4: Fix the MIDL_STUB_MESSAGE fields set by NdrServerInitializeNew. --- dlls/rpcrt4/ndr_clientserver.c | 44 +++++++++++++++++++++++++++++---------- 1 files changed, 32 insertions(+), 12 deletions(-) diff --git a/dlls/rpcrt4/ndr_clientserver.c b/dlls/rpcrt4/ndr_clientserver.c index fe424a6..a43b920 100644 --- a/dlls/rpcrt4/ndr_clientserver.c +++ b/dlls/rpcrt4/ndr_clientserver.c @@ -106,22 +106,42 @@ unsigned char* WINAPI NdrServerInitializeNew( PRPC_MESSAGE pRpcMsg, PMIDL_STUB_M { TRACE("(pRpcMsg == ^%p, pStubMsg == ^%p, pStubDesc == ^%p)\n", pRpcMsg, pStubMsg, pStubDesc); - assert( pRpcMsg && pStubMsg && pStubDesc ); - - /* not everyone allocates stack space for w2kReserved */ - memset(pStubMsg, 0, FIELD_OFFSET(MIDL_STUB_MESSAGE,pCSInfo)); - - pStubMsg->ReuseBuffer = TRUE; - pStubMsg->IsClient = FALSE; - pStubMsg->StubDesc = pStubDesc; - pStubMsg->pfnAllocate = pStubDesc->pfnAllocate; - pStubMsg->pfnFree = pStubDesc->pfnFree; pStubMsg->RpcMsg = pRpcMsg; pStubMsg->Buffer = pStubMsg->BufferStart = pRpcMsg->Buffer; + pStubMsg->BufferEnd = pStubMsg->Buffer + pRpcMsg->BufferLength; pStubMsg->BufferLength = pRpcMsg->BufferLength; - pStubMsg->BufferEnd = pStubMsg->Buffer + pStubMsg->BufferLength; + pStubMsg->IsClient = FALSE; + pStubMsg->ReuseBuffer = FALSE; + pStubMsg->pAllocAllNodesContext = NULL; + pStubMsg->pPointerQueueState = NULL; + pStubMsg->IgnoreEmbeddedPointers = 0; + pStubMsg->PointerBufferMark = NULL; + pStubMsg->uFlags = 0; + pStubMsg->pfnAllocate = pStubDesc->pfnAllocate; + pStubMsg->pfnFree = pStubDesc->pfnFree; + pStubMsg->StackTop = NULL; + pStubMsg->StubDesc = pStubDesc; + pStubMsg->FullPtrXlatTables = NULL; + pStubMsg->FullPtrRefId = 0; + pStubMsg->PointerLength = 0; + pStubMsg->fInDontFree = 0; + pStubMsg->fDontCallFreeInst = 0; + pStubMsg->fInOnlyParam = 0; + pStubMsg->fHasReturn = 0; + pStubMsg->fHasExtensions = 0; + pStubMsg->fHasNewCorrDesc = 0; + pStubMsg->fUnused = 0; + pStubMsg->dwDestContext = MSHCTX_DIFFERENTMACHINE; + pStubMsg->pvDestContext = NULL; + pStubMsg->pRpcChannelBuffer = NULL; + pStubMsg->pArrayInfo = NULL; + pStubMsg->dwStubPhase = 0; + /* FIXME: LowStackMark */ + pStubMsg->pAsyncMsg = NULL; + pStubMsg->pCorrInfo = NULL; + pStubMsg->pCorrMemory = NULL; + pStubMsg->pMemoryList = NULL; - /* FIXME: determine the proper return value */ return NULL; }

1 0

Rob Shearman : rpcrt4: Fix a case of destroying a context handle without first releasing the lock .
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: f6a29a1e73a4ad4bba7eac7be3d37266548684a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f6a29a1e73a4ad4bba7eac7be… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Dec 30 16:45:19 2007 +0000 rpcrt4: Fix a case of destroying a context handle without first releasing the lock. Reverse the order of the release statements in NDRSContextMarshall2 for the case where the context handle doesn't have valid data so that the release that releases the lock comes first and then the optional second release doesn't need to release the lock. --- dlls/rpcrt4/ndr_contexthandle.c | 10 +++++++--- 1 files changed, 7 insertions(+), 3 deletions(-) diff --git a/dlls/rpcrt4/ndr_contexthandle.c b/dlls/rpcrt4/ndr_contexthandle.c index 7b3c3fd..21d8747 100644 --- a/dlls/rpcrt4/ndr_contexthandle.c +++ b/dlls/rpcrt4/ndr_contexthandle.c @@ -259,21 +259,25 @@ void WINAPI NDRSContextMarshall2(RPC_BINDING_HANDLE hBinding, RpcRaiseException(status); ndr->attributes = 0; RpcContextHandle_GetUuid(SContext, &ndr->uuid); + + RPCRT4_RemoveThreadContextHandle(SContext); + RpcServerAssoc_ReleaseContextHandle(binding->Assoc, SContext, TRUE); } else { if (!RpcContextHandle_IsGuardCorrect(SContext, CtxGuard)) RpcRaiseException(ERROR_INVALID_HANDLE); memset(ndr, 0, sizeof(*ndr)); + + RPCRT4_RemoveThreadContextHandle(SContext); /* Note: release the context handle twice in this case to release * one ref being kept around for the data and one ref for the * unmarshall/marshall sequence */ - if (!RpcServerAssoc_ReleaseContextHandle(binding->Assoc, SContext, FALSE)) + if (!RpcServerAssoc_ReleaseContextHandle(binding->Assoc, SContext, TRUE)) return; /* this is to cope with the case of the data not being valid * before and so not having a further reference */ + RpcServerAssoc_ReleaseContextHandle(binding->Assoc, SContext, FALSE); } - RPCRT4_RemoveThreadContextHandle(SContext); - RpcServerAssoc_ReleaseContextHandle(binding->Assoc, SContext, TRUE); } /***********************************************************************

1 0

Rob Shearman : rpcrt4: Keep a track of server context handles allocated during processing of a request .
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: 51c051c0c4fc26e45ab6c04684acf73886eeb7a1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=51c051c0c4fc26e45ab6c0468… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Dec 30 16:44:54 2007 +0000 rpcrt4: Keep a track of server context handles allocated during processing of a request. Release them after processing of a request has finished to avoid a slow memory leak if the association isn't released for ages. --- dlls/rpcrt4/ndr_contexthandle.c | 2 + dlls/rpcrt4/rpc_binding.h | 4 ++ dlls/rpcrt4/rpc_server.c | 5 +++ dlls/rpcrt4/rpcrt4_main.c | 60 +++++++++++++++++++++++++++++++++++++++ 4 files changed, 71 insertions(+), 0 deletions(-) diff --git a/dlls/rpcrt4/ndr_contexthandle.c b/dlls/rpcrt4/ndr_contexthandle.c index 0b3eeb1..7b3c3fd 100644 --- a/dlls/rpcrt4/ndr_contexthandle.c +++ b/dlls/rpcrt4/ndr_contexthandle.c @@ -272,6 +272,7 @@ void WINAPI NDRSContextMarshall2(RPC_BINDING_HANDLE hBinding, return; /* this is to cope with the case of the data not being valid * before and so not having a further reference */ } + RPCRT4_RemoveThreadContextHandle(SContext); RpcServerAssoc_ReleaseContextHandle(binding->Assoc, SContext, TRUE); } @@ -335,5 +336,6 @@ NDR_SCONTEXT WINAPI NDRSContextUnmarshall2(RPC_BINDING_HANDLE hBinding, if (status != RPC_S_OK) RpcRaiseException(status); + RPCRT4_PushThreadContextHandle(SContext); return SContext; } diff --git a/dlls/rpcrt4/rpc_binding.h b/dlls/rpcrt4/rpc_binding.h index b774074..ed8d4ea 100644 --- a/dlls/rpcrt4/rpc_binding.h +++ b/dlls/rpcrt4/rpc_binding.h @@ -22,6 +22,7 @@ #define __WINE_RPC_BINDING_H #include "wine/rpcss_shared.h" +#include "rpcndr.h" #include "security.h" #include "wine/list.h" @@ -189,5 +190,8 @@ RPC_STATUS RpcTransport_ParseTopOfTower(const unsigned char *tower_data, size_t void RPCRT4_SetThreadCurrentConnection(RpcConnection *Connection); void RPCRT4_SetThreadCurrentCallHandle(RpcBinding *Binding); RpcBinding *RPCRT4_GetThreadCurrentCallHandle(void); +void RPCRT4_PushThreadContextHandle(NDR_SCONTEXT SContext); +void RPCRT4_RemoveThreadContextHandle(NDR_SCONTEXT SContext); +NDR_SCONTEXT RPCRT4_PopThreadContextHandle(void); #endif diff --git a/dlls/rpcrt4/rpc_server.c b/dlls/rpcrt4/rpc_server.c index a1262da..34c71c7 100644 --- a/dlls/rpcrt4/rpc_server.c +++ b/dlls/rpcrt4/rpc_server.c @@ -172,6 +172,7 @@ static void RPCRT4_process_packet(RpcConnection* conn, RpcPktHdr* hdr, RPC_MESSA void *buf = msg->Buffer; RPC_STATUS status; BOOL exception; + NDR_SCONTEXT context_handle; msg->Handle = (RPC_BINDING_HANDLE)conn->server_binding; @@ -307,6 +308,10 @@ static void RPCRT4_process_packet(RpcConnection* conn, RpcPktHdr* hdr, RPC_MESSA } __ENDTRY RPCRT4_SetThreadCurrentCallHandle(NULL); + /* release any unmarshalled context handles */ + while ((context_handle = RPCRT4_PopThreadContextHandle()) != NULL) + RpcServerAssoc_ReleaseContextHandle(conn->server_binding->Assoc, context_handle, TRUE); + if (!exception) response = RPCRT4_BuildResponseHeader(msg->DataRepresentation, msg->BufferLength); diff --git a/dlls/rpcrt4/rpcrt4_main.c b/dlls/rpcrt4/rpcrt4_main.c index 887e049..e48cf58 100644 --- a/dlls/rpcrt4/rpcrt4_main.c +++ b/dlls/rpcrt4/rpcrt4_main.c @@ -146,6 +146,12 @@ static CRITICAL_SECTION threaddata_cs = { &threaddata_cs_debug, -1, 0, 0, 0, 0 } struct list threaddata_list = LIST_INIT(threaddata_list); +struct context_handle_list +{ + struct context_handle_list *next; + NDR_SCONTEXT context_handle; +}; + struct threaddata { struct list entry; @@ -153,6 +159,7 @@ struct threaddata DWORD thread_id; RpcConnection *connection; RpcBinding *server_binding; + struct context_handle_list *context_handle_list; }; /*********************************************************************** @@ -936,6 +943,59 @@ RpcBinding *RPCRT4_GetThreadCurrentCallHandle(void) return tdata->server_binding; } +void RPCRT4_PushThreadContextHandle(NDR_SCONTEXT SContext) +{ + struct threaddata *tdata = get_or_create_threaddata(); + struct context_handle_list *context_handle_list; + + if (!tdata) return; + + context_handle_list = HeapAlloc(GetProcessHeap(), 0, sizeof(*context_handle_list)); + if (!context_handle_list) return; + + context_handle_list->context_handle = SContext; + context_handle_list->next = tdata->context_handle_list; + tdata->context_handle_list = context_handle_list; +} + +void RPCRT4_RemoveThreadContextHandle(NDR_SCONTEXT SContext) +{ + struct threaddata *tdata = get_or_create_threaddata(); + struct context_handle_list *current, *prev; + + if (!tdata) return; + + for (current = tdata->context_handle_list, prev = NULL; current; prev = current, current = current->next) + { + if (current->context_handle == SContext) + { + if (prev) + prev->next = current->next; + else + tdata->context_handle_list = current->next; + HeapFree(GetProcessHeap(), 0, current); + return; + } + } +} + +NDR_SCONTEXT RPCRT4_PopThreadContextHandle(void) +{ + struct threaddata *tdata = get_or_create_threaddata(); + struct context_handle_list *context_handle_list; + NDR_SCONTEXT context_handle; + + if (!tdata) return NULL; + + context_handle_list = tdata->context_handle_list; + if (!context_handle_list) return NULL; + tdata->context_handle_list = context_handle_list->next; + + context_handle = context_handle_list->context_handle; + HeapFree(GetProcessHeap(), 0, context_handle_list); + return context_handle; +} + /****************************************************************************** * RpcCancelThread (rpcrt4.@) */

1 0

Rob Shearman : include: Add defines for more message flags in rpcdcep.h.
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: b574533e508cf5c3185e8d3b9f2d8ba0ba7536ef URL: http://source.winehq.org/git/wine.git/?a=commit;h=b574533e508cf5c3185e8d3b9… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Dec 30 16:45:42 2007 +0000 include: Add defines for more message flags in rpcdcep.h. --- include/rpcdcep.h | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/include/rpcdcep.h b/include/rpcdcep.h index f2a2196..5514790 100644 --- a/include/rpcdcep.h +++ b/include/rpcdcep.h @@ -61,6 +61,21 @@ typedef struct _RPC_MESSAGE #define RPC_NCA_FLAGS_BROADCAST 0x00000002 #define RPC_NCA_FLAGS_MAYBE 0x00000004 +#define RPC_BUFFER_COMPLETE 0x00001000 +#define RPC_BUFFER_PARTIAL 0x00002000 +#define RPC_BUFFER_EXTRA 0x00004000 +#define RPC_BUFFER_ASYNC 0x00008000 +#define RPC_BUFFER_NONOTIFY 0x00010000 + +#define RPCFLG_MESSAGE 0x01000000 +#define RPCFLG_HAS_MULTI_SYNTAXES 0x02000000 +#define RPCFLG_HAS_CALLBACK 0x04000000 +#define RPCFLG_AUTO_COMPLETE 0x08000000 +#define RPCFLG_LOCAL_CALL 0x10000000 +#define RPCFLG_INPUT_SYNCHRONOUS 0x20000000 +#define RPCFLG_ASYNCHRONOUS 0x40000000 +#define RPCFLG_NON_NDR 0x80000000 + typedef void (__RPC_STUB *RPC_DISPATCH_FUNCTION)(PRPC_MESSAGE Message); typedef RPC_STATUS (RPC_ENTRY *RPC_FORWARD_FUNCTION)(UUID *InterfaceId, RPC_VERSION *InterfaceVersion, UUID *ObjectId, unsigned char *Rpcpro, void **ppDestEndpoint);

1 0

Rob Shearman : widl: Check that the buffer has enough space before unmarshalling base types in generated code .
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: 9aaff94fb7c49ec48f909b86dc219b81913afce8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9aaff94fb7c49ec48f909b86d… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Dec 30 16:45:05 2007 +0000 widl: Check that the buffer has enough space before unmarshalling base types in generated code. --- tools/widl/typegen.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/tools/widl/typegen.c b/tools/widl/typegen.c index dee5ad8..f40399f 100644 --- a/tools/widl/typegen.c +++ b/tools/widl/typegen.c @@ -2611,6 +2611,12 @@ void print_phase_basetype(FILE *file, int indent, enum remoting_phase phase, } else if (phase == PHASE_UNMARSHAL) { + print_file(file, indent, "if (_StubMsg.Buffer + sizeof("); + write_type_decl(file, is_ptr(type) ? type->ref : type, NULL); + fprintf(file, ") > _StubMsg.BufferEnd)\n"); + print_file(file, indent, "{\n"); + print_file(file, indent + 1, "RpcRaiseException(RPC_X_BAD_STUB_DATA);\n"); + print_file(file, indent, "}\n"); if (pass == PASS_IN || pass == PASS_RETURN) print_file(file, indent, ""); else

1 0

Jacek Caban : ole32: Fixed parameter checking and improved debug traces in IBindCtx:: QueryInterface.
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: 2c6cbea080da4e902d31d32ade832519507a8a0d URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c6cbea080da4e902d31d32ad… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sun Dec 30 17:44:32 2007 +0100 ole32: Fixed parameter checking and improved debug traces in IBindCtx::QueryInterface. --- dlls/ole32/bindctx.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/ole32/bindctx.c b/dlls/ole32/bindctx.c index da68f33..e0f297e 100644 --- a/dlls/ole32/bindctx.c +++ b/dlls/ole32/bindctx.c @@ -74,11 +74,11 @@ BindCtxImpl_QueryInterface(IBindCtx* iface,REFIID riid,void** ppvObject) { BindCtxImpl *This = (BindCtxImpl *)iface; - TRACE("(%p,%p,%p)\n",This,riid,ppvObject); + TRACE("(%p %s %p)\n",This, debugstr_guid(riid), ppvObject); /* Perform a sanity check on the parameters.*/ - if ( (This==0) || (ppvObject==0) ) - return E_INVALIDARG; + if (!ppvObject) + return E_POINTER; /* Initialize the return parameter.*/ *ppvObject = 0;

1 0

Jacek Caban : urlmon: Added more tests.
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: bc35dda746736b3fc2a0e810c49e7954229a896c URL: http://source.winehq.org/git/wine.git/?a=commit;h=bc35dda746736b3fc2a0e810c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sun Dec 30 17:55:02 2007 +0100 urlmon: Added more tests. --- dlls/urlmon/tests/url.c | 744 +++++++++++++++++++++++++++++++++++++++-------- 1 files changed, 621 insertions(+), 123 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=bc35dda746736b3fc2a0e…

1 0

Jacek Caban : urlmon: Added BindToObject implementation.
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: 85f472b37d91941ee6600debb81dbe299f332894 URL: http://source.winehq.org/git/wine.git/?a=commit;h=85f472b37d91941ee6600debb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sun Dec 30 17:45:00 2007 +0100 urlmon: Added BindToObject implementation. --- dlls/urlmon/binding.c | 226 ++++++++++++++++++++++++++++++++++++++++----- dlls/urlmon/tests/url.c | 16 +++- dlls/urlmon/umon.c | 15 ++- dlls/urlmon/urlmon_main.h | 1 + 4 files changed, 227 insertions(+), 31 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=85f472b37d91941ee6600…

1 0

Jacek Caban : urlmon: Call AddRef in EnumFORMATETC::QueryInterface.
by Alexandre Julliard 31 Dec '07

31 Dec '07

Module: wine Branch: master Commit: a16ae2e8f2d341a265a41393d9fe46972da424e6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a16ae2e8f2d341a265a41393d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sun Dec 30 17:44:15 2007 +0100 urlmon: Call AddRef in EnumFORMATETC::QueryInterface. --- dlls/urlmon/format.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/urlmon/format.c b/dlls/urlmon/format.c index 874c881..0c87ca5 100644 --- a/dlls/urlmon/format.c +++ b/dlls/urlmon/format.c @@ -44,6 +44,7 @@ static HRESULT WINAPI EnumFORMATETC_QueryInterface(IEnumFORMATETC *iface, REFIID *ppv = NULL; if(IsEqualGUID(&IID_IUnknown, riid) || IsEqualGUID(&IID_IEnumFORMATETC, riid)) { + IEnumFORMATETC_AddRef(iface); *ppv = iface; return S_OK; }

1 0

← Newer
1
2
3
4
5
6
7
...
83
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits December 2007