From a068e459aecb51abac2f86ad1895ae844fdf1bb2 Mon Sep 17 00:00:00 2001 From: Chinmay Garde Date: Wed, 7 Oct 2020 17:57:04 -0700 Subject: [PATCH] Avoid leaking the FlutterEngineAOTData structure in FlutterEngineCollectAOTData. (#21680) --- shell/platform/embedder/embedder.cc | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/shell/platform/embedder/embedder.cc b/shell/platform/embedder/embedder.cc index 506a5418cff..971ef22024f 100644 --- a/shell/platform/embedder/embedder.cc +++ b/shell/platform/embedder/embedder.cc @@ -651,13 +651,13 @@ FlutterEngineResult FlutterEngineCreateAOTData( } FlutterEngineResult FlutterEngineCollectAOTData(FlutterEngineAOTData data) { - if (data) { - data->loaded_elf = nullptr; - data->vm_snapshot_data = nullptr; - data->vm_snapshot_instrs = nullptr; - data->vm_isolate_data = nullptr; - data->vm_isolate_instrs = nullptr; + if (!data) { + // Deleting a null object should be a no-op. + return kSuccess; } + + // Created in a unique pointer in `FlutterEngineCreateAOTData`. + delete data; return kSuccess; }