Nikolay Sivov : rpcss: Fix potential leaks when removing endpoint ( Coverity).
Module: wine Branch: master Commit: a87c510e9a7ff3fd748920525425b7e317a0bbdf URL: http://source.winehq.org/git/wine.git/?a=commit;h=a87c510e9a7ff3fd7489205254... Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun May 22 21:44:47 2016 +0300 rpcss: Fix potential leaks when removing endpoint (Coverity). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/rpcss/epmp.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/programs/rpcss/epmp.c b/programs/rpcss/epmp.c index 2eb6039..3243192 100644 --- a/programs/rpcss/epmp.c +++ b/programs/rpcss/epmp.c @@ -165,6 +165,11 @@ void __cdecl ept_delete(handle_t h, if (rpc_status != RPC_S_OK) break; entry = find_ept_entry(&iface, &syntax, protseq, endpoint, address, &entries[i].object); + + I_RpcFree(protseq); + I_RpcFree(endpoint); + I_RpcFree(address); + if (entry) delete_registered_ept_entry(entry); else @@ -172,9 +177,6 @@ void __cdecl ept_delete(handle_t h, *status = EPT_S_NOT_REGISTERED; break; } - I_RpcFree(protseq); - I_RpcFree(endpoint); - I_RpcFree(address); } LeaveCriticalSection(&csEpm);
participants (1)
-
Alexandre Julliard