You are absolutely correct.  I've filed CMRs for v1.4 and v1.5.

Thanks for the heads up!


On Dec 1, 2009, at 4:26 PM, David Singleton wrote:


It looks like MPI_Cart_create argument checking was fixed in 1.3.4
but not MPI_Graph_create.

vayu1:~/openmpi-1.3.4 > diff -w -u ompi/mpi/c/cart_create.c ompi/mpi/ c/graph_create.c

...

-int MPI_Cart_create(MPI_Comm old_comm, int ndims, int *dims,
-                    int *periods, int reorder, MPI_Comm *comm_cart) {
+int MPI_Graph_create(MPI_Comm old_comm, int nnodes, int *index,
+                     int *edges, int reorder, MPI_Comm *comm_graph)
+{

...

+        if ((0 > reorder) || (1 < reorder)) {


David

David Singleton wrote:
>
> Kiril Dichev has already pointed a problem with MPI_Cart_create
> http://www.open-mpi.org/community/lists/devel/2009/08/6627.php
> MPI_Graph_create has the same problem.  I checked all other
> functions with logical in arguments and no others do anything
> similar.
>
_______________________________________________
devel mailing list
de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/devel



--
Jeff Squyres
jsquy...@cisco.com

Reply via email to