From f7bbf0e282f10f649cb087be85269b5f7acce6ea Mon Sep 17 00:00:00 2001 From: Peter Serwylo Date: Mon, 25 Jan 2016 18:02:17 +1100 Subject: [PATCH] Use `getApplicationContext()` instead of `getContext()` to prevent memory leaks. Using whatever `Activity` as the `Context` used to construct the first content provider means that it will be help onto in memory until the application is GC'ed. --- F-Droid/src/org/fdroid/fdroid/data/FDroidProvider.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/F-Droid/src/org/fdroid/fdroid/data/FDroidProvider.java b/F-Droid/src/org/fdroid/fdroid/data/FDroidProvider.java index 792e78555..42936570d 100644 --- a/F-Droid/src/org/fdroid/fdroid/data/FDroidProvider.java +++ b/F-Droid/src/org/fdroid/fdroid/data/FDroidProvider.java @@ -86,7 +86,7 @@ public abstract class FDroidProvider extends ContentProvider { } protected final synchronized SQLiteDatabase db() { - return getOrCreateDb(getContext()).getWritableDatabase(); + return getOrCreateDb(getContext().getApplicationContext()).getWritableDatabase(); } @Override