Free communicator only if MPI has not been finalized

[kmittal2]

Currently comm_free causes a segfault when the memory allocated by findpts is attempted to be freed after MPI_Finalize has been called.

[stgeke]

Isn't this expected? I would argue that it's illegal to call comm_free after MPI_Finalize.

[kmittal2]

Yes, it is expected.. We are trying to eliminate the need for the user to free findpts data manually by moving the call to findpts_free to the destructor of the class that sets up the findpts object. This destructor can potentially be called after MPI_Finalize depending on how the user has setup their driver.

A non-invasive solution is to skip findpts_free in the destructor if MPI_Finalize has been called, but that would result in a memory leak.