From: Boris Ranto <branto@redhat.com>
Date: Sat, 6 Feb 2016 01:00:22 +0000 (+0100)
Subject: rgw/rgw_orphan: check the return value of save_state
X-Git-Tag: v10.1.0~413^2^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=a48c081d203c5cc10e36546b2d1c0a429ba3480f;p=ceph-ci.git

rgw/rgw_orphan: check the return value of save_state

It was discovered by covscan that we do not check the return value of
save_state in this one case.

Signed-off-by: Boris Ranto <branto@redhat.com>
---

diff --git a/src/rgw/rgw_orphan.cc b/src/rgw/rgw_orphan.cc
index f59e1f75db4..861b89c84d2 100644
--- a/src/rgw/rgw_orphan.cc
+++ b/src/rgw/rgw_orphan.cc
@@ -578,7 +578,11 @@ int RGWOrphanSearch::build_linked_oids_index()
     return ret;
   }
 
-  save_state();
+  ret = save_state();
+  if (ret < 0) {
+    cerr << __func__ << ": ERROR: failed to write state ret=" << ret << std::endl;
+    return ret;
+  }
 
   return 0;
 }