commit c85693d02db99e5eb4ffd4c6464a4aca201c26a2
author Brian Lindahl <blindahl@google.com> Fri Mar 31 07:31:51 2023 -0600
committer Brian Lindahl <blindahl@google.com> Fri Mar 31 07:45:18 2023 -0600
tree 75efe1e34e6de90e8dc46213a9accfc595e53062
parent 25e0d09c766e1a573d1dfc8be285f5da4aed43e4

Fix bug where buffer slots to clear were not set correctly

Bug: 276417624
Bug: 258196272
Test: VtsHalGraphicsComposer3_TargetTest
Change-Id: Ie077f14140d71f88a497c4a8995d73dd07c31598

graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h

diff --git a/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h b/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h
index 22020c0..4a5ac69 100644
--- a/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h
+++ b/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h
@@ -144,10 +144,8 @@
 
     void setLayerBufferSlotsToClear(int64_t display, int64_t layer,
                                     const std::vector<uint32_t>& slotsToClear) {
-        LayerCommand& layerCommand = getLayerCommand(display, layer);
-        for (auto slot : slotsToClear) {
-            layerCommand.bufferSlotsToClear.emplace(static_cast<int32_t>(slot));
-        }
+        getLayerCommand(display, layer)
+                .bufferSlotsToClear.emplace(slotsToClear.begin(), slotsToClear.end());
     }
 
     void setLayerSurfaceDamage(int64_t display, int64_t layer, const std::vector<Rect>& damage) {