diff options
author | George Mount <mount@google.com> | 2016-03-01 12:09:20 -0800 |
---|---|---|
committer | George Mount <mount@google.com> | 2016-03-01 12:09:20 -0800 |
commit | 2bf1d3b17e04fd03ff447d57f770c89f935f52ae (patch) | |
tree | 1550a21d4dc47f1ab5bdf578d091eecd2571c0f6 /baseLibrary | |
parent | b522c7650bf7d9ec566845bc9eb37e761eea853d (diff) | |
download | data-binding-2bf1d3b17e04fd03ff447d57f770c89f935f52ae.tar.gz |
Check that callbacks are not null before adding them.
Fix future NullPointerException by checking for null callbacks
when they are added.
Change-Id: Icadd10d8f647a28cfe45977e18072cea17ea8a88
Diffstat (limited to 'baseLibrary')
-rw-r--r-- | baseLibrary/src/main/java/android/databinding/CallbackRegistry.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java b/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java index 0cd7b430..322b2794 100644 --- a/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java +++ b/baseLibrary/src/main/java/android/databinding/CallbackRegistry.java @@ -210,6 +210,9 @@ public class CallbackRegistry<C, T, A> implements Cloneable { * @param callback The callback to add. */ public synchronized void add(C callback) { + if (callback == null) { + throw new IllegalArgumentException("callback cannot be null"); + } int index = mCallbacks.lastIndexOf(callback); if (index < 0 || isRemoved(index)) { mCallbacks.add(callback); |