| commit | fbc33cab7b95e1f6795ef3783ce54387925143a4 | |
| author | Paul Bauer <paul.bauer.q@gmail.com> | |
| Mon, 16 Apr 2018 14:16:23 +0000 (16 16:16 +0200) | ||
| committer | Mark Abraham <mark.j.abraham@gmail.com> | |
| Tue, 17 Apr 2018 10:57:28 +0000 (17 12:57 +0200) | ||
| tree | 033f52aa5cb465d6abee17e30361ede46159abb1 | treesnapshot (tar.gz zip) |
| parent | e9a53455cd161589367c05481f81d3344fb6be06 | commitdiff |
Fix legacy topology memory error
The change to port mtop to c++ has exposed a number of issues in the
legacy topology data structure, where pointers would be copied from a
temporary mtop object to the local topology, and subsequently
invalidated when mtop was correctly free'd.
This change is a bit of wooden mallet applied to the problem, by brute
forcing the assignment of the values that were previously copied by
pointer.
Fixes #2479
Change-Id: I7d702666fa48cb3840f914ee3bd28787d3caeec8
The change to port mtop to c++ has exposed a number of issues in the
legacy topology data structure, where pointers would be copied from a
temporary mtop object to the local topology, and subsequently
invalidated when mtop was correctly free'd.
This change is a bit of wooden mallet applied to the problem, by brute
forcing the assignment of the values that were previously copied by
pointer.
Fixes #2479
Change-Id: I7d702666fa48cb3840f914ee3bd28787d3caeec8
| src/gromacs/topology/mtop_util.cpp | diffblobblamehistory | |
| src/gromacs/topology/topology.cpp | diffblobblamehistory |