Memory sanitizing

[clyne]

Valgrind reports a number of dynamic memory errors in the code base. We should try to eliminate as many as possible, starting with #2864, #2865, #2866, #2867, #2868.

N.B. some of the above issues report memory leaks in test drivers. The benefit of fixing leaks in test drivers is pretty minimal....

[shaomeng]

A few numbers collected on 10/1/2021 to demonstrate this problem: I opened VAPOR, loaded a CF dataset, closed the dataset, and closed VAPOR. This process is monitored by Valgrind and it reports the following issues:

• Conditional jump or move depends on uninitialised value(s) 40 occurrences.
• Invalid read of size xyz 119 occurrences.
• Use of uninitialised value of size xyz 30 occurences.
• invalid file descriptor 6 occurences.
• definitely lost memory blocks 222 occurrences.

[clyne]

Addressed by separate sub issues (see above)