diff options
author | Jonathan Basseri <misterikkit@google.com> | 2015-04-23 23:34:15 -0700 |
---|---|---|
committer | Jonathan Basseri <misterikkit@google.com> | 2015-04-23 23:34:15 -0700 |
commit | 9892718663be17e00cc20e82d75a532161bcd613 (patch) | |
tree | c4327c31e3b1d6c55e1e6623564e3311bb956e5c /telephony | |
parent | 3e452a9b9603a7a4298a2e1e1737cac01be567d0 (diff) | |
download | frameworks_base-9892718663be17e00cc20e82d75a532161bcd613.zip frameworks_base-9892718663be17e00cc20e82d75a532161bcd613.tar.gz frameworks_base-9892718663be17e00cc20e82d75a532161bcd613.tar.bz2 |
Never return a reference to sDefaults.
Final static fields in Java are still mutable, so CarrierConfigManager
should not give callers the opportunity to mutate them.
Change-Id: I413597fc5f59c249997967bb0aad2d09b54fa613
Diffstat (limited to 'telephony')
-rw-r--r-- | telephony/java/android/telephony/CarrierConfigManager.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/telephony/java/android/telephony/CarrierConfigManager.java b/telephony/java/android/telephony/CarrierConfigManager.java index 7d1a2fa..831a194 100644 --- a/telephony/java/android/telephony/CarrierConfigManager.java +++ b/telephony/java/android/telephony/CarrierConfigManager.java @@ -208,13 +208,13 @@ public class CarrierConfigManager { } /** - * Returns a bundle with the default value for every supported configuration variable. + * Returns a new bundle with the default value for every supported configuration variable. * * @hide */ @SystemApi public static Bundle getDefaultConfig() { - return sDefaults; + return new Bundle(sDefaults); } /** @hide */ |