Re: [OMPI users] False positive from valgrind in sec_basic.c

2014-05-24 Thread Ralph Castain
It looks like the answer is "no", Jeff - at least, that didn't solve it for folks on the bug tracker cited by George. Setting the CFLAGS seemed the only solution until valgrind resolves the issue, and since that bug is a couple of years old with no further activity, it seems unlikely that will h

Re: [OMPI users] False positive from valgrind in sec_basic.c

2014-05-22 Thread Jeff Squyres (jsquyres)
Would a better solution be something like: char default_credential[8] = "12345"; char *bar = strdup(default_credential) ? On May 22, 2014, at 12:52 AM, George Bosilca wrote: > This is more subtle that described here. It's a vectorization problem > and frankly it should appear on all loop-base

Re: [OMPI users] False positive from valgrind in sec_basic.c

2014-05-22 Thread George Bosilca
This is more subtle that described here. It's a vectorization problem and frankly it should appear on all loop-based string operations and for most compilers (confirmed with gcc, clang and icc). The proposed patch is merely a band-aid ... More info @ https://bugs.launchpad.net/ubuntu/+source/valgr

Re: [OMPI users] False positive from valgrind in sec_basic.c

2014-05-21 Thread Ralph Castain
Wow, that is hilarious. I see no problem with adding the extra characters :-) Scheduled it for 1.8.2 (copied you on ticket) On May 21, 2014, at 3:29 PM, W Spector wrote: > Hi, > > When running under valgrind, I get warnings from each MPI process at MPI_Init > time. The warnings come from fu

[OMPI users] False positive from valgrind in sec_basic.c

2014-05-21 Thread W Spector
Hi, When running under valgrind, I get warnings from each MPI process at MPI_Init time. The warnings come from function sec_basic.c at lines 70 and 71 (openmpi v1.8.1): my_cred.credential = strdup("12345"); my_cred.size = strlen(my_cred.credential)+1; // include the