EinsteinInitialData/TwoPunctures: Allocated memory is not freed

[Zach Etienne]

In Newton.c , lines 375-376:

  if (*normres <= tol)
    return 0;

returns from the function after temporary memory for the Newton() function has been allocated, but does not free that memory. valgrind confirms this issue.

I attach an updated version of Newton.c that should fix the issue.

[Roland Haas]

Can you provide a diff instead of the edited file, please? The edited file makes it harder to see what was actually changed and thus review.

[Zach Etienne]

I attach the patch. Note that 90%+ of the patch is just updated indentation to account for the update in logic needed for the fix.

[Zach Etienne]

Please review.

[Roland Haas]

Please apply.

[Roland Haas]

Applied as git hash 6b47026d "TwoPunctures: fix memory leak in Newton function" of einsteininitialdata