diff options
author | Wiwit Rifa'i <wiwitrifai@google.com> | 2022-03-02 16:27:13 +0800 |
---|---|---|
committer | Wiwit Rifa'i <wiwitrifai@google.com> | 2022-03-02 09:46:14 +0000 |
commit | e99e2f48c694ab97c4d642cae22882e648051f99 (patch) | |
tree | cb336af13ccbf662f438a71464cb45e9571b2472 | |
parent | c67d83ad3dc4b7da3c3d01d289f0bb646392d4a9 (diff) |
hwc3: fix null dereference of outDimmingStage in validateDisplay()
We should not pass nullptr to outDimmingStage parameter to
validateDisplay() since it will eventually be dereferenced inside
validateDisplay().
Bug: 222153332
Test: build
Change-Id: I510eb3ef5525cb148cbc9bad116acbadc2fa0f5f
-rw-r--r-- | hwc3/ComposerClient.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/hwc3/ComposerClient.cpp b/hwc3/ComposerClient.cpp index 6997087..b576b68 100644 --- a/hwc3/ComposerClient.cpp +++ b/hwc3/ComposerClient.cpp @@ -553,9 +553,10 @@ void ComposerClient::destroyResources() { std::vector<int64_t> requestedLayers; std::vector<int32_t> requestMasks; ClientTargetProperty clientTargetProperty; + DimmingStage dimmingStage; mHal->validateDisplay(display, &changedLayers, &compositionTypes, &displayRequestMask, &requestedLayers, &requestMasks, &clientTargetProperty, - nullptr /* outDimmingStage */); + &dimmingStage); mHal->acceptDisplayChanges(display); ndk::ScopedFileDescriptor presentFence; |