From 129b1451338d5a1770baac4cc0547056991d3e04 Mon Sep 17 00:00:00 2001 From: jiabin Date: Wed, 31 Jan 2024 23:56:26 +0000 Subject: [PATCH] AHAL: add default values for dynamic port config of hardcoded config. If the values(format, sample rate, channel mask) are left as unset, they will be null. In that case, when using them will require checking nullablitiy to avoid crash. Bug: 323255506 Test: make and boot Change-Id: Ic66bf0b6f53750ff8700e531cca13f03fbb72936 --- audio/aidl/default/Configuration.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/audio/aidl/default/Configuration.cpp b/audio/aidl/default/Configuration.cpp index 2a8e58f843..54e2d18122 100644 --- a/audio/aidl/default/Configuration.cpp +++ b/audio/aidl/default/Configuration.cpp @@ -110,6 +110,9 @@ static AudioPortConfig createDynamicPortConfig(int32_t id, int32_t portId, int32 AudioPortConfig config; config.id = id; config.portId = portId; + config.format = AudioFormatDescription{}; + config.channelMask = AudioChannelLayout{}; + config.sampleRate = Int{.value = 0}; config.gain = AudioGainConfig(); config.flags = isInput ? AudioIoFlags::make(flags) : AudioIoFlags::make(flags);