Patch two memory leaks that cause more leaks

[tepperly]

This PR plugs some memory leaks.

• Patch two leaked pointers that are the root cause of multiple other leaks.

Overall, I would strongly recommend replacing most if not all the raw pointers with std::unique_ptr. std::unique_ptr has numerous benefits over raw pointers including:

• automatic deletion when the pointer goes out of scope
• automatic initialization to nullptr if it's not initialized explicitly
• essentially no runtime or memory overhead compared to raw pointers

In the case of dir_cv_ and res_cv_, it's not clear to me why their pointers in the first place. IMHO, it seems like judicious use references instead of pointers could get rid of a variety of new/delete's or std::unique_ptr.

[tepperly]

FYI, I don't believe I have permission to merge this. Someone from the project will need to shepherd this PR into the code if you agree with it.

[cnpetra]

FYI, I don't believe I have permission to merge this. Someone from the project will need to shepherd this PR into the code if you agree with it.

is designed like that :) I just approved it.