Commit ad24c559 authored by Gregory M. Turner's avatar Gregory M. Turner Committed by Alexandre Julliard

- Remove an erroneous check in NdrSendReceive.

- Change fixmes.
parent ea469ad7
...@@ -266,16 +266,13 @@ unsigned char *WINAPI NdrSendReceive( MIDL_STUB_MESSAGE *stubmsg, unsigned char ...@@ -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"); ERR("RPC Message not present in stub message. No action taken.\n");
return NULL; 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; stubmsg->RpcMsg->DataRepresentation = NDR_LOCAL_DATA_REPRESENTATION;
if (I_RpcSendReceive(stubmsg->RpcMsg) != RPC_S_OK) { if (I_RpcSendReceive(stubmsg->RpcMsg) != RPC_S_OK) {
WARN("I_RpcSendReceive did not return success.\n"); WARN("I_RpcSendReceive did not return success.\n");
/* FIXME: raise exception? */
} }
/* FIXME: is this the right return value? */ /* FIXME: is this the right return value? */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment