ofi: Fix memory handler unregistration

Resetting the installed_memory_monitor flag should be in the error
handling case, rather than blindly set, so that the memory
hook handler can be properly deregistered during shutdown.

This change fixes a segfault during shutdown if the common/ofi
component was built as a dynamic object.

Signed-off-by: Brian Barrett <bbarrett@amazon.com>

Author: bwbarrett

opal/mca/common/ofi/common_ofi.c

@@ -187,9 +187,9 @@ int opal_common_ofi_export_memory_monitor(void)
         if (NULL != opal_common_ofi_monitor) {
             free(opal_common_ofi_monitor);
         }
-    }
 
-    opal_common_ofi_installed_memory_monitor = false;
+        opal_common_ofi_installed_memory_monitor = false;
+    }
 
     OPAL_THREAD_UNLOCK(&opal_common_ofi_mutex);
 #endif