rgw: add check for update return value

Signed-off-by: Orit Wasserman <owasserm@redhat.com>
(cherry picked from commit 125955e0625461065dc4755b900e51c3598cadb4)

diff --git a/src/rgw/rgw_rados.cc b/src/rgw/rgw_rados.cc
index 4291a1361bacfdd9c6aa7ed23829b668c506edac..c677e9b759b7ac23fcd038a6a63205b7eb26a706 100644 (file)
--- a/src/rgw/rgw_rados.cc
+++ b/src/rgw/rgw_rados.cc
@@ -3718,7 +3718,11 @@ int RGWRados::init_zg_from_period(bool *initialized)
          master->second.name << " id:" << master->second.id << " as master" << dendl;
        if (zonegroup.get_id() == zg.get_id()) {
          zonegroup.master_zone = master->second.id;
-         zonegroup.update();
+         ret = zonegroup.update();
+         if (ret < 0) {
+           ldout(cct, 0) << "error updating zonegroup : " << cpp_strerror(-ret) << dendl;
+           return ret;
+         }
        } else {
          RGWZoneGroup fixed_zg(zg.get_id(),zg.get_name());
          ret = fixed_zg.init(cct, this);
@@ -3727,7 +3731,11 @@ int RGWRados::init_zg_from_period(bool *initialized)
            return ret;
          }
          fixed_zg.master_zone = master->second.id;
-         fixed_zg.update();
+         ret = fixed_zg.update();
+         if (ret < 0) {
+           ldout(cct, 0) << "error initializing zonegroup : " << cpp_strerror(-ret) << dendl;
+           return ret;
+         }
        }
       } else {
        ldout(cct, 0) << "zonegroup " << zg.get_name() << " missing zone for master_zone=" <<
@@ -3781,7 +3789,11 @@ int RGWRados::init_zg_from_local(bool *creating_defaults)
        ldout(cct, 0) << "zonegroup " << zonegroup.get_name() << " missing master_zone, setting zone " <<
          master->second.name << " id:" << master->second.id << " as master" << dendl;
        zonegroup.master_zone = master->second.id;
-       zonegroup.update();
+       ret = zonegroup.update();
+       if (ret < 0) {
+         ldout(cct, 0) << "error initializing zonegroup : " << cpp_strerror(-ret) << dendl;
+         return ret;
+       }
       } else {
        ldout(cct, 0) << "zonegroup " << zonegroup.get_name() << " missing zone for "
           "master_zone=" << zonegroup.master_zone << dendl;