* do reference count correctly for the spawn handles - any allocated handle

should not drop to zero until we make it to ompi_rte_finalize()
* do the spawn handle cleanup before mca_pcm_base_close() so we don't call
  into unmapped memory

This commit was SVN r3437.

src/runtime/ompi_rte_finalize.c

@@ -32,6 +32,9 @@
  */
 int ompi_rte_finalize(void)
 {
+  ompi_rte_wait_finalize();
+  ompi_rte_internal_fini_spawn();
+
   mca_pcm_base_close();
   mca_llm_base_close();
   mca_pcmclient_base_close();
@@ -39,9 +42,6 @@ int ompi_rte_finalize(void)
   mca_gpr_base_close();
   mca_oob_base_close();
 
-  ompi_rte_wait_finalize();
-  ompi_rte_internal_fini_spawn();
-
   ompi_event_fini();
 
     ompi_session_dir_finalize();

src/runtime/ompi_rte_pcm.c

@@ -105,6 +105,8 @@ ompi_rte_get_spawn_handle(int criteria, bool have_threads)
     }
 
     ompi_pointer_array_add(&avail_handles, ptr);
+    /* retain for the array */
+    OBJ_RETAIN(ptr);
 
  cleanup:
     OMPI_THREAD_UNLOCK(&avail_handles_mutex);