From: Brad Hubbard <bhubbard@redhat.com>
Date: Sun, 25 Dec 2016 05:29:52 +0000 (+1000)
Subject: test/librados/c_read_operations.cc: Fix valgrind errors
X-Git-Tag: v12.0.0~365^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=8c90a2d2075e7188957843fac14d2501d2837752;p=ceph-ci.git

test/librados/c_read_operations.cc: Fix valgrind errors

Fixes: http://tracker.ceph.com/issues/18354

Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
---

diff --git a/src/test/librados/c_read_operations.cc b/src/test/librados/c_read_operations.cc
index 7fab5cf4e99..1b1d440a860 100644
--- a/src/test/librados/c_read_operations.cc
+++ b/src/test/librados/c_read_operations.cc
@@ -51,8 +51,8 @@ protected:
 
     const char *zeros[len];
     size_t zero_lens[len];
-    memset(zeros, 0, len);
-    memset(zero_lens, 0, len * sizeof(size_t));
+    memset(zeros, 0, sizeof(zeros));
+    memset(zero_lens, 0, sizeof(zero_lens));
     compare_omap_vals(keys, vals, lens, len, iter_vals);
     compare_omap_vals(keys, zeros, zero_lens, len, iter_keys);
     compare_omap_vals(keys, vals, lens, len, iter_vals_by_key);
@@ -159,6 +159,7 @@ TEST_F(CReadOpsTest, AssertExists) {
   ASSERT_EQ(0, rados_aio_read_op_operate(op, ioctx, completion, obj, 0));
   rados_aio_wait_for_complete(completion);
   ASSERT_EQ(-ENOENT, rados_aio_get_return_value(completion));
+  rados_aio_release(completion);
   rados_release_read_op(op);
 
   write_object();