Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6a20784e authored by Mike Lockwood's avatar Mike Lockwood Committed by Android Git Automerger
Browse files

am afd1c014: Merge "SqliteDatabase: Fix local reference leak in... 

am afd1c014: Merge "SqliteDatabase: Fix local reference leak in custom_function_callback()" into honeycomb-mr1

* commit 'afd1c014':
  SqliteDatabase: Fix local reference leak in custom_function_callback()
parents fbe9ca72 afd1c014

core/jni/android_database_SQLiteDatabase.cpp

+8 −3
Original line number Diff line number Diff line
@@ -64,7 +64,7 @@ enum { 


static jfieldID offset_db_handle;

static jmethodID method_custom_function_callback;

static jclass string_class = NULL;

static jclass string_class;

static jint sSqliteSoftHeapLimit = 0;



static char *createStr(const char *path, short extra) {
@@ -406,8 +406,6 @@ static void custom_function_callback(sqlite3_context * context, int argc, sqlite 
    jobject function = (jobject)sqlite3_user_data(context);



    // pack up the arguments into a string array

    if (!string_class)

        string_class = (jclass)env->NewGlobalRef(env->FindClass("java/lang/String"));

    jobjectArray strArray = env->NewObjectArray(argc, string_class, NULL);

    if (!strArray)

        goto done;
@@ -425,6 +423,7 @@ static void custom_function_callback(sqlite3_context * context, int argc, sqlite 
    }



    env->CallVoidMethod(function, method_custom_function_callback, strArray);

    env->DeleteLocalRef(strArray);



done:

    if (env->ExceptionCheck()) {
@@ -489,6 +488,12 @@ int register_android_database_SQLiteDatabase(JNIEnv *env) 
        return -1;

    }



    string_class = (jclass)env->NewGlobalRef(env->FindClass("java/lang/String"));

    if (string_class == NULL) {

        LOGE("Can't find java/lang/String\n");

        return -1;

    }



    offset_db_handle = env->GetFieldID(clazz, "mNativeHandle", "I");

    if (offset_db_handle == NULL) {

        LOGE("Can't find SQLiteDatabase.mNativeHandle\n");