- Fix a problem reported on the users list where we would segfault in finalize after calling spawn if the user did not call MPI_Comm_disconnect

- Fix the app context constructor so it initializes all the fields.

This commit was SVN r18079.

ompi/mca/dpm/base/dpm_base_common_fns.c

@@ -66,7 +66,7 @@ int ompi_dpm_base_dyn_finalize (void)
         max = opal_pointer_array_get_size(&ompi_mpi_communicators);
         for ( i=3; i<max; i++ ) {
             comm = (ompi_communicator_t*)opal_pointer_array_get_item(&ompi_mpi_communicators,i);
-            if ( OMPI_COMM_IS_DYNAMIC(comm)) {
+            if (NULL != comm &&  OMPI_COMM_IS_DYNAMIC(comm)) {
                 objs[j++]=ompi_dpm_base_disconnect_init(comm);
             }
         }

orte/runtime/orte_globals_class_instances.h

@@ -47,6 +47,7 @@ static void orte_app_context_construct(orte_app_context_t* app_context)
     app_context->argv=NULL;
     app_context->env=NULL;
     app_context->cwd=NULL;
+    app_context->user_specified_cwd=false;
     app_context->hostfile=NULL;
     app_context->add_hostfile=NULL;
     app_context->dash_host = NULL;