summaryrefslogtreecommitdiffstats
path: root/services
diff options
context:
space:
mode:
authorjiabin <jiabin@google.com>2018-04-05 17:16:20 -0700
committerJiabin Huang <jiabin@google.com>2018-04-06 17:22:28 +0000
commitebe0777edcf3b9c6bde9771d65399e2363dc6e40 (patch)
treec2176c544738944bc5efd73becc6243c58909ec5 /services
parentedcb4521760ec872d2ecf7df9ddf8a42c4c7a7bd (diff)
downloadframeworks_av-ebe0777edcf3b9c6bde9771d65399e2363dc6e40.tar.gz
frameworks_av-ebe0777edcf3b9c6bde9771d65399e2363dc6e40.tar.bz2
frameworks_av-ebe0777edcf3b9c6bde9771d65399e2363dc6e40.zip
Init gain config to prevent uninit leak.
In AudioPortConfig, we only initialize index for audio_gain_config, but not other fields. That may cause uninit leak at listAudioPorts and listAudioPatches. Bug: 77238250 Bug: 77238762 Test: try repo steps at the bug description. Change-Id: I57e3bd0598f9aa698a6fa3d3c0218b046de34e2f
Diffstat (limited to 'services')
-rw-r--r--services/audiopolicy/common/managerdefinitions/src/AudioPort.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/services/audiopolicy/common/managerdefinitions/src/AudioPort.cpp b/services/audiopolicy/common/managerdefinitions/src/AudioPort.cpp
index 4e24f19daa..169a599033 100644
--- a/services/audiopolicy/common/managerdefinitions/src/AudioPort.cpp
+++ b/services/audiopolicy/common/managerdefinitions/src/AudioPort.cpp
@@ -827,6 +827,7 @@ AudioPortConfig::AudioPortConfig()
mSamplingRate = 0;
mChannelMask = AUDIO_CHANNEL_NONE;
mFormat = AUDIO_FORMAT_INVALID;
+ memset(&mGain, 0, sizeof(struct audio_gain_config));
mGain.index = -1;
}