From: Marcus Meissner Subject: [PATCH] rpcrt4: make sure we do not have *data allocated on error (Coverity) Message-Id: <1432155033-4121-1-git-send-email-marcus@jet.franken.de> Date: Wed, 20 May 2015 22:50:33 +0200 713688 Resource leak callers expect *data only allocated when success is returned. --- dlls/rpcrt4/rpc_transport.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/rpcrt4/rpc_transport.c b/dlls/rpcrt4/rpc_transport.c index 34f434d..4080b14 100644 --- a/dlls/rpcrt4/rpc_transport.c +++ b/dlls/rpcrt4/rpc_transport.c @@ -2359,6 +2359,7 @@ static RPC_STATUS rpcrt4_http_read_http_packet(HINTERNET request, RpcHttpAsyncDa if (!RPCRT4_IsValidHttpPacket(hdr, *data, data_len)) { ERR("invalid http packet\n"); + if (*data) HeapFree(GetProcessHeap(), 0, *data); return RPC_S_PROTOCOL_ERROR; } -- 1.8.4.5