There is a double call to ompi_btl_openib_connect_base_open in

mca_btl_openib_mca_setup_qps().  It looks like someone just forgot to
clean-up the previous call when they added the check for the return
code.

I ran a quick IMB test over IB to verify everything is still working.

This commit was SVN r16870.

ompi/mca/btl/openib/btl_openib_mca.c

@@ -672,10 +672,7 @@ static int mca_btl_openib_mca_setup_qps(void)
     mca_btl_openib_component.credits_qp = smallest_pp_qp;
 
     /* Register any MCA params for the connect pseudo-components */
-
-    ompi_btl_openib_connect_base_open();
-
-    if ( OMPI_SUCCESS != ompi_btl_openib_connect_base_open())
+    if (OMPI_SUCCESS != ompi_btl_openib_connect_base_open())
         goto error;
 
     ret = OMPI_SUCCESS;