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

Reply via email to