diff options
| author | defanor <defanor@uberspace.net> | 2023-10-22 13:46:03 +0300 |
|---|---|---|
| committer | defanor <defanor@uberspace.net> | 2023-10-22 13:46:03 +0300 |
| commit | 2a5c3db244708d528f357adeb7e6ec947161fbad (patch) | |
| tree | 5ff275e4677e1be481540ae7daf1e66e6338e4b8 /src/rexmpp_tls.c | |
| parent | 205fb31965b51b650b40bb3f0e54ec0cfdead36a (diff) | |
Fix a couple of bugs, refactor a little
Found a couple of issues with -fanalyzer, though
CWE-401 (analyzer-malloc-leak) appears to produce false positives
still.
Diffstat (limited to 'src/rexmpp_tls.c')
| -rw-r--r-- | src/rexmpp_tls.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/rexmpp_tls.c b/src/rexmpp_tls.c index e483a2c..2a7c903 100644 --- a/src/rexmpp_tls.c +++ b/src/rexmpp_tls.c @@ -582,8 +582,8 @@ rexmpp_tls_send (rexmpp_t *s, size_t data_size, ssize_t *written) { -#if defined(USE_GNUTLS) *written = -1; +#if defined(USE_GNUTLS) ssize_t ret = gnutls_record_send(tls_ctx->gnutls_session, data, data_size); @@ -597,7 +597,6 @@ rexmpp_tls_send (rexmpp_t *s, return REXMPP_TLS_E_OTHER; } #elif defined(USE_OPENSSL) - *written = -1; int ret = SSL_write_ex(tls_ctx->openssl_conn, data, data_size, (size_t*)written); if (ret > 0) { @@ -608,7 +607,6 @@ rexmpp_tls_send (rexmpp_t *s, #else (void)data; (void)data_size; - (void)written; (void)tls_ctx; rexmpp_log(s, LOG_ERR, "rexmpp is compiled without TLS support"); return REXMPP_TLS_E_OTHER; |