Robert Shearman : rpcrt4: We don' t need to free object pointers if we are told we must allocate.

[Alexandre Julliard]

Module: wine
Branch: refs/heads/master
Commit: 9c885c918594662d63ee7029bc928a636e85c9fc
URL:    http://source.winehq.org/git/?p=wine.git;a=commit;h=9c885c918594662d63ee7029...

Author: Robert Shearman <rob(a)codeweavers.com>
Date:   Fri May 19 16:04:11 2006 +0100

rpcrt4: We don't need to free object pointers if we are told we must allocate.

---

 dlls/rpcrt4/ndr_marshall.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/rpcrt4/ndr_marshall.c b/dlls/rpcrt4/ndr_marshall.c
index a38ddf7..a787d1f 100644
--- a/dlls/rpcrt4/ndr_marshall.c
+++ b/dlls/rpcrt4/ndr_marshall.c
@@ -822,7 +822,7 @@ static void PointerUnmarshall(PMIDL_STUB
   case RPC_FC_OP: /* object pointer - we must free data before overwriting it */
     pointer_id = NDR_LOCAL_UINT32_READ(Buffer);
     TRACE("pointer_id is 0x%08lx\n", pointer_id);
-    if (*pPointer)
+    if (!fMustAlloc && *pPointer)
         FIXME("free object pointer %p\n", *pPointer);
     break;
   case RPC_FC_FP: