From 3f59f02913f4277f419052d546bb7448f74aec4d Mon Sep 17 00:00:00 2001 From: David Zafman Date: Wed, 5 Mar 2014 15:08:41 -0800 Subject: [PATCH] Revert "c_read_operations.cc: fix resource leak" This reverts commit 3cd751b0a280909510c3e633cc8cd4b9f5e3b2d9. A rados_release_read_op() has already been performed, but coverity didn't recognize that as releasing memory. Fixes: #7621 Signed-off-by: David Zafman Reviewed-by: Samuel Just --- src/test/librados/c_read_operations.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/src/test/librados/c_read_operations.cc b/src/test/librados/c_read_operations.cc index 053b6147ba5..106f6b4f162 100644 --- a/src/test/librados/c_read_operations.cc +++ b/src/test/librados/c_read_operations.cc @@ -365,7 +365,6 @@ TEST_F(CReadOpsTest, ExecUserBuf) { // buffer too short bytes_read = 1024; - rados_release_read_op(op); op = rados_create_read_op(); rados_read_op_exec_user_buf(op, "rbd", "get_all_features", NULL, 0, out, sizeof(features) - 1, &bytes_read, &rval);