Merge "[DO NOT MERGE] Throw exception if slot has invalid offset" into klp-dev am: ac87aee354
am: a821245d70
Change-Id: I1e47c963728906cb2283a353c882017368d07747
This commit is contained in:
@@ -179,6 +179,10 @@ static jbyteArray nativeGetBlob(JNIEnv* env, jclass clazz, jlong windowPtr,
|
|||||||
if (type == CursorWindow::FIELD_TYPE_BLOB || type == CursorWindow::FIELD_TYPE_STRING) {
|
if (type == CursorWindow::FIELD_TYPE_BLOB || type == CursorWindow::FIELD_TYPE_STRING) {
|
||||||
size_t size;
|
size_t size;
|
||||||
const void* value = window->getFieldSlotValueBlob(fieldSlot, &size);
|
const void* value = window->getFieldSlotValueBlob(fieldSlot, &size);
|
||||||
|
if (!value) {
|
||||||
|
throw_sqlite3_exception(env, "Native could not read blob slot");
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
jbyteArray byteArray = env->NewByteArray(size);
|
jbyteArray byteArray = env->NewByteArray(size);
|
||||||
if (!byteArray) {
|
if (!byteArray) {
|
||||||
env->ExceptionClear();
|
env->ExceptionClear();
|
||||||
@@ -214,6 +218,10 @@ static jstring nativeGetString(JNIEnv* env, jclass clazz, jlong windowPtr,
|
|||||||
if (type == CursorWindow::FIELD_TYPE_STRING) {
|
if (type == CursorWindow::FIELD_TYPE_STRING) {
|
||||||
size_t sizeIncludingNull;
|
size_t sizeIncludingNull;
|
||||||
const char* value = window->getFieldSlotValueString(fieldSlot, &sizeIncludingNull);
|
const char* value = window->getFieldSlotValueString(fieldSlot, &sizeIncludingNull);
|
||||||
|
if (!value) {
|
||||||
|
throw_sqlite3_exception(env, "Native could not read string slot");
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
if (sizeIncludingNull <= 1) {
|
if (sizeIncludingNull <= 1) {
|
||||||
return gEmptyString;
|
return gEmptyString;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user