Skip to content

Commit

Permalink
Merge pull request #12713 from badone/wip-ceph_test_rados_api_watch_n…
Browse files Browse the repository at this point in the history
…otify-valgrind-fixes

test_rados_watch_notify: Fix trivial memory leaks

Reviewed-by: Kefu Chai <kchai@redhat.com>
  • Loading branch information
tchaikov committed Jan 3, 2017
2 parents 38f8bc3 + 08be44a commit d50ddd3
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/test/librados/watch_notify.cc
Expand Up @@ -362,6 +362,7 @@ TEST_F(LibRadosWatchNotify, Watch2Timeout) {
ASSERT_EQ(1u, notify_cookies.size());
ASSERT_GT(rados_watch_check(ioctx, handle), 0);

rados_buffer_free(reply_buf);
rados_unwatch2(ioctx, handle);
}

Expand Down Expand Up @@ -732,6 +733,7 @@ TEST_F(LibRadosWatchNotify, WatchNotify2Timeout) {
ASSERT_EQ(0, rados_aio_wait_for_complete(comp));
ASSERT_EQ(0, rados_aio_get_return_value(comp));
rados_aio_release(comp);
rados_buffer_free(reply_buf);

}

Expand Down

0 comments on commit d50ddd3

Please sign in to comment.