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
