Thanks!  I've committed a slightly different fix here:

    https://svn.open-mpi.org/trac/ompi/changeset/21044


On Apr 20, 2009, at 10:39 AM, Roberto Ammendola wrote:


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;
          }
     }


_______________________________________________
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel


--
Jeff Squyres
Cisco Systems

Reply via email to