diff options
| author | Dianne Hackborn <hackbod@google.com> | 2013-03-22 18:42:14 -0700 |
|---|---|---|
| committer | Dianne Hackborn <hackbod@google.com> | 2013-03-22 18:42:14 -0700 |
| commit | 9fa39bd255f2bc248941ae5924a0a70cc25de19e (patch) | |
| tree | 6facf9c8c5e151edf6d50cfe5db60d87cb15bca9 /core/java/android | |
| parent | 1a89c5324badd10dac142a5a0c40a203503db65f (diff) | |
| download | frameworks_base-9fa39bd255f2bc248941ae5924a0a70cc25de19e.zip frameworks_base-9fa39bd255f2bc248941ae5924a0a70cc25de19e.tar.gz frameworks_base-9fa39bd255f2bc248941ae5924a0a70cc25de19e.tar.bz2 | |
App ops: don't crash when provide read access is off.
If the caller supplied an empty selection string (instead
of null) we would crash due to creating a bad SQLite statement.
Change-Id: I462803b80c81815ed9a3a320c23060daa28e8114
Diffstat (limited to 'core/java/android')
| -rw-r--r-- | core/java/android/content/ContentProvider.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/java/android/content/ContentProvider.java b/core/java/android/content/ContentProvider.java index 4968268..cf627d7 100644 --- a/core/java/android/content/ContentProvider.java +++ b/core/java/android/content/ContentProvider.java @@ -609,7 +609,7 @@ public abstract class ContentProvider implements ComponentCallbacks2 { // selection statement with a dummy one that will always be false. // This way we will get a cursor back that has the correct structure // but contains no rows. - if (selection == null) { + if (selection == null || selection.isEmpty()) { selection = "'A' = 'B'"; } else { selection = "'A' = 'B' AND (" + selection + ")"; |