mirror of
https://github.com/openstreetmap/mod_tile.git
synced 2025-08-05 18:51:43 +00:00
Fix some minor memory leaks in the testing frame work.
There are still a number of memory leaks in the queueing tests
This commit is contained in:
@ -156,6 +156,7 @@ TEST_CASE( "renderd/queueing", "request queueing") {
|
||||
struct item *item2 = request_queue_fetch_request(queue);
|
||||
|
||||
REQUIRE( item == item2 );
|
||||
free(item2);
|
||||
|
||||
request_queue_close(queue);
|
||||
}
|
||||
@ -188,6 +189,11 @@ TEST_CASE( "renderd/queueing", "request queueing") {
|
||||
item2 = request_queue_fetch_request(queue);
|
||||
REQUIRE( itemB == item2 );
|
||||
|
||||
free(itemR);
|
||||
free(itemB);
|
||||
free(itemD);
|
||||
free(itemRP);
|
||||
|
||||
request_queue_close(queue);
|
||||
}
|
||||
|
||||
@ -407,14 +413,19 @@ TEST_CASE( "renderd/queueing", "request queueing") {
|
||||
REQUIRE (item->fd == 1);
|
||||
item = request_queue_fetch_request(queue);
|
||||
REQUIRE (item->fd == FD_INVALID);
|
||||
free(item);
|
||||
item = request_queue_fetch_request(queue);
|
||||
REQUIRE (item->fd == 3);
|
||||
free(item);
|
||||
item = request_queue_fetch_request(queue);
|
||||
REQUIRE (item->fd == FD_INVALID);
|
||||
free(item);
|
||||
item = request_queue_fetch_request(queue);
|
||||
REQUIRE (item->fd == 5);
|
||||
free(item);
|
||||
item = request_queue_fetch_request(queue);
|
||||
REQUIRE (item->fd == 6);
|
||||
free(item);
|
||||
|
||||
request_queue_close(queue);
|
||||
}
|
||||
@ -532,8 +543,6 @@ TEST_CASE( "storage-backend", "Tile storage backend" ) {
|
||||
tiles.save(store);
|
||||
|
||||
store->close_storage(store);
|
||||
|
||||
|
||||
}
|
||||
|
||||
SECTION("storage/stat/full metatile", "should complete") {
|
||||
@ -568,8 +577,7 @@ TEST_CASE( "storage-backend", "Tile storage backend" ) {
|
||||
}
|
||||
|
||||
store->close_storage(store);
|
||||
|
||||
|
||||
free(tile_dir);
|
||||
}
|
||||
|
||||
SECTION("storage/read/full metatile", "should complete") {
|
||||
@ -609,6 +617,8 @@ TEST_CASE( "storage-backend", "Tile storage backend" ) {
|
||||
}
|
||||
}
|
||||
|
||||
free(buf);
|
||||
free(buf_tmp);
|
||||
store->close_storage(store);
|
||||
|
||||
|
||||
@ -655,6 +665,8 @@ TEST_CASE( "storage-backend", "Tile storage backend" ) {
|
||||
}
|
||||
}
|
||||
|
||||
free(buf);
|
||||
free(buf_tmp);
|
||||
store->close_storage(store);
|
||||
}
|
||||
|
||||
@ -694,7 +706,8 @@ TEST_CASE( "storage-backend", "Tile storage backend" ) {
|
||||
|
||||
REQUIRE ( sinfo.size < 0 );
|
||||
|
||||
|
||||
free(buf);
|
||||
free(buf_tmp);
|
||||
store->close_storage(store);
|
||||
}
|
||||
|
||||
@ -735,7 +748,8 @@ TEST_CASE( "storage-backend", "Tile storage backend" ) {
|
||||
REQUIRE ( sinfo.size > 0 );
|
||||
REQUIRE ( sinfo.expired > 0 );
|
||||
|
||||
|
||||
free(buf);
|
||||
free(buf_tmp);
|
||||
store->close_storage(store);
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user