godotengine/godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #5641 from sanikoyes/Pr-fix-android-crash

Browse Source 
Fix crash in FileAccessJAndroid::file_exists (does not free local ref)
This commit is contained in:
Rémi Verschelde 2016-07-12 07:26:53 +02:00 committed by GitHub
commit f82e7d3a97
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
platform/android/file_access_jandroid.cpp
View File

@ -182,8 +182,10 @@ bool FileAccessJAndroid::file_exists(const String& p_path) {
jstring js = env->NewStringUTF(path.utf8().get_data()); jstring js = env->NewStringUTF(path.utf8().get_data());
int res = env->CallIntMethod(io,_file_open,js,false); int res = env->CallIntMethod(io,_file_open,js,false);
if (res<=0) if (res<=0) {
env->DeleteLocalRef(js);
return false; return false;
}
env->CallVoidMethod(io,_file_close,res); env->CallVoidMethod(io,_file_close,res);
env->DeleteLocalRef(js); env->DeleteLocalRef(js);
return true; return true;