looking at the old code, I understand what you tried to fix, I'll commit
a proper version in a couple of min.
Thanks
Edgar
On 5/28/2015 8:44 AM, Edgar Gabriel wrote:
ok, I see you moved the malloc up, the malloc was originally just for
the receiving side of the communication, you moved it up to cover both.
That is however unfortunately not correct.
I will fix it in a couple of mins.
Thanks
Edgar
On 5/28/2015 8:25 AM, Edgar Gabriel wrote:
Gilles,
I saw you a fixed a couple of the coverty warnings in ompio, but
unfortunately it also broke some things.
--
Question to you: in io_ompio.c line 2345, you introduced a malloc for
f_procs_in_group that was not there before. That array is allocated a
couple of lines earlier in the subroutine merge_groups
Since the values are not set, we segfault right away a couple of lines
later in the subsequent isend, where f_procs_in_group[0] simply does not
have a value.
Can I ask what the problem was that you tried to fix? Because purely
from the logic perspective, that malloc needs to go.
---
Thanks
Edgar
--
Edgar Gabriel
Associate Professor
Parallel Software Technologies Lab http://pstl.cs.uh.edu
Department of Computer Science University of Houston
Philip G. Hoffman Hall, Room 524 Houston, TX-77204, USA
Tel: +1 (713) 743-3857 Fax: +1 (713) 743-3335
--
