Skip to content

Commit c43c85f

Browse files
committed
test_rgw_admin_log.cc: fix use-after-free
Fix for: CID 1054876 (#1 of 1): Use after free (USE_AFTER_FREE) pass_freed_arg: Passing freed pointer bucket_obj as an argument to put_bucket_obj Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
1 parent c60a170 commit c43c85f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/test/test_rgw_admin_log.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -744,7 +744,6 @@ TEST(TestRGWAdmin, datalog_list) {
744744
char *bucket_obj = (char *)malloc(TEST_BUCKET_OBJECT_SIZE);
745745
ASSERT_TRUE(bucket_obj != NULL);
746746
EXPECT_EQ(put_bucket_obj(TEST_BUCKET_OBJECT, bucket_obj, TEST_BUCKET_OBJECT_SIZE), 0);
747-
free(bucket_obj);
748747
sleep(1);
749748
ss << "/admin/log?type=data&id=" << shard_id << "&start-time=" << start_time;
750749
rest_req = ss.str();
@@ -777,6 +776,7 @@ TEST(TestRGWAdmin, datalog_list) {
777776

778777
sleep(1);
779778
EXPECT_EQ(put_bucket_obj(TEST_BUCKET_OBJECT, bucket_obj, TEST_BUCKET_OBJECT_SIZE), 0);
779+
free(bucket_obj);
780780
sleep(20);
781781
ss.str("");
782782
ss << "/admin/log?type=data&id=" << shard_id << "&start-time=" << start_time;

0 commit comments

Comments
 (0)