From: David Zafman <david.zafman@inktank.com>
Date: Wed, 16 Apr 2014 21:02:13 +0000 (-0700)
Subject: common/obj_bencher: Fix error return check from read that is negative on error
X-Git-Tag: v0.80-rc1~36^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=a3d452acdf2fcf9ad10002c5f24c2548d12952bd;p=ceph.git

common/obj_bencher: Fix error return check from read that is negative on error

Fixed read return value in d99f1d9f68db41231e0ffff4082b05d6d095c231

Fixes: #8100

Signed-off-by: David Zafman <david.zafman@inktank.com>
---

diff --git a/src/common/obj_bencher.cc b/src/common/obj_bencher.cc
index 70675995930dc..887b21b5ba22c 100644
--- a/src/common/obj_bencher.cc
+++ b/src/common/obj_bencher.cc
@@ -584,7 +584,7 @@ int ObjBencher::seq_read_bench(int seconds_to_run, int num_objects, int concurre
     completion_wait(slot);
     lock.Lock();
     r = completion_ret(slot);
-    if (r != 0) {
+    if (r < 0) {
       cerr << "read got " << r << std::endl;
       lock.Unlock();
       goto ERR;
@@ -627,7 +627,7 @@ int ObjBencher::seq_read_bench(int seconds_to_run, int num_objects, int concurre
     completion_wait(slot);
     lock.Lock();
     r = completion_ret(slot);
-    if (r != 0) {
+    if (r < 0) {
       cerr << "read got " << r << std::endl;
       lock.Unlock();
       goto ERR;
@@ -771,7 +771,7 @@ int ObjBencher::rand_read_bench(int seconds_to_run, int num_objects, int concurr
     completion_wait(slot);
     lock.Lock();
     r = completion_ret(slot);
-    if (r != 0) {
+    if (r < 0) {
       cerr << "read got " << r << std::endl;
       lock.Unlock();
       goto ERR;
@@ -814,7 +814,7 @@ int ObjBencher::rand_read_bench(int seconds_to_run, int num_objects, int concurr
     completion_wait(slot);
     lock.Lock();
     r = completion_ret(slot);
-    if (r != 0) {
+    if (r < 0) {
       cerr << "read got " << r << std::endl;
       lock.Unlock();
       goto ERR;