From ad24c5599e982f0515ee6de3b271238b4a080831 Mon Sep 17 00:00:00 2001 From: "Gregory M. Turner" Date: Tue, 21 Oct 2003 23:49:57 +0000 Subject: [PATCH] - Remove an erroneous check in NdrSendReceive. - Change fixmes. --- dlls/rpcrt4/ndr_midl.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/dlls/rpcrt4/ndr_midl.c b/dlls/rpcrt4/ndr_midl.c index 1797f9bf227..a22aa4829e4 100644 --- a/dlls/rpcrt4/ndr_midl.c +++ b/dlls/rpcrt4/ndr_midl.c @@ -266,16 +266,13 @@ unsigned char *WINAPI NdrSendReceive( MIDL_STUB_MESSAGE *stubmsg, unsigned char ERR("RPC Message not present in stub message. No action taken.\n"); return NULL; } - if (stubmsg->RpcMsg->Buffer != buffer) { - ERR("Ambiguous buffer doesn't match rpc message buffer. No action taken.\n"); - return NULL; - } - /* not sure where MS does this; for now I'll stick it here */ + /* FIXME: Seems wrong. Where should this really come from, and when? */ stubmsg->RpcMsg->DataRepresentation = NDR_LOCAL_DATA_REPRESENTATION; if (I_RpcSendReceive(stubmsg->RpcMsg) != RPC_S_OK) { WARN("I_RpcSendReceive did not return success.\n"); + /* FIXME: raise exception? */ } /* FIXME: is this the right return value? */