From a212555ad2320238d42f439b96126493700cc860 Mon Sep 17 00:00:00 2001 From: Malcolm Chen Date: Thu, 19 Mar 2020 15:46:00 -0700 Subject: [PATCH] Fix bug that getBaseBandVersion() may write the value incorrectly. getBaseBandVersion tries to get value from a legacy API and rewrites it back. However the read value is the full list but the write is to a specific phone, causing the list to grow unexpectedly. Bug: 149342541 Test: manual - using test app, call API and check result. Change-Id: I78b0443c5e38a30010796fcb69f01a8c7ebaf160 --- .../android/telephony/TelephonyManager.java | 20 ------------------- 1 file changed, 20 deletions(-) diff --git a/telephony/java/android/telephony/TelephonyManager.java b/telephony/java/android/telephony/TelephonyManager.java index d6224ba2f731e..164764d344825 100644 --- a/telephony/java/android/telephony/TelephonyManager.java +++ b/telephony/java/android/telephony/TelephonyManager.java @@ -9818,22 +9818,6 @@ public class TelephonyManager { return getBasebandVersionForPhone(phoneId); } - /** - * Get baseband version for the default phone using the legacy approach. - * This change was added in P, to ensure backward compatiblity. - * - * @return baseband version. - * @hide - */ - private String getBasebandVersionLegacy(int phoneId) { - if (SubscriptionManager.isValidPhoneId(phoneId)) { - String prop = "gsm.version.baseband" - + ((phoneId == 0) ? "" : Integer.toString(phoneId)); - return SystemProperties.get(prop); - } - return null; - } - /** * Get baseband version by phone id. * @@ -9841,10 +9825,6 @@ public class TelephonyManager { * @hide */ public String getBasebandVersionForPhone(int phoneId) { - String version = getBasebandVersionLegacy(phoneId); - if (version != null && !version.isEmpty()) { - setBasebandVersionForPhone(phoneId, version); - } return getTelephonyProperty(phoneId, TelephonyProperties.baseband_version(), ""); }