From abf1b728b9dcf13f12cb66e65f552ade8b0b8926 Mon Sep 17 00:00:00 2001 From: Jeff Squyres Date: Fri, 19 Oct 2007 12:36:26 +0000 Subject: [PATCH] Minor code maintenance fix -- put the THREAD_UNLOCK outside the if statement so that you only have to have it once. This commit was SVN r16512. --- orte/mca/oob/tcp/oob_tcp.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/orte/mca/oob/tcp/oob_tcp.c b/orte/mca/oob/tcp/oob_tcp.c index 52967f7465..d984175aaa 100644 --- a/orte/mca/oob/tcp/oob_tcp.c +++ b/orte/mca/oob/tcp/oob_tcp.c @@ -1057,12 +1057,11 @@ int mca_oob_tcp_resolve(mca_oob_tcp_peer_t* peer) OPAL_THREAD_LOCK(&mca_oob_tcp_component.tcp_lock); addr = (mca_oob_tcp_addr_t *)orte_hash_table_get_proc(&mca_oob_tcp_component.tcp_peer_names, &peer->peer_name); + OPAL_THREAD_UNLOCK(&mca_oob_tcp_component.tcp_lock); if(NULL != addr) { - OPAL_THREAD_UNLOCK(&mca_oob_tcp_component.tcp_lock); mca_oob_tcp_peer_resolved(peer, addr); return ORTE_SUCCESS; } - OPAL_THREAD_UNLOCK(&mca_oob_tcp_component.tcp_lock); /* if we don't know it, then report unknown - don't try to go get it */ return ORTE_ERR_ADDRESSEE_UNKNOWN;