From a4b91cee0931e0b94a948a2f23358b21136b60bb Mon Sep 17 00:00:00 2001 From: Eric Laurent Date: Thu, 10 Jan 2019 15:05:41 -0800 Subject: [PATCH] fix regression RecordingActivityMonitor commit c7a0cdfe introduced a regression in audio recording activity monitoring by using the wrong key to lookup for a client in active recordings when a session stops. Bug: 122113404 Test: android.media.cts.AudioRecordingConfigurationTest#testAudioManagerGetActiveRecordConfigurations Change-Id: I06d87d086f391ae6e9adcd88f2235e002f88e682 --- .../java/com/android/server/audio/RecordingActivityMonitor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/audio/RecordingActivityMonitor.java b/services/core/java/com/android/server/audio/RecordingActivityMonitor.java index 905f826939807..9d6628cf0c405 100644 --- a/services/core/java/com/android/server/audio/RecordingActivityMonitor.java +++ b/services/core/java/com/android/server/audio/RecordingActivityMonitor.java @@ -206,7 +206,7 @@ public final class RecordingActivityMonitor implements AudioSystem.AudioRecordin switch (event) { case AudioManager.RECORD_CONFIG_EVENT_STOP: // return failure if an unknown recording session stopped - configChanged = (mRecordConfigs.remove(new Integer(session)) != null); + configChanged = (mRecordConfigs.remove(new Integer(portId)) != null); if (configChanged) { sEventLogger.log(new RecordingEvent(event, uid, session, source, null)); }