| commit | ddefa7b9f88ca3c0b03f1105725d864f47c94429 | [log] [tgz] |
|---|---|---|
| author | Greg Kaiser <gkaiser@google.com> | Mon Jun 13 06:19:26 2022 -0700 |
| committer | Greg Kaiser <gkaiser@google.com> | Mon Jun 13 06:19:26 2022 -0700 |
| tree | ad9b3cb0b7cf2b84e7ff537bd0f2154f67b95462 | |
| parent | 7d79eb7410f19b4edfa1a43b5452e4f2b4d52085 [diff] [blame] |
Set prior to performing std::move() We were calling a set method after a std::move(), which is not assured to work. Test: TreeHugger Bug: 230462538 Change-Id: I4fb18f4ba94f6046c502b850dc0ce6143ad6e3b6
diff --git a/services/surfaceflinger/Tracing/LayerTracing.cpp b/services/surfaceflinger/Tracing/LayerTracing.cpp index 1b80fa3..afa3e50 100644 --- a/services/surfaceflinger/Tracing/LayerTracing.cpp +++ b/services/surfaceflinger/Tracing/LayerTracing.cpp
@@ -129,8 +129,8 @@ entry.set_excludes_composition_state(true); } mFlinger.dumpDisplayProto(entry); - mBuffer->emplace(std::move(entry)); entry.set_vsync_id(vsyncId); + mBuffer->emplace(std::move(entry)); } } // namespace android