[kernel] Free memory for kernel data.

runtime/bin/dfe.cc

 // Copyright (c) 2017, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "bin/dfe.h"
 #include "bin/dartutils.h"
 #include "bin/error_exit.h"
 #include "bin/file.h"
 
 #include "vm/kernel.h"
@@ skipping 24 matching lines @@
     delete reinterpret_cast<kernel::Program*>(kernel_platform_);
     kernel_platform_ = NULL;
   }
 
   if (kernel_vmservice_io_ != NULL) {
     delete reinterpret_cast<kernel::Program*>(kernel_vmservice_io_);
     kernel_vmservice_io_ = NULL;
   }
 }
 
+void DFE::unset_kernel_vmservice_io() {

[Kevin Millikin (Google), 2017/08/14 06:20:50]

"unset" isn't really a word.  Usually "clear" is the opposite of "set", though
here I'd probably use something like ReleaseVMServiceIO to make it seem more
like it's giving up ownership.

[jensj, 2017/08/14 08:38:12]

Renaming to 'clear_kernel_vmservice_io' as I find 'Release' seems like it
releases resources (e.g. frees the memory), which it doesn't.

+  kernel_vmservice_io_ = NULL;
+}
+
 void DFE::SetKernelBinaries(const char* name) {
   intptr_t len = snprintf(NULL, 0, "%s%s%s", name, File::PathSeparator(),
                           kPlatformBinaryName) +
                  1;
   platform_binary_filename_ = new char[len];
   snprintf(platform_binary_filename_, len, "%s%s%s", name,
            File::PathSeparator(), kPlatformBinaryName);
 
   len = snprintf(NULL, 0, "%s%s%s", name, File::PathSeparator(),
                  kVMServiceIOBinaryName) +
@@ skipping 105 matching lines @@
         *kernel_ir = buffer;
         return true;
       }
     }
   }
   return false;
 }
 
 }  // namespace bin
 }  // namespace dart