X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=webcit%2Fcrypto.c;h=8792a341e0780c3515dcfe201e2ca41f32c8b444;hb=fb6f6fa4ec4e3277e30d84326d48e6850822d318;hp=6c270ad5b75bad57bd667163778014946933b27b;hpb=b9af9a3fafd99a3fb81cb09f01390531ffe6f29d;p=citadel.git diff --git a/webcit/crypto.c b/webcit/crypto.c index 6c270ad5b..8792a341e 100644 --- a/webcit/crypto.c +++ b/webcit/crypto.c @@ -1,34 +1,31 @@ -/* $Id$ */ +/* + * Copyright (c) 1996-2012 by the citadel.org team + * + * This program is open source software. You can redistribute it and/or + * modify it under the terms of the GNU General Public License, version 3. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ +#include "sysdep.h" #ifdef HAVE_OPENSSL - -#include -#include -#include -#include -#include -#include - -#include #include "webcit.h" #include "webserver.h" -#define CTDL_CRYPTO_DIR "./keys" -#define CTDL_KEY_PATH CTDL_CRYPTO_DIR "/citadel.key" -#define CTDL_CSR_PATH CTDL_CRYPTO_DIR "/citadel.csr" -#define CTDL_CER_PATH CTDL_CRYPTO_DIR "/citadel.cer" -#define SIGN_DAYS 365 - - -/* Shared Diffie-Hellman parameters */ -#define DH_P "1A74527AEE4EE2568E85D4FB2E65E18C9394B9C80C42507D7A6A0DBE9A9A54B05A9A96800C34C7AA5297095B69C88901EEFD127F969DCA26A54C0E0B5C5473EBAEB00957D2633ECAE3835775425DE66C0DE6D024DBB17445E06E6B0C78415E589B8814F08531D02FD43778451E7685541079CFFB79EF0D26EFEEBBB69D1E80383" -#define DH_G "2" -#define DH_L 1024 -#define CIT_CIPHERS "ALL:RC4+RSA:+SSLv2:@STRENGTH" /* see ciphers(1) */ +/* where to find the keys */ +#define CTDL_CRYPTO_DIR ctdl_key_dir +#define CTDL_KEY_PATH file_crpt_file_key +#define CTDL_CSR_PATH file_crpt_file_csr +#define CTDL_CER_PATH file_crpt_file_cer +#define SIGN_DAYS 3650 /* how long our certificate should live */ SSL_CTX *ssl_ctx; /* SSL context */ pthread_mutex_t **SSLCritters; /* Things needing locking */ +char *ssl_cipher_list = DEFAULT_SSL_CIPHER_LIST; pthread_key_t ThreadSSL; /* Per-thread SSL context */ @@ -37,48 +34,25 @@ static unsigned long id_callback(void) return (unsigned long) pthread_self(); } - /* - * Set up the cert things on the server side. We do need both the - * private key (in key_file) and the cert (in cert_file). - * Both files may be identical. - * - * This function is taken from OpenSSL apps/s_cb.c - */ - -static int ctdl_install_certificate(SSL_CTX * ctx, - const char *cert_file, const char *key_file) +void shutdown_ssl(void) { - if (cert_file != NULL) { - if (SSL_CTX_use_certificate_file(ctx, cert_file, - SSL_FILETYPE_PEM) <= 0) { - lprintf(3, "unable to get certificate from '%s'", - cert_file); - return (0); - } - if (key_file == NULL) - key_file = cert_file; - if (SSL_CTX_use_PrivateKey_file(ctx, key_file, - SSL_FILETYPE_PEM) <= 0) { - lprintf(3, "unable to get private key from '%s'", - key_file); - return (0); - } - /* Now we know that a key and cert have been set against - * the SSL context */ - if (!SSL_CTX_check_private_key(ctx)) { - lprintf(3, - "Private key does not match the certificate public key"); - return (0); - } - } - return (1); + ERR_free_strings(); + + /* Openssl requires these while shutdown. + * Didn't find a way to get out of this clean. + * int i, n = CRYPTO_num_locks(); + * for (i = 0; i < n; i++) + * free(SSLCritters[i]); + * free(SSLCritters); + */ } - +/* + * initialize ssl engine, load certs and initialize openssl internals + */ void init_ssl(void) { - SSL_METHOD *ssl_method; - DH *dh; + const SSL_METHOD *ssl_method; RSA *rsa=NULL; X509_REQ *req = NULL; X509 *cer = NULL; @@ -86,31 +60,34 @@ void init_ssl(void) EVP_PKEY *req_pkey = NULL; X509_NAME *name = NULL; FILE *fp; + char buf[SIZ]; + int rv = 0; - if (!access("/var/run/egd-pool", F_OK)) + if (!access("/var/run/egd-pool", F_OK)) { RAND_egd("/var/run/egd-pool"); + } if (!RAND_status()) { - lprintf(3, - "PRNG not adequately seeded, won't do SSL/TLS\n"); + syslog(3, "PRNG not adequately seeded, won't do SSL/TLS\n"); return; } - SSLCritters = - malloc(CRYPTO_num_locks() * sizeof(pthread_mutex_t *)); + SSLCritters = malloc(CRYPTO_num_locks() * sizeof(pthread_mutex_t *)); if (!SSLCritters) { - lprintf(1, "citserver: can't allocate memory!!\n"); + syslog(1, "citserver: can't allocate memory!!\n"); /* Nothing's been initialized, just die */ - exit(1); + ShutDownWebcit(); + exit(WC_EXIT_SSL); } else { int a; for (a = 0; a < CRYPTO_num_locks(); a++) { SSLCritters[a] = malloc(sizeof(pthread_mutex_t)); if (!SSLCritters[a]) { - lprintf(1, + syslog(1, "citserver: can't allocate memory!!\n"); - /* Nothing's been initialized, just die */ - exit(1); + /** Nothing's been initialized, just die */ + ShutDownWebcit(); + exit(WC_EXIT_SSL); } pthread_mutex_init(SSLCritters[a], NULL); } @@ -123,70 +100,57 @@ void init_ssl(void) SSL_load_error_strings(); ssl_method = SSLv23_server_method(); if (!(ssl_ctx = SSL_CTX_new(ssl_method))) { - lprintf(3, "SSL_CTX_new failed: %s\n", - ERR_reason_error_string(ERR_get_error())); + syslog(3, "SSL_CTX_new failed: %s\n", ERR_reason_error_string(ERR_get_error())); return; } - if (!(SSL_CTX_set_cipher_list(ssl_ctx, CIT_CIPHERS))) { - lprintf(3, "SSL: No ciphers available\n"); - SSL_CTX_free(ssl_ctx); - ssl_ctx = NULL; + + syslog(9, "Requesting cipher list: %s\n", ssl_cipher_list); + if (!(SSL_CTX_set_cipher_list(ssl_ctx, ssl_cipher_list))) { + syslog(3, "SSL_CTX_set_cipher_list failed: %s\n", ERR_reason_error_string(ERR_get_error())); return; } -#if 0 -#if SSLEAY_VERSION_NUMBER >= 0x00906000L - SSL_CTX_set_mode(ssl_ctx, SSL_CTX_get_mode(ssl_ctx) | - SSL_MODE_AUTO_RETRY); -#endif -#endif CRYPTO_set_locking_callback(ssl_lock); CRYPTO_set_id_callback(id_callback); - /* Load DH parameters into the context */ - dh = DH_new(); - if (!dh) { - lprintf(3, "init_ssl() can't allocate a DH object: %s\n", - ERR_reason_error_string(ERR_get_error())); - SSL_CTX_free(ssl_ctx); - ssl_ctx = NULL; - return; - } - if (!(BN_hex2bn(&(dh->p), DH_P))) { - lprintf(3, "init_ssl() can't assign DH_P: %s\n", - ERR_reason_error_string(ERR_get_error())); - SSL_CTX_free(ssl_ctx); - ssl_ctx = NULL; - return; - } - if (!(BN_hex2bn(&(dh->g), DH_G))) { - lprintf(3, "init_ssl() can't assign DH_G: %s\n", - ERR_reason_error_string(ERR_get_error())); - SSL_CTX_free(ssl_ctx); - ssl_ctx = NULL; - return; - } - dh->length = DH_L; - SSL_CTX_set_tmp_dh(ssl_ctx, dh); - DH_free(dh); - - /* Get our certificates in order. + /* + * Get our certificates in order. (FIXME: dirify. this is a setup job.) * First, create the key/cert directory if it's not there already... */ mkdir(CTDL_CRYPTO_DIR, 0700); /* - * Generate a key pair if we don't have one. + * Before attempting to generate keys/certificates, first try + * link to them from the Citadel server if it's on the same host. + * We ignore any error return because it either meant that there + * was nothing in Citadel to link from (in which case we just + * generate new files) or the target files already exist (which + * is not fatal either). + */ + if (!strcasecmp(ctdlhost, "uds")) { + sprintf(buf, "%s/keys/citadel.key", ctdlport); + rv = symlink(buf, CTDL_KEY_PATH); + if (!rv) syslog(1, "%s\n", strerror(errno)); + sprintf(buf, "%s/keys/citadel.csr", ctdlport); + rv = symlink(buf, CTDL_CSR_PATH); + if (!rv) syslog(1, "%s\n", strerror(errno)); + sprintf(buf, "%s/keys/citadel.cer", ctdlport); + rv = symlink(buf, CTDL_CER_PATH); + if (!rv) syslog(1, "%s\n", strerror(errno)); + } + + /* + * If we still don't have a private key, generate one. */ if (access(CTDL_KEY_PATH, R_OK) != 0) { - lprintf(5, "Generating RSA key pair.\n"); + syslog(5, "Generating RSA key pair.\n"); rsa = RSA_generate_key(1024, /* modulus size */ 65537, /* exponent */ NULL, /* no callback */ - NULL); /* no callback */ + NULL /* no callback */ + ); if (rsa == NULL) { - lprintf(3, "Key generation failed: %s\n", - ERR_reason_error_string(ERR_get_error())); + syslog(3, "Key generation failed: %s\n", ERR_reason_error_string(ERR_get_error())); } if (rsa != NULL) { fp = fopen(CTDL_KEY_PATH, "w"); @@ -200,21 +164,28 @@ void init_ssl(void) NULL, /* no callbk */ NULL /* no callbk */ ) != 1) { - lprintf(3, "Cannot write key: %s\n", - ERR_reason_error_string(ERR_get_error())); + syslog(3, "Cannot write key: %s\n", + ERR_reason_error_string(ERR_get_error())); unlink(CTDL_KEY_PATH); } fclose(fp); } + else { + syslog(3, "Cannot write key: %s\n", CTDL_KEY_PATH); + ShutDownWebcit(); + exit(0); + } RSA_free(rsa); } } /* - * Generate a CSR if we don't have one. + * If there is no certificate file on disk, we will be generating a self-signed certificate + * in the next step. Therefore, if we have neither a CSR nor a certificate, generate + * the CSR in this step so that the next step may commence. */ - if (access(CTDL_CSR_PATH, R_OK) != 0) { - lprintf(5, "Generating a certificate signing request.\n"); + if ( (access(CTDL_CER_PATH, R_OK) != 0) && (access(CTDL_CSR_PATH, R_OK) != 0) ) { + syslog(5, "Generating a certificate signing request.\n"); /* * Read our key from the file. No, we don't just keep this @@ -230,11 +201,11 @@ void init_ssl(void) if (rsa) { - /* Create a public key from the private key */ + /** Create a public key from the private key */ if (pk=EVP_PKEY_new(), pk != NULL) { EVP_PKEY_assign_RSA(pk, rsa); if (req = X509_REQ_new(), req != NULL) { - + const char *env; /* Set the public key */ X509_REQ_set_pubkey(req, pk); X509_REQ_set_version(req, 0L); @@ -244,30 +215,58 @@ void init_ssl(void) /* Tell it who we are */ /* + * We used to add these fields to the subject, but + * now we don't. Someone doing this for real isn't + * going to use the webcit-generated CSR anyway. + * X509_NAME_add_entry_by_txt(name, "C", MBSTRING_ASC, "US", -1, -1, 0); - + * X509_NAME_add_entry_by_txt(name, "ST", MBSTRING_ASC, "New York", -1, -1, 0); - + * X509_NAME_add_entry_by_txt(name, "L", MBSTRING_ASC, "Mount Kisco", -1, -1, 0); */ - X509_NAME_add_entry_by_txt(name, "O", - MBSTRING_ASC, "FIXME.FIXME.org", -1, -1, 0); - - X509_NAME_add_entry_by_txt(name, "OU", - MBSTRING_ASC, "Citadel server", -1, -1, 0); - - X509_NAME_add_entry_by_txt(name, "CN", - MBSTRING_ASC, "FIXME.FIXME.org", -1, -1, 0); + env = getenv("O"); + if (env == NULL) + env = "Organization name", + + X509_NAME_add_entry_by_txt( + name, "O", + MBSTRING_ASC, + (unsigned char*)env, + -1, -1, 0 + ); + + env = getenv("OU"); + if (env == NULL) + env = "Citadel server"; + + X509_NAME_add_entry_by_txt( + name, "OU", + MBSTRING_ASC, + (unsigned char*)env, + -1, -1, 0 + ); + + env = getenv("CN"); + if (env == NULL) + env = "*"; + + X509_NAME_add_entry_by_txt( + name, "CN", + MBSTRING_ASC, + (unsigned char*)env, + -1, -1, 0 + ); X509_REQ_set_subject_name(req, name); /* Sign the CSR */ if (!X509_REQ_sign(req, pk, EVP_md5())) { - lprintf(3, "X509_REQ_sign(): error\n"); + syslog(3, "X509_REQ_sign(): error\n"); } else { /* Write it to disk. */ @@ -277,6 +276,11 @@ void init_ssl(void) PEM_write_X509_REQ(fp, req); fclose(fp); } + else { + syslog(3, "Cannot write key: %s\n", CTDL_CSR_PATH); + ShutDownWebcit(); + exit(0); + } } X509_REQ_free(req); @@ -287,7 +291,7 @@ void init_ssl(void) } else { - lprintf(3, "Unable to read private key.\n"); + syslog(3, "Unable to read private key.\n"); } } @@ -297,7 +301,7 @@ void init_ssl(void) * Generate a self-signed certificate if we don't have one. */ if (access(CTDL_CER_PATH, R_OK) != 0) { - lprintf(5, "Generating a self-signed certificate.\n"); + syslog(5, "Generating a self-signed certificate.\n"); /* Same deal as before: always read the key from disk because * it may or may not have just been generated. @@ -326,17 +330,19 @@ void init_ssl(void) if (req) { if (cer = X509_new(), cer != NULL) { + ASN1_INTEGER_set(X509_get_serialNumber(cer), 0); X509_set_issuer_name(cer, req->req_info->subject); X509_set_subject_name(cer, req->req_info->subject); - X509_gmtime_adj(X509_get_notBefore(cer),0); + X509_gmtime_adj(X509_get_notBefore(cer), 0); X509_gmtime_adj(X509_get_notAfter(cer),(long)60*60*24*SIGN_DAYS); + req_pkey = X509_REQ_get_pubkey(req); X509_set_pubkey(cer, req_pkey); EVP_PKEY_free(req_pkey); /* Sign the cert */ if (!X509_sign(cer, pk, EVP_md5())) { - lprintf(3, "X509_sign(): error\n"); + syslog(3, "X509_sign(): error\n"); } else { /* Write it to disk. */ @@ -346,6 +352,11 @@ void init_ssl(void) PEM_write_X509(fp, cer); fclose(fp); } + else { + syslog(3, "Cannot write key: %s\n", CTDL_CER_PATH); + ShutDownWebcit(); + exit(0); + } } X509_free(cer); } @@ -355,26 +366,26 @@ void init_ssl(void) } } - /* * Now try to bind to the key and certificate. + * Note that we use SSL_CTX_use_certificate_chain_file() which allows + * the certificate file to contain intermediate certificates. */ - if (ctdl_install_certificate(ssl_ctx, - CTDL_CER_PATH, - CTDL_KEY_PATH) != 1) - { - lprintf(3, "Cannot install certificate: %s\n", + SSL_CTX_use_certificate_chain_file(ssl_ctx, CTDL_CER_PATH); + SSL_CTX_use_PrivateKey_file(ssl_ctx, CTDL_KEY_PATH, SSL_FILETYPE_PEM); + if ( !SSL_CTX_check_private_key(ssl_ctx) ) { + syslog(3, "Cannot install certificate: %s\n", ERR_reason_error_string(ERR_get_error())); } - + } /* - * starttls() starts SSL/TLS encryption for the current session. + * starts SSL/TLS encryption for the current session. */ int starttls(int sock) { - int retval, bits, alg_bits; + int retval, bits, alg_bits;/*r; */ SSL *newssl; pthread_setspecific(ThreadSSL, NULL); @@ -383,13 +394,11 @@ int starttls(int sock) { return(1); } if (!(newssl = SSL_new(ssl_ctx))) { - lprintf(3, "SSL_new failed: %s\n", - ERR_reason_error_string(ERR_get_error())); + syslog(3, "SSL_new failed: %s\n", ERR_reason_error_string(ERR_get_error())); return(2); } if (!(SSL_set_fd(newssl, sock))) { - lprintf(3, "SSL_set_fd failed: %s\n", - ERR_reason_error_string(ERR_get_error())); + syslog(3, "SSL_set_fd failed: %s\n", ERR_reason_error_string(ERR_get_error())); SSL_free(newssl); return(3); } @@ -401,88 +410,131 @@ int starttls(int sock) { * revert to unencrypted communications. */ long errval; + const char *ssl_error_reason = NULL; + + errval = SSL_get_error(newssl, retval); + ssl_error_reason = ERR_reason_error_string(ERR_get_error()); + if (ssl_error_reason == NULL) { + syslog(3, "SSL_accept failed: errval=%ld, retval=%d %s\n", errval, retval, strerror(errval)); + } + else { + syslog(3, "SSL_accept failed: %s\n", ssl_error_reason); + } + sleeeeeeeeeep(1); + retval = SSL_accept(newssl); + } + if (retval < 1) { + long errval; + const char *ssl_error_reason = NULL; errval = SSL_get_error(newssl, retval); - lprintf(3, "SSL_accept failed: %s\n", - ERR_reason_error_string(ERR_get_error())); + ssl_error_reason = ERR_reason_error_string(ERR_get_error()); + if (ssl_error_reason == NULL) { + syslog(3, "SSL_accept failed: errval=%ld, retval=%d (%s)\n", errval, retval, strerror(errval)); + } + else { + syslog(3, "SSL_accept failed: %s\n", ssl_error_reason); + } SSL_free(newssl); newssl = NULL; return(4); } - BIO_set_close(newssl->rbio, BIO_NOCLOSE); - bits = - SSL_CIPHER_get_bits(SSL_get_current_cipher(newssl), - &alg_bits); - lprintf(5, "SSL/TLS using %s on %s (%d of %d bits)\n", + else { + syslog(15, "SSL_accept success\n"); + } + /*r = */BIO_set_close(newssl->rbio, BIO_NOCLOSE); + bits = SSL_CIPHER_get_bits(SSL_get_current_cipher(newssl), &alg_bits); + syslog(15, "SSL/TLS using %s on %s (%d of %d bits)\n", SSL_CIPHER_get_name(SSL_get_current_cipher(newssl)), SSL_CIPHER_get_version(SSL_get_current_cipher(newssl)), bits, alg_bits); pthread_setspecific(ThreadSSL, newssl); + syslog(15, "SSL started\n"); return(0); } /* - * endtls() shuts down the TLS connection + * shuts down the TLS connection * * WARNING: This may make your session vulnerable to a known plaintext * attack in the current implmentation. */ void endtls(void) { - lprintf(5, "Ending SSL/TLS\n"); + /*SSL_CTX *ctx;*/ + + if (THREADSSL == NULL) return; + + syslog(15, "Ending SSL/TLS\n"); SSL_shutdown(THREADSSL); + /*ctx = */SSL_get_SSL_CTX(THREADSSL); + + /* I don't think this is needed, and it crashes the server anyway + * + * if (ctx != NULL) { + * syslog(9, "Freeing CTX at %x\n", (int)ctx ); + * SSL_CTX_free(ctx); + * } + */ + SSL_free(THREADSSL); pthread_setspecific(ThreadSSL, NULL); } /* - * ssl_lock() callback for OpenSSL mutex locks + * callback for OpenSSL mutex locks */ void ssl_lock(int mode, int n, const char *file, int line) { - if (mode & CRYPTO_LOCK) + if (mode & CRYPTO_LOCK) { pthread_mutex_lock(SSLCritters[n]); - else + } + else { pthread_mutex_unlock(SSLCritters[n]); + } } /* - * client_write_ssl() Send binary data to the client encrypted. + * Send binary data to the client encrypted. */ -void client_write_ssl(char *buf, int nbytes) +void client_write_ssl(const StrBuf *Buf) { + const char *buf; int retval; int nremain; + long nbytes; char junk[1]; - nremain = nbytes; + if (THREADSSL == NULL) return; + + nbytes = nremain = StrLength(Buf); + buf = ChrPtr(Buf); while (nremain > 0) { if (SSL_want_write(THREADSSL)) { if ((SSL_read(THREADSSL, junk, 0)) < 1) { - lprintf(9, "SSL_read in client_write: %s\n", ERR_reason_error_string(ERR_get_error())); + syslog(9, "SSL_read in client_write: %s\n", + ERR_reason_error_string(ERR_get_error())); } } - retval = - SSL_write(THREADSSL, &buf[nbytes - nremain], nremain); + retval = SSL_write(THREADSSL, &buf[nbytes - nremain], nremain); if (retval < 1) { long errval; errval = SSL_get_error(THREADSSL, retval); - if (errval == SSL_ERROR_WANT_READ || - errval == SSL_ERROR_WANT_WRITE) { - sleep(1); + if (errval == SSL_ERROR_WANT_READ || errval == SSL_ERROR_WANT_WRITE) { + sleeeeeeeeeep(1); continue; } - lprintf(9, "SSL_write got error %ld, ret %d\n", errval, retval); - if (retval == -1) - lprintf(9, "errno is %d\n", errno); + syslog(9, "SSL_write got error %ld, ret %d\n", errval, retval); + if (retval == -1) { + syslog(9, "errno is %d\n", errno); + } endtls(); - client_write(&buf[nbytes - nremain], nremain); return; } nremain -= retval; @@ -491,63 +543,40 @@ void client_write_ssl(char *buf, int nbytes) /* - * client_read_ssl() - read data from the encrypted layer. + * read data from the encrypted layer. */ -int client_read_ssl(char *buf, int bytes, int timeout) +int client_read_sslbuffer(StrBuf *buf, int timeout) { -#if 0 - fd_set rfds; - struct timeval tv; - int retval; - int s; -#endif - int len, rlen; + char sbuf[16384]; /* OpenSSL communicates in 16k blocks, so let's speak its native tongue. */ + int rlen; char junk[1]; + SSL *pssl = THREADSSL; - len = 0; - while (len < bytes) { -#if 0 - /* - * This code is disabled because we don't need it when - * using blocking reads (which we are). -IO - */ - FD_ZERO(&rfds); - s = BIO_get_fd(THREADSSL->rbio, NULL); - FD_SET(s, &rfds); - tv.tv_sec = timeout; - tv.tv_usec = 0; - - retval = select(s + 1, &rfds, NULL, NULL, &tv); - - if (FD_ISSET(s, &rfds) == 0) { - return (0); - } + if (pssl == NULL) return(-1); -#endif - if (SSL_want_read(THREADSSL)) { - if ((SSL_write(THREADSSL, junk, 0)) < 1) { - lprintf(9, "SSL_write in client_read: %s\n", ERR_reason_error_string(ERR_get_error())); + while (1) { + if (SSL_want_read(pssl)) { + if ((SSL_write(pssl, junk, 0)) < 1) { + syslog(9, "SSL_write in client_read\n"); } } - rlen = SSL_read(THREADSSL, &buf[len], bytes - len); + rlen = SSL_read(pssl, sbuf, sizeof(sbuf)); if (rlen < 1) { long errval; - errval = SSL_get_error(THREADSSL, rlen); - if (errval == SSL_ERROR_WANT_READ || - errval == SSL_ERROR_WANT_WRITE) { - sleep(1); + errval = SSL_get_error(pssl, rlen); + if (errval == SSL_ERROR_WANT_READ || errval == SSL_ERROR_WANT_WRITE) { + sleeeeeeeeeep(1); continue; } - lprintf(9, "SSL_read got error %ld\n", errval); + syslog(9, "SSL_read got error %ld\n", errval); endtls(); - return (client_read_to - (&buf[len], bytes - len, timeout)); + return (-1); } - len += rlen; + StrBufAppendBufPlain(buf, sbuf, rlen, 0); + return rlen; } - return (1); + return (0); } - #endif /* HAVE_OPENSSL */