Merged this to master; thanks! -Greg On Fri, Jan 13, 2012 at 3:58 AM, Andrey Stepachev <octo@xxxxxxxxxxxxxx> wrote: > In case of nonexistent file, calling Client::replication() > triggers assert. > > Signed-off-by: Andrey Stepachev <octo@xxxxxxxxxxxxxx> > --- > src/client/hadoop/CephFSInterface.cc | 5 ++++- > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/src/client/hadoop/CephFSInterface.cc b/src/client/hadoop/CephFSInterface.cc > index 4a050f2..5ffca64 100644 > --- a/src/client/hadoop/CephFSInterface.cc > +++ b/src/client/hadoop/CephFSInterface.cc > @@ -750,9 +750,12 @@ JNIEXPORT jint JNICALL Java_org_apache_hadoop_fs_ceph_CephTalker_ceph_1replicati > ceph_mount_info *cmount = get_ceph_mount_t(env, obj); > int fh = 0; > fh = ceph_open(cmount, c_path, O_RDONLY, 0); > + env->ReleaseStringUTFChars(j_path, c_path); > + if (fh < 0) { > + return fh; > + } > int replication = ceph_get_file_replication(cmount, fh); > ceph_close(cmount, fh); > - env->ReleaseStringUTFChars(j_path, c_path); > return replication; > } > > -- > 1.7.5.4 > > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html