commit e11e168d19efed8ddb35168e1de78666ed364926
author Vishnu Nair <vishnun@google.com> Thu Aug 17 14:21:15 2023 -0700
committer Vishnu Nair <vishnun@google.com> Thu Aug 17 21:27:00 2023 +0000
tree 10391a23d3c87848033970601359361c2a04af9e
parent 774bcbe5c307aeb0e61203294948709fad2b4b45

[sf] Fix override flag in TransactionTraceWriter

Bug: 238781169, 294989213
Test: presubmit
Test: force traces via shell
Change-Id: Ib222e6621f5dfe66693de42ef290aa23cf95b592

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 9de623f..848df03 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -915,7 +915,7 @@
             auto writeFn = [&]() {
                 const std::string filename =
                         TransactionTracing::DIR_NAME + prefix + TransactionTracing::FILE_NAME;
-                if (overwrite && access(filename.c_str(), F_OK) == 0) {
+                if (!overwrite && access(filename.c_str(), F_OK) == 0) {
                     ALOGD("TransactionTraceWriter: file=%s already exists", filename.c_str());
                     return;
                 }