kiwix/java-libkiwix
1
0
Fork 0
mirror of https://github.com/kiwix/java-libkiwix.git synced 2025-08-04 03:26:08 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #116 from kiwix/fix_native_crash_on_android

Browse Source 
Fixed the native crash happening on Android.
This commit is contained in:
MohitMaliFtechiz 2025-01-27 17:35:15 +05:30 committed by GitHub
commit 719a3106c4
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/src/main/cpp/macros.h
View File

@ -53,6 +53,9 @@ catch(const zim::ZimFileFormatError& e) { \
} catch(const zim::EntryNotFound& e) { \ } catch(const zim::EntryNotFound& e) { \
throwException(env, "org/kiwix/libzim/EntryNotFoundException", e.what()); \ throwException(env, "org/kiwix/libzim/EntryNotFoundException", e.what()); \
return RET; \ return RET; \
} catch (const NativeHandleDisposedException& e) { \
throwException(env, "java/lang/IllegalStateException", e.what()); \
return RET; \
} catch (const std::ios_base::failure& e) { \ } catch (const std::ios_base::failure& e) { \
throwException(env, "java/io/IOException", e.what()); \ throwException(env, "java/io/IOException", e.what()); \
return RET; \ return RET; \

8
lib/src/main/cpp/utils.h
View File

@ -96,6 +96,11 @@ inline void setHandle(JNIEnv* env, jobject thisObj, Args && ...args)
} }
#define SET_HANDLE(NATIVE_TYPE, OBJ, VALUE) setHandle<NATIVE_TYPE>(env, OBJ, VALUE) #define SET_HANDLE(NATIVE_TYPE, OBJ, VALUE) setHandle<NATIVE_TYPE>(env, OBJ, VALUE)
class NativeHandleDisposedException : public std::runtime_error {
public:
explicit NativeHandleDisposedException(const std::string& message)
: std::runtime_error(message) {}
};
// Return a shared_ptr for the handle // Return a shared_ptr for the handle
template<typename T> template<typename T>
@ -104,6 +109,9 @@ shared_ptr<T> getPtr(JNIEnv* env, jobject thisObj, const char* handleName = "nat
jclass thisClass = env->GetObjectClass(thisObj); jclass thisClass = env->GetObjectClass(thisObj);
jfieldID fidNumber = env->GetFieldID(thisClass, handleName, "J"); jfieldID fidNumber = env->GetFieldID(thisClass, handleName, "J");
auto handle = reinterpret_cast<shared_ptr<T>*>(env->GetLongField(thisObj, fidNumber)); auto handle = reinterpret_cast<shared_ptr<T>*>(env->GetLongField(thisObj, fidNumber));
if (handle == nullptr) {
throw NativeHandleDisposedException("The native object has already been disposed");
}
return *handle; return *handle;
} }
#define GET_PTR(NATIVE_TYPE) getPtr<NATIVE_TYPE>(env, thisObj) #define GET_PTR(NATIVE_TYPE) getPtr<NATIVE_TYPE>(env, thisObj)