summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVikas batchu <quic_vikabatc@quicinc.com>2024-01-23 18:22:51 +0530
committerVikas Batchu <quic_vikabatc@quicinc.com>2024-02-08 11:55:49 +0000
commitd8db1f2be3178d7b880f9c0e14ab9b602efd2dff (patch)
tree7f00b4139830dd501138778d22d227d213b593e2
parentf4fc89eba0b9d418b59c08c4b3bed50e5a9fd450 (diff)
sf: use mStateLock in dumpDrawingStateProto
-- Fix race condition while collecting dumpsys. CRs-Fixed: 3707059 Change-Id: Id5411cf8faa053fbcab3345d4ca5e6e1479262da
-rw-r--r--services/surfaceflinger/SurfaceFlinger.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index efed751f2a..4c5546e47c 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -6230,6 +6230,7 @@ void SurfaceFlinger::dumpWideColorInfo(std::string& result) const {
}
LayersProto SurfaceFlinger::dumpDrawingStateProto(uint32_t traceFlags) const {
+ Mutex::Autolock _l(mStateLock);
std::unordered_set<uint64_t> stackIdsToSkip;
// Determine if virtual layers display should be skipped