diff options
author | Jean-Baptiste Queru <jbq@google.com> | 2010-12-28 10:08:11 -0800 |
---|---|---|
committer | Android Code Review <code-review@android.com> | 2010-12-28 10:08:11 -0800 |
commit | 06a4ce4c149ca42dd084fe9cdd9b5aaa8bc8a2c6 (patch) | |
tree | 22ad2395dd098de5cc93f01d71c7b9c9198cfdf5 | |
parent | c6a607b5a0fbb99641390e67464cfe72c3836be3 (diff) | |
parent | 1bf26926cc3c48e66f5ac68f5a4f229e54680206 (diff) | |
download | Gallery3D-06a4ce4c149ca42dd084fe9cdd9b5aaa8bc8a2c6.tar.gz |
Merge "Make the get/deletePhoto queries locale safe."
-rw-r--r-- | src/com/cooliris/media/PhotoAppWidgetProvider.java | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/com/cooliris/media/PhotoAppWidgetProvider.java b/src/com/cooliris/media/PhotoAppWidgetProvider.java index cca6413..4c0a8d9 100644 --- a/src/com/cooliris/media/PhotoAppWidgetProvider.java +++ b/src/com/cooliris/media/PhotoAppWidgetProvider.java @@ -155,8 +155,9 @@ public class PhotoAppWidgetProvider extends AppWidgetProvider { Bitmap bitmap = null; try { SQLiteDatabase db = getReadableDatabase(); - String selection = String.format("%s=%d", FIELD_APPWIDGET_ID, appWidgetId); - c = db.query(TABLE_PHOTOS, PHOTOS_PROJECTION, selection, null, null, null, null, null); + String[] selectionArgs = { String.valueOf(appWidgetId) }; + c = db.query(TABLE_PHOTOS, PHOTOS_PROJECTION, FIELD_APPWIDGET_ID + "=?", + selectionArgs, null, null, null, null); if (c != null && LOGD) { Log.d(TAG, "getPhoto query count=" + c.getCount()); @@ -184,8 +185,8 @@ public class PhotoAppWidgetProvider extends AppWidgetProvider { public void deletePhoto(int appWidgetId) { try { SQLiteDatabase db = getWritableDatabase(); - String whereClause = String.format("%s=%d", FIELD_APPWIDGET_ID, appWidgetId); - db.delete(TABLE_PHOTOS, whereClause, null); + String[] selectionArgs = { String.valueOf(appWidgetId) }; + db.delete(TABLE_PHOTOS, FIELD_APPWIDGET_ID + "=?", selectionArgs); } catch (SQLiteException e) { Log.e(TAG, "Could not delete photo from database", e); } |