diff options
author | Jeff Sharkey <jsharkey@android.com> | 2009-10-29 16:39:36 -0700 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2009-10-29 17:38:27 -0700 |
commit | 824838d74eb0316f6987a1d98d2d9e9fa8d4e15b (patch) | |
tree | d6bb8df57fe9af97dbb63cc700dc03fa321c6586 /core/java/android/content/ContentProviderOperation.java | |
parent | 3a8141416a0b1d517d4729eba4bcf914c468e20b (diff) | |
download | frameworks_base-824838d74eb0316f6987a1d98d2d9e9fa8d4e15b.zip frameworks_base-824838d74eb0316f6987a1d98d2d9e9fa8d4e15b.tar.gz frameworks_base-824838d74eb0316f6987a1d98d2d9e9fa8d4e15b.tar.bz2 |
Avoid NPE when callers send null selection args.
Fixes http://b/2226007
Diffstat (limited to 'core/java/android/content/ContentProviderOperation.java')
-rw-r--r-- | core/java/android/content/ContentProviderOperation.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/core/java/android/content/ContentProviderOperation.java b/core/java/android/content/ContentProviderOperation.java index 0467516..ca36df2 100644 --- a/core/java/android/content/ContentProviderOperation.java +++ b/core/java/android/content/ContentProviderOperation.java @@ -545,8 +545,12 @@ public class ContentProviderOperation implements Parcelable { "only updates, deletes, and asserts can have selections"); } mSelection = selection; - mSelectionArgs = new String[selectionArgs.length]; - System.arraycopy(selectionArgs, 0, mSelectionArgs, 0, selectionArgs.length); + if (selectionArgs == null) { + mSelectionArgs = null; + } else { + mSelectionArgs = new String[selectionArgs.length]; + System.arraycopy(selectionArgs, 0, mSelectionArgs, 0, selectionArgs.length); + } return this; } |