diff options
| author | 2022-10-18 07:49:34 +0000 | |
|---|---|---|
| committer | 2022-10-18 07:49:34 +0000 | |
| commit | 761c38a4888b6b14a38c8c42a11f719785a57c07 (patch) | |
| tree | b3c7b5ad6420dfff2a431f1ecce3e1ad027d0202 | |
| parent | 0ff19726270d8b10626e36a7fcd0fbadff44b6e1 (diff) | |
| parent | e754249f83d8ba830249a07217557307674fd782 (diff) | |
Merge "Fix potential NPE when preference is null" am: 2b6a72ba1f am: b5e5983379 am: e754249f83
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2256881
Change-Id: Ic7c7c34dae8b03734ce5b381a862560cd91dedfb
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | packages/SettingsLib/src/com/android/settingslib/core/AbstractPreferenceController.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/packages/SettingsLib/src/com/android/settingslib/core/AbstractPreferenceController.java b/packages/SettingsLib/src/com/android/settingslib/core/AbstractPreferenceController.java index 988055e7d8db..9567a3b38896 100644 --- a/packages/SettingsLib/src/com/android/settingslib/core/AbstractPreferenceController.java +++ b/packages/SettingsLib/src/com/android/settingslib/core/AbstractPreferenceController.java @@ -41,8 +41,10 @@ public abstract class AbstractPreferenceController { setVisible(screen, prefKey, true /* visible */); if (this instanceof Preference.OnPreferenceChangeListener) { final Preference preference = screen.findPreference(prefKey); - preference.setOnPreferenceChangeListener( - (Preference.OnPreferenceChangeListener) this); + if (preference != null) { + preference.setOnPreferenceChangeListener( + (Preference.OnPreferenceChangeListener) this); + } } } else { setVisible(screen, prefKey, false /* visible */); |