[mpich-discuss] mpich2.1.4.1p1: intercomm_create.c

[Dave Goodell]

On Apr 18, 2013, at 9:43 PM CDT, Richard Muddle <rlmuddle at googlemail.com> wrote:

> In src/mpi/comm/intercomm_create.c from lines 58, the calculation of maxi seems to be incorrect when maxlpid is a multiple of 32.
> 
>     /* Compute the max index and zero the pids array */
>     maxi = (maxlpid + 31) / 32;
> 
>     if (maxi >= MAX_LPID32_ARRAY) {
> 	MPIU_CHKLMEM_MALLOC(lpidmask,int32_t*,maxi*sizeof(int32_t),
> 			    mpi_errno,"lpidmask");
>     }

Hi Richard,

Thanks for letting us know.  I believe I already fixed this bug a few years ago:

http://trac.mpich.org/projects/mpich/ticket/1574
http://git.mpich.org/mpich.git/commitdiff/40d88ed0
http://git.mpich.org/mpich.git/commitdiff/7159a5f9

Looks like those fixes were released in MPICH2 1.5b2:

----8<----
% git describe --contains 7159a5f9
v1.5b2~192
----8<----

-Dave