diff options
| author | Joseph Hunkeler <jhunkeler@gmail.com> | 2024-03-18 15:22:13 -0400 |
|---|---|---|
| committer | Joseph Hunkeler <jhunkeler@gmail.com> | 2024-03-18 15:22:13 -0400 |
| commit | 1d3f68c9ccc381460c6509d446124fb50f3f8d45 (patch) | |
| tree | ae5dc643508b34cf800420b6fb38b9b16c53e956 /src | |
| parent | 9b234cdb4b52a61bb00586e65c5ba59544e0b1b5 (diff) | |
| download | stasis-1d3f68c9ccc381460c6509d446124fb50f3f8d45.tar.gz | |
Add missing cleanup
* Closes dangling file streams scattered throughout
* Frees pointers related to Delivery.docker missed during implementation
Diffstat (limited to 'src')
| -rw-r--r-- | src/deliverable.c | 5 | ||||
| -rw-r--r-- | src/utils.c | 2 |
2 files changed, 5 insertions, 2 deletions
diff --git a/src/deliverable.c b/src/deliverable.c index 0b166d9..17007e5 100644 --- a/src/deliverable.c +++ b/src/deliverable.c @@ -136,6 +136,7 @@ void delivery_free(struct Delivery *ctx) { guard_free(ctx->storage.build_sources_dir); guard_free(ctx->storage.build_testing_dir); guard_free(ctx->storage.mission_dir); + guard_free(ctx->storage.docker_artifact_dir); guard_free(ctx->info.time_str_epoch); guard_free(ctx->info.build_name); guard_free(ctx->info.build_number); @@ -169,6 +170,7 @@ void delivery_free(struct Delivery *ctx) { guard_free(ctx->rules.build_number_fmt); ini_free(&ctx->rules._handle); + guard_free(ctx->deploy.docker.test_script); guard_free(ctx->deploy.docker.registry); guard_free(ctx->deploy.docker.image_compression); guard_strlist_free(ctx->deploy.docker.tags); @@ -883,7 +885,7 @@ struct StrList *delivery_build_wheels(struct Delivery *ctx) { strlist_append(result, rec->d_name); } } - + closedir(dp); } popd(); } @@ -1094,6 +1096,7 @@ int delivery_index_wheel_artifacts(struct Delivery *ctx) { sprintf(path_src, "%s/%s", ctx->storage.wheel_artifact_dir, rec->d_name); rename(path_src, path_dest); } + closedir(dp); return 0; } diff --git a/src/utils.c b/src/utils.c index 85a3b56..941923a 100644 --- a/src/utils.c +++ b/src/utils.c @@ -484,7 +484,7 @@ int isempty_dir(const char *path) { } count++; } - + closedir(dp); return count == 0; } |