commit 964a32fada06a2501084ee0c2a9c3ece3e3ba52b
author Daniele Di Proietto <ddiproietto@google.com> Wed Feb 28 15:30:34 2024 +0000
committer Daniele Di Proietto <ddiproietto@google.com> Wed Feb 28 16:16:02 2024 +0000
tree 57c8fa4f1650853cefedea2666775abc8bd71442
parent 1f40922d0be39fee83dc4d180c63fedb1536cfa3

Fix TraceBlob move assignment operator

There was a subtle bug in TraceBlob move assignment operator, where the
memory owned by this trace blob would be leaked.

Fix this, as suggested by Primiano, by explicitly implementing the move
constructor and using the move constructor from the move assignment
operator.

This was not an actual problem, nothing in the codebase calls TraceBlob.
I'm fixing it anyway because TraceBlob is part of the public interface.

Change-Id: Ia0bed4c89612470abc80ccbfd0df9ea3434c180b

Android.bp

diff --git a/Android.bp b/Android.bp
index ece4ad0..6f451f9 100644
--- a/Android.bp
+++ b/Android.bp
@@ -12286,6 +12286,7 @@
     srcs: [
         "src/trace_processor/forwarding_trace_parser_unittest.cc",
         "src/trace_processor/ref_counted_unittest.cc",
+        "src/trace_processor/trace_blob_unittest.cc",
     ],
 }