]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
test_rgw_admin_log.cc: refactor to use calloc()
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Thu, 9 Oct 2014 15:14:37 +0000 (17:14 +0200)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Thu, 9 Oct 2014 15:14:37 +0000 (17:14 +0200)
Refactor code to use calloc() instead of malloc() to make
sure the char* TEST_BUCKET_OBJECT_SIZE memory is set to
zero before call put_bucket_obj().

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/test/test_rgw_admin_log.cc

index 2049b254e6cdfaf40e168c2d41d4785f189fe5ab..7ffea978d78be8b381b6888bde73620538c1f179 100644 (file)
@@ -741,7 +741,7 @@ TEST(TestRGWAdmin, datalog_list) {
   sleep(1);
   ASSERT_EQ(0, create_bucket());
   
-  char *bucket_obj = (char *)malloc(TEST_BUCKET_OBJECT_SIZE);
+  char *bucket_obj = (char *)calloc(1, TEST_BUCKET_OBJECT_SIZE);
   ASSERT_TRUE(bucket_obj != NULL);
   EXPECT_EQ(put_bucket_obj(TEST_BUCKET_OBJECT, bucket_obj, TEST_BUCKET_OBJECT_SIZE), 0);
   sleep(1); 
@@ -980,7 +980,7 @@ TEST(TestRGWAdmin, datalog_trim) {
   
   ASSERT_EQ(0, create_bucket());
 
-  char *bucket_obj = (char *)malloc(TEST_BUCKET_OBJECT_SIZE);
+  char *bucket_obj = (char *)calloc(1, TEST_BUCKET_OBJECT_SIZE);
   ASSERT_TRUE(bucket_obj != NULL);
   EXPECT_EQ(put_bucket_obj(TEST_BUCKET_OBJECT, bucket_obj, TEST_BUCKET_OBJECT_SIZE), 0);
   ASSERT_EQ(0, delete_obj(TEST_BUCKET_OBJECT));
@@ -1398,7 +1398,7 @@ TEST(TestRGWAdmin, bilog_list) {
 
   ASSERT_EQ(0, create_bucket());
 
-  char *bucket_obj = (char *)malloc(TEST_BUCKET_OBJECT_SIZE);
+  char *bucket_obj = (char *)calloc(1, TEST_BUCKET_OBJECT_SIZE);
   ASSERT_TRUE(bucket_obj != NULL);
   EXPECT_EQ(put_bucket_obj(TEST_BUCKET_OBJECT, bucket_obj, TEST_BUCKET_OBJECT_SIZE), 0);
   free(bucket_obj);
@@ -1428,7 +1428,7 @@ TEST(TestRGWAdmin, bilog_list) {
   get_bilog_list(entries);
   EXPECT_EQ(2U, entries.size());
 
-  bucket_obj = (char *)malloc(TEST_BUCKET_OBJECT_SIZE);
+  bucket_obj = (char *)calloc(1, TEST_BUCKET_OBJECT_SIZE);
   ASSERT_TRUE(bucket_obj != NULL);
   EXPECT_EQ(put_bucket_obj(TEST_BUCKET_OBJECT_1, bucket_obj, TEST_BUCKET_OBJECT_SIZE), 0);
   free(bucket_obj);
@@ -1539,7 +1539,7 @@ TEST(TestRGWAdmin, bilog_trim) {
   g_test->send_request(string("DELETE"), rest_req);
   EXPECT_EQ(400U, g_test->get_resp_code()); /*Bad request*/
 
-  char *bucket_obj = (char *)malloc(TEST_BUCKET_OBJECT_SIZE);
+  char *bucket_obj = (char *)calloc(1, TEST_BUCKET_OBJECT_SIZE);
   ASSERT_TRUE(bucket_obj != NULL);
   EXPECT_EQ(put_bucket_obj(TEST_BUCKET_OBJECT, bucket_obj, TEST_BUCKET_OBJECT_SIZE), 0);
   free(bucket_obj);