diff options
author | Tyler Gunn <tgunn@google.com> | 2021-04-30 16:23:15 -0700 |
---|---|---|
committer | Tyler Gunn <tgunn@google.com> | 2021-04-30 16:23:15 -0700 |
commit | 4cd4248dce8689ce9caca3ecfe533a778a64c39b (patch) | |
tree | 53af4ec49bff19789fc7de9dd78d77aff8084bfb /telecomm | |
parent | c05b6dbe204459c37fc938339922831f479e481e (diff) |
Fix potential strict mode violation when stopping RTT.
When the RTT call was cleaned up, the "close()" method was not being
called. In practice I think the Dialer app never really stops RTT, but
the APIs (and test app) do allow it to be stopped mid-session. I'm
hypothesizing that is why we don't see this.
Test: Use Telecom test dialer to start and end RTT session. Observe no
strict mode violation.
Fixes: 186869727
Change-Id: Id474b493ac3fe39e97360f3ac326279fc9a9a5eb
Diffstat (limited to 'telecomm')
-rwxr-xr-x | telecomm/java/android/telecom/Call.java | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/telecomm/java/android/telecom/Call.java b/telecomm/java/android/telecom/Call.java index d77ab2b24b31..f0d43faed5ed 100755 --- a/telecomm/java/android/telecom/Call.java +++ b/telecomm/java/android/telecom/Call.java @@ -2543,6 +2543,7 @@ public final class Call { } else if (mRttCall != null && parcelableCall.getParcelableRttCall() == null && parcelableCall.getIsRttCallChanged()) { isRttChanged = true; + mRttCall.close(); mRttCall = null; } |