diff options
author | Ankit Goyal <layog@google.com> | 2022-04-13 10:38:49 -0700 |
---|---|---|
committer | Ankit Goyal <layog@google.com> | 2022-04-13 11:03:31 -0700 |
commit | 647eca137a4a4c15f77395c0e00a019deef0f439 (patch) | |
tree | 863ab62e25b1f2f7ed7a6ed922a4a0082df39ed6 | |
parent | 0a6200f15ff328b1ae52a0a537a69ba72f0a735c (diff) | |
download | gchips-647eca137a4a4c15f77395c0e00a019deef0f439.tar.gz |
Allow empty SMPTE2094-40 metadata
Sometimes app can unset HDR10+ metadata by sending an empty vector.
Fix: 229032171
Bug: 226556946
Test: No log spam in logcat
Change-Id: Iacfcedf1bd618cdae0411e8c8ee643787170e73e
-rw-r--r-- | gralloc4/src/hidl_common/SharedMetadata.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/gralloc4/src/hidl_common/SharedMetadata.cpp b/gralloc4/src/hidl_common/SharedMetadata.cpp index 4247bd8..b5964f7 100644 --- a/gralloc4/src/hidl_common/SharedMetadata.cpp +++ b/gralloc4/src/hidl_common/SharedMetadata.cpp @@ -146,12 +146,6 @@ void get_smpte2094_40(const private_handle_t *hnd, std::optional<std::vector<uin android::status_t set_smpte2094_40(const private_handle_t *hnd, const std::optional<std::vector<uint8_t>> &smpte2094_40) { - if (!smpte2094_40.has_value() || smpte2094_40->size() == 0) - { - MALI_GRALLOC_LOGE("Empty SMPTE 2094-40 data"); - return android::BAD_VALUE; - } - auto *metadata = reinterpret_cast<shared_metadata *>(hnd->attr_base); const size_t size = smpte2094_40.has_value() ? smpte2094_40->size() : 0; if (size > metadata->smpte2094_40.capacity()) |