Commit Graph

49 Commits

Author SHA1 Message Date
Rob Shearman 24ba6a5cc6 rpcrt4: Fix memory leaks in the ndr_marshall tests. 2008-01-09 13:38:02 +01:00
Rob Shearman e0c67a189b rpcrt4: Implement NdrMapCommAndFaultStatus.
Add tests for this function.
2008-01-09 12:25:50 +01:00
Rob Shearman 31676530c2 rpcrt4: Set the ProcNum of RPC_MESSAGE in NdrClientInitializeNew to include RPC_VALID_FLAGS_BIT.
Mask out RPC_VALID_FLAGS_BIT from ProcNum when it is used to build a 
packet to send to the server.
2008-01-07 22:55:28 +01:00
Rob Shearman 4d6aa7ca95 rpcrt4: Add tests for the RPC_MESSAGE members set by NdrClientInitializeNew. 2008-01-07 22:55:22 +01:00
Rob Shearman 0e6d5202c8 rpcrt4: Don't set Buffer to NULL in I_RpcFreeBuffer. 2008-01-07 12:04:31 +01:00
Rob Shearman 1bb6c1b4d1 rpcrt4: Add some tests for NdrGetBuffer and NdrFreeBuffer. 2008-01-07 12:04:28 +01:00
Rob Shearman 07622b0f05 rpcrt4: Add tests for NdrServerInitializeNew. 2007-12-31 17:14:22 +01:00
Rob Shearman 274dc73ba1 rpcrt4: Add some tests for non-conformant strings. 2007-12-31 16:46:16 +01:00
Rob Shearman 259879d1f5 rpcrt4: Fix NdrConformantStringUnmarshall to use buffer memory if possible. 2007-12-17 11:58:06 +01:00
Rob Shearman c7261b732b rpcrt4: Partially revert commit 18faf3184b2ea263d77c2a7ad92eef27bc4ba08f.
Tests show that NdrPointerFree (and hence NdrFree) don't check whether 
the memory was allocated with NdrAllocate before freeing it. This makes 
sense as servers don't need to use NdrAllocate to allocate memory being 
returned and so this commit caused that memory to be leaked.

NdrAllocate hasn't been changed as the tests show it is correct and it 
appears that the memory list is used to implement the RpcSs memory model.
2007-12-14 12:24:21 +01:00
Rob Shearman ded4b86625 Revert "rpcrt4: Try to free every non-stack pointer in PointerFree now that NdrFree does the sanity checking for us.".
This reverts commit 6db648302ce43f625a073426a35a6f40b7869d8b.

Tests show the assumptions in a commit that this commit depends on are 
not valid.
2007-12-14 12:23:38 +01:00
Rob Shearman 9642714d06 rpcrt4: Add tests for freeing non-NdrAllocate allocated memory blocks. 2007-12-13 14:30:40 +01:00
Rob Shearman e12b487e26 rpcrt4: Try to free every non-stack pointer in PointerFree now that NdrFree does the sanity checking for us. 2007-12-12 16:02:53 +01:00
Rob Shearman 9c8c74f388 rpcrt4: Store allocated memory in a singly-linked list to keep track of what we should and shouldn't free in NdrFree. 2007-12-12 16:02:43 +01:00
Rob Shearman 33c4c25dad rpcrt4: Make the NdrAllocate tests pass on XP SP2 and greater. 2007-12-12 16:01:33 +01:00
Rob Shearman abbceb137a rpcrt4: Fix NdrConformantArrayUnmarshall to use buffer memory if applicable and to reuse memory for embedded pointers. 2007-12-08 19:57:32 +01:00
Alexandre Julliard c0b12351e0 Avoid size_t types in traces. 2007-12-07 18:49:18 +01:00
Rob Shearman 2ebee18198 rpcrt4: Improve PointerFree to not free buffer memory. 2007-12-07 16:34:17 +01:00
Rob Shearman 13e47d762a rpcrt4: Set the memory pointer in NdrBaseTypeUnmarshall to the buffer if we're unmarshalling on a server without memory being passed in. 2007-12-05 13:49:08 +01:00
Rob Shearman c866b06806 rpcrt4: Add tests for a pointer to a conformant strings. 2007-12-04 18:18:03 +01:00
Rob Shearman eef207e3f1 rpcrt4: Initialise the memory in test_simple_struct_marshal before passing it to NdrSimpleStructUnmarshall.
Otherwise, the unmarshalling code will read from unintialised memory
when trying to reuse the embedded pointers in the memory block.

Also fix a couple of memory leaks.
2007-12-03 13:10:21 +01:00
Rob Shearman 4171309731 rpcrt4: Fix the tests for up_enum16.
It is different to the other base types as it has a different size on 
the wire to in memory, so it can't just be set to the buffer when 
unmarshalling.
2007-11-27 16:42:17 +01:00
Rob Shearman 686c0f094c rpcrt4: Even though FC_ENUM16 is an unsigned type, the highest allowable value that can be marshalled is SHRT_MAX, not USHRT_MAX. 2007-07-16 22:46:47 +02:00
Francois Gouget 8d7ad785e3 rpcrt4/tests: Fix a Visual C++ double to float conversion warning. 2007-05-29 13:46:19 +02:00
Francois Gouget f27a7153a5 rpcrt4/tests: Fix signed/unsigned and 32/64bit issues with the wiredatalen parameter. 2007-05-29 13:46:14 +02:00
Dmitry Timoshkov 3c9e7a7f33 wine: Switch to using 'long' for INT_PTR type for 64-bit compatibility. 2007-05-25 20:37:56 +02:00
Huw Davies 6fcff278b3 rpcrt4: Tests that show that the allocation rules for unmarshaling a conformant array are the same as those for a simple structure. 2007-03-13 11:52:37 +01:00
Huw Davies e3abd2b3ff rpcrt4: Implement NdrSimpleType{Marshall,Unmarshall}. 2007-01-24 11:59:01 +01:00
Francois Gouget fa7c32f56c rpcrt4: Give a proper name to some formerly reserved MIDL_STUB_MESSAGE fields.
Define NTDDI_VERSION so we get definitions for the Reserved51_X fields with the PSDK.
This fixes compilation of the ndr_marshall.c with the PSDK.
2007-01-11 11:37:13 +01:00
Francois Gouget 9f586f1b79 rpcrt4/tests: Replace some '#if 0's with 'if (0)'s, or remove them if not needed anymore.
Fix the code so it compiles without warnings.
2007-01-08 12:07:59 +01:00
Andrew Talbot 82a042d0df rpcrt4/tests: Cast-qual warnings fix. 2006-12-14 12:51:10 +01:00
Marcus Meissner f0280cbd1e made functions and variables static in some testcases. 2006-12-11 12:18:29 +01:00
Alexandre Julliard c0f9d369dc rpcrt4/tests: Replace || by |. 2006-11-09 18:20:49 +01:00
Michael Stefaniuc 6bf97313bc rpcrt4/tests: Fix the warnings introduced by the "unsigned long" to "ULONG" header changes. 2006-11-09 10:43:45 +01:00
Michael Stefaniuc 14c551bc3a rpcrt4/tests: Win64 printf format warning fixes. 2006-10-09 12:44:16 +02:00
Francois Gouget 1011bc47bf rpcrt4: Don't return void values. This fixes Visual C++ warnings. 2006-10-02 11:54:43 +02:00
Stefan Huehner 998e256c82 Fix some -Wstrict-prototype warnings () -> (void). 2006-08-21 12:06:26 +02:00
Huw Davies d9b6672530 rpcrt4: Add tests for NdrAllocate. 2006-08-17 11:19:17 +02:00
Robert Shearman 11da87d098 rpcrt4: NdrClientInitializeNew shouldn't clear all of the stub message, only selected fields. 2006-06-02 11:22:38 +02:00
Robert Shearman bf985900e2 rpcrt4: For full pointers the state consists of flags, so or new flags into the field instead of assigning.
The return value of all full pointer functions that return an int 
depends on the passed in query type being a flag in the state.
2006-05-30 12:24:22 +02:00
Robert Shearman 63307241dd rpcrt4: Store the QueryType in the state table in NdrFullPointerQueryRefId.
Implement NdrFullPointerFree.
2006-05-27 10:54:39 +02:00
Robert Shearman 7af506b2b4 rpcrt4: Fix NdrFullPointerQueryRefId to make more tests pass. 2006-05-27 10:54:21 +02:00
Robert Shearman 767631d853 rpcrt4: Implement remaining part of NdrFullPointerQueryPointer and implement NdrFullPointerInsertRefId. 2006-05-27 10:53:48 +02:00
Robert Shearman eb0fe9e6e3 rpcrt4: Implement some full pointer functions and add stubs for others.
Add some tests.
2006-05-27 10:53:29 +02:00
Alexandre Julliard 23f9e358ad Tests should not use wine/unicode.h. 2006-05-24 18:05:09 +02:00
Huw Davies 00598e49cf rpcrt4: Uncomment test now that it works correctly. 2006-05-23 17:51:50 +02:00
Jonathan Ernst 360a3f9142 Update the address of the Free Software Foundation. 2006-05-23 14:11:13 +02:00
Huw Davies f151ecb0d1 rpcrt4: Use IsClient to determine whether we can use the Buffer memory or not.
Add simple struct tests.
2006-05-20 14:03:56 +02:00
Huw Davies effac656bb rpcrt4: Don't update StubMsg->MemorySize while unmarshaling.
Add a bunch of marshaling tests (based on a program by Rob Shearman).
2006-05-17 21:46:43 +02:00