I found a small bug in function mca_mpool_base_module_create() where if a
bad name is provided for mpool component, last iterated component is
returned, instead of NULL. I suggest a solution:

===================================================================
--- ompi/mca/mpool/base/mpool_base_lookup.c     (revision 21042)
+++ ompi/mca/mpool/base/mpool_base_lookup.c     (working copy)
@@ -75,11 +75,14 @@
     for (item = opal_list_get_first(&mca_mpool_base_components);
          item != opal_list_get_end(&mca_mpool_base_components);
          item = opal_list_get_next(item)) {
+         mca_mpool_base_component_t* c;
+
          mca_base_component_list_item_t *cli =
            (mca_base_component_list_item_t *) item;
-         component =
+         c =
            (mca_mpool_base_component_t *) cli->cli_component;
-         if(0 == strcmp(component->mpool_version.mca_component_name,
name)) {
+         if(0 == strcmp(c->mpool_version.mca_component_name, name)) {
+             component = c;
              break;
          }
     }


Reply via email to