Merge pull request 'Fix file descriptor leakage' (#35 ) from lda/Cytoplasm:tls-file-leak into master

Reviewed-on: #35
[MOD] As it turns out, you do need to free thr fd.
2024-06-07 14:00:44 -05:00 · 2024-06-03 19:42:09 +02:00
1 changed files with 0 additions and 2 deletions
--- a/src/Tls/TlsOpenSSL.c
+++ b/src/Tls/TlsOpenSSL.c
@ -295,9 +295,7 @@ TlsClose(void *cookie)
    SSL_free(ssl->ssl);
    SSL_CTX_free(ssl->ctx);

-#if 0
    close(ssl->fd);
-#endif

    Free(ssl);
Author	SHA1	Message	Date
Jordan Bancino	b9dc169917	Merge pull request 'Fix file descriptor leakage' (#35 ) from lda/Cytoplasm:tls-file-leak into master Some checks failed Compile Cytoplasm / Compile Cytoplasm (x86, alpine-v3.19) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86, debian-v12.4) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86, freebsd-v14.0) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86, netbsd-v9.3) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86_64, alpine-v3.19) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86_64, debian-v12.4) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86_64, freebsd-v14.0) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86_64, netbsd-v9.3) (push) Has been cancelled Details Compile Cytoplasm / Compile Cytoplasm (x86_64, openbsd-v7.4) (push) Has been cancelled Details Reviewed-on: #35	2024-06-07 14:00:44 -05:00
lda	d983296dd2	[MOD] As it turns out, you do need to free thr fd.	2024-06-03 19:42:09 +02:00