diff options
| author | Egor Tensin <Egor.Tensin@gmail.com> | 2023-07-07 15:44:52 +0200 |
|---|---|---|
| committer | Egor Tensin <Egor.Tensin@gmail.com> | 2023-07-07 15:53:37 +0200 |
| commit | 6fb2fd3dac7316f7155437d350ab372ed248b866 (patch) | |
| tree | 40f4e7cbdfb03e2357e782e4f551c5f93ad54751 | |
| parent | sqlite: minor refactoring (diff) | |
| download | cimple-6fb2fd3dac7316f7155437d350ab372ed248b866.tar.gz cimple-6fb2fd3dac7316f7155437d350ab372ed248b866.zip | |
server: fix a possible leak
| -rw-r--r-- | src/server.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/server.c b/src/server.c index 2fbb13c..da9cfc3 100644 --- a/src/server.c +++ b/src/server.c @@ -13,6 +13,7 @@ #include "file.h" #include "log.h" #include "msg.h" +#include "net.h" #include "process.h" #include "protocol.h" #include "run_queue.h" @@ -235,11 +236,12 @@ static int server_handle_cmd_new_worker(UNUSED const struct msg *request, if (ret < 0) return ret; + const int fd = ret; struct worker *worker = NULL; - ret = worker_create(&worker, ret); + ret = worker_create(&worker, fd); if (ret < 0) - return ret; + goto close; ret = server_enqueue_worker(server, worker); if (ret < 0) @@ -250,6 +252,9 @@ static int server_handle_cmd_new_worker(UNUSED const struct msg *request, destroy_worker: worker_destroy(worker); +close: + net_close(fd); + return ret; } |