diff options
| author | Allen Byrne <byrn@hdfgroup.org> | 2018-08-05 16:12:39 (GMT) |
|---|---|---|
| committer | Allen Byrne <byrn@hdfgroup.org> | 2018-08-06 13:14:04 (GMT) |
| commit | 762c14fde587f2ff4f9b9cdaa6b2232a481edde4 (patch) | |
| tree | d375060fdb22395e121c3f4d3b07e940ff634640 /java/src/jni/h5aImp.c | |
| parent | 51d31c5ff89f4cf9f1d29d994e5ab6594c3dbd1d (diff) | |
| download | hdf5-762c14fde587f2ff4f9b9cdaa6b2232a481edde4.zip hdf5-762c14fde587f2ff4f9b9cdaa6b2232a481edde4.tar.gz hdf5-762c14fde587f2ff4f9b9cdaa6b2232a481edde4.tar.bz2 | |
Improve error handling of exceptions
Diffstat (limited to 'java/src/jni/h5aImp.c')
| -rw-r--r-- | java/src/jni/h5aImp.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/java/src/jni/h5aImp.c b/java/src/jni/h5aImp.c index 13f5207..437f25f 100644 --- a/java/src/jni/h5aImp.c +++ b/java/src/jni/h5aImp.c @@ -966,7 +966,7 @@ Java_hdf_hdf5lib_H5_H5Aget_1info_1by_1idx UNPIN_JAVA_STRING(obj_name, aName); if (status < 0) { - h5libraryError(env); + h5libraryError(env); } /* end if */ else { args[0].z = ainfo.corder_valid; @@ -1002,7 +1002,7 @@ Java_hdf_hdf5lib_H5_H5Aget_1info_1by_1name UNPIN_JAVA_STRING_TWO(obj_name, aName, attr_name, attrName); if (status < 0) { - h5libraryError(env); + h5libraryError(env); } /* end if */ else { args[0].z = ainfo.corder_valid; @@ -1166,8 +1166,12 @@ H5A_iterate_cb constructor = CBENVPTR->GetMethodID(CBENVPAR cls, "<init>", "(ZJIJ)V"); if (constructor != 0) { cb_info_t = CBENVPTR->NewObjectA(CBENVPAR cls, constructor, args); - - status = CBENVPTR->CallIntMethod(CBENVPAR visit_callback, mid, g_id, str, cb_info_t, op_data); + if (cb_info_t == NULL) { + printf("FATAL ERROR: Creation failed\n"); + } + else { + status = CBENVPTR->CallIntMethod(CBENVPAR visit_callback, mid, g_id, str, cb_info_t, op_data); + } } /* end if (constructor != 0) */ } /* end if (cls != 0) */ } /* end if (mid != 0) */ |