Free CurrentUnitData in dtor

[yingcong-wu]

Summary

In Fuzzer, CurrentUnitData is allocated but never get free, so free it in the destructor.

[yingcong-wu]

formatting

[vitalybuka]

Fuzzer is never deleted. I added ~Fuzzer() = delete; to be sure

compiler-rt/lib/fuzzer/FuzzerLoop.cpp

162–164

patch is wrong, code like this is not in the tree

[yingcong-wu]

In D145035#4176999, @vitalybuka wrote:

Fuzzer is never deleted. I added ~Fuzzer() = delete; to be sure

Okay. I will close this revision.

compiler-rt/lib/fuzzer/FuzzerLoop.cpp

Fuzzer::Fuzzer(UserCallback CB, InputCorpus &Corpus, MutationDispatcher &MD,
   if (!Options.OutputCorpus.empty() && Options.ReloadIntervalSec)
     EpochOfLastReadOfOutputCorpus = GetEpoch(Options.OutputCorpus);
   MaxInputLen = MaxMutationLen = Options.MaxLen;
   TmpMaxMutationLen = 0;  // Will be set once we load the corpus.
   AllocateCurrentUnitData();
   CurrentUnitSize = 0;
   memset(BaseSha1, 0, sizeof(BaseSha1));
 }
 
-Fuzzer::~Fuzzer() {
-  delete[] CurrentUnitData;
-}
+Fuzzer::~Fuzzer() { delete[] CurrentUnitData; }
 
 void Fuzzer::AllocateCurrentUnitData() {
   if (CurrentUnitData || MaxInputLen == 0)
     return;
   CurrentUnitData = new uint8_t[MaxInputLen];
 }
 
 void Fuzzer::StaticDeathCallback() {
   assert(F);