Reported by Coverity. Fixes: a2a06860b8c4 ("ipc: fix memory leak on request failure") Cc: Herakliusz Lipiec <herakliusz.lip...@intel.com> Cc: sta...@dpdk.org Cc: Anatoly Burakov <anatoly.bura...@intel.com> Signed-off-by: Aaron Conole <acon...@redhat.com> --- lib/librte_eal/common/eal_common_proc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/lib/librte_eal/common/eal_common_proc.c b/lib/librte_eal/common/eal_common_proc.c index d23728604..3498e6b07 100644 --- a/lib/librte_eal/common/eal_common_proc.c +++ b/lib/librte_eal/common/eal_common_proc.c @@ -1005,8 +1005,10 @@ rte_mp_request_sync(struct rte_mp_msg *req, struct rte_mp_reply *reply, /* unlocks the mutex while waiting for response, * locks on receive */ - if (mp_request_sync(path, req, reply, &end)) + if (mp_request_sync(path, req, reply, &end)) { + pthread_mutex_unlock(&pending_requests.lock); goto err; + } } pthread_mutex_unlock(&pending_requests.lock); /* unlock the directory */ -- 2.19.1