]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
test/librados_test_stub: fixed cls_cxx_map_get_keys/vals return value 14484/head
authorJason Dillaman <dillaman@redhat.com>
Wed, 12 Apr 2017 14:47:28 +0000 (10:47 -0400)
committerJason Dillaman <dillaman@redhat.com>
Wed, 12 Apr 2017 14:49:05 +0000 (10:49 -0400)
Fixes: http://tracker.ceph.com/issues/19597
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
src/test/librados_test_stub/LibradosTestStub.cc

index bd69ca234a7d938df75fd295edf5e32d0e0b25d0..dcbc8e25f7d0838600fbba5e4d8ec1855f0b263f 100644 (file)
@@ -1130,7 +1130,7 @@ int cls_cxx_map_get_keys(cls_method_context_t hctx, const string &start_obj,
       keys->insert(last_key);
     }
   } while (!vals.empty());
-  return 0;
+  return keys->size();
 }
 
 int cls_cxx_map_get_val(cls_method_context_t hctx, const string &key,
@@ -1158,8 +1158,12 @@ int cls_cxx_map_get_vals(cls_method_context_t hctx, const string &start_obj,
                          std::map<string, bufferlist> *vals) {
   librados::TestClassHandler::MethodContext *ctx =
     reinterpret_cast<librados::TestClassHandler::MethodContext*>(hctx);
-  return ctx->io_ctx_impl->omap_get_vals(ctx->oid, start_obj, filter_prefix,
-      max_to_get, vals);
+  int r = ctx->io_ctx_impl->omap_get_vals(ctx->oid, start_obj, filter_prefix,
+                                         max_to_get, vals);
+  if (r < 0) {
+    return r;
+  }
+  return vals->size();
 }
 
 int cls_cxx_map_remove_key(cls_method_context_t hctx, const string &key) {