X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=src%2Fconnection.c;h=5a08de7431f07733179982b48e3cf7bac8e4d737;hb=b7ce41b809a3b3533d398c0da8fd9787c149434c;hp=5c7f7ae600f7813f2691996811ee9d4231112fd8;hpb=ef57fa373cf6dfbbbc5cf21c144de008681635f4;p=tlsproxy%2Ftlsproxy.git diff --git a/src/connection.c b/src/connection.c index 5c7f7ae..5a08de7 100644 --- a/src/connection.c +++ b/src/connection.c @@ -1,7 +1,7 @@ /* * Handle connections. * - * Copyright (C) 2011-2012 Simon Ruderich + * Copyright (C) 2011-2013 Simon Ruderich * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -21,14 +21,12 @@ #include "connection.h" #include "verify.h" -/* close() */ -#include -/* getaddrinfo() */ +#include +#include +#include #include -/* poll() */ #include -/* errno */ -#include +#include /* Maximum length of a HTTP request line. Longer request lines are aborted @@ -39,11 +37,11 @@ /* Format string used to send HTTP/1.0 error responses to the client. * - * %s is used 4 times, first three are the error code (no %n$s!), the last is - * the message. */ + * %s is used 5 times, first is the error code, then additional headers, next + * two are the error code (no %n$s!), the last is the message. */ #define HTTP_RESPONSE_FORMAT "HTTP/1.0 %s\r\n\ Content-Type: text/html; charset=US-ASCII\r\n\ -\r\n\ +%s\r\n\ \n\ \n\ %s\n\ @@ -62,8 +60,10 @@ static int initialize_tls_session_server(int peer_socket, gnutls_session_t *session, gnutls_certificate_credentials_t *x509_cred); +static int fdopen_read_write(int socket, FILE **read_fd, FILE **write_fd); static int read_http_request(FILE *client_fd, char *request, size_t length); static void send_bad_request(FILE *client_fd); +static void send_authentication_required(FILE *client_fd); static void send_forwarding_failure(FILE *client_fd); static void tls_send_invalid_cert_message(gnutls_session_t session); @@ -83,13 +83,13 @@ static int parse_request(const char *buffer, char *host, char *port, void handle_connection(int client_socket) { int server_socket; - FILE *client_fd, *server_fd; + FILE *client_fd_read, *client_fd_write, *server_fd_read, *server_fd_write; char buffer[MAX_REQUEST_LINE]; char host[MAX_REQUEST_LINE]; char port[5 + 1]; - int version_minor; /* HTTP/1.x */ + int version_minor; /* x in HTTP/1.x */ int result; /* client_x509_cred is used when talking to the client (acting as a TSL @@ -106,94 +106,95 @@ void handle_connection(int client_socket) { * certificate. */ int validation_failed; - LOG(LOG_DEBUG, "new connection"); + LOG(DEBUG, "new connection"); server_socket = -1; - client_fd = NULL; - server_fd = NULL; + client_fd_read = NULL; + client_fd_write = NULL; + server_fd_read = NULL; + server_fd_write = NULL; client_session_init = 0; server_session_init = 0; client_session_started = 0; server_session_started = 0; validation_failed = 0; - client_fd = fdopen(client_socket, "a+"); - if (NULL == client_fd) { - LOG_PERROR(LOG_WARNING, "fdopen(): client failed"); + if (fdopen_read_write(client_socket, &client_fd_read, + &client_fd_write) != 0) { goto out; } /* Read request line (CONNECT ..) and headers (they are discarded). */ - result = read_http_request(client_fd, buffer, sizeof(buffer)); - if (-1 == result) { - /* Read error. */ - LOG(LOG_WARNING, "read_http_request(): client read error"); + result = read_http_request(client_fd_read, buffer, sizeof(buffer)); + if (result == -1) { + LOG(WARNING, "read_http_request(): client read error"); goto out; - } else if (-2 == result) { - /* EOF */ - LOG(LOG_WARNING, "read_http_request(): client EOF"); - send_bad_request(client_fd); + } else if (result == -2) { + LOG(WARNING, "read_http_request(): client EOF"); + send_bad_request(client_fd_write); + goto out; + } else if (result == -3) { + LOG(DEBUG, "read_http_request(): proxy authentication failed"); + send_authentication_required(client_fd_write); goto out; } - if (0 != parse_request(buffer, host, port, &version_minor)) { - LOG(LOG_WARNING, "bad request: %s", buffer); - send_bad_request(client_fd); + if (parse_request(buffer, host, port, &version_minor) != 0) { + LOG(WARNING, "bad request: %s", buffer); + send_bad_request(client_fd_write); goto out; } - LOG(LOG_DEBUG, "target: %s:%s (HTTP 1.%d)", host, port, version_minor); + LOG(DEBUG, "target: %s:%s (HTTP 1.%d)", host, port, version_minor); /* Connect to proxy server or directly to server. */ - if (NULL != global_proxy_host && NULL != global_proxy_port) { - LOG(LOG_DEBUG, "connecting to %s:%s", global_proxy_host, - global_proxy_port); + if (global_proxy_host != NULL && global_proxy_port != NULL) { + LOG(DEBUG, "connecting to %s:%s", global_proxy_host, + global_proxy_port); server_socket = connect_to_host(global_proxy_host, global_proxy_port); } else { - LOG(LOG_DEBUG, "connecting to %s:%s", host, port); + LOG(DEBUG, "connecting to %s:%s", host, port); server_socket = connect_to_host(host, port); } - if (-1 == server_socket) { - LOG(LOG_WARNING, "failed to connect to server"); - send_forwarding_failure(client_fd); + if (server_socket < 0) { + LOG(WARNING, "failed to connect to server"); + send_forwarding_failure(client_fd_write); goto out; } - server_fd = fdopen(server_socket, "a+"); - if (NULL == server_fd) { - LOG_PERROR(LOG_WARNING, "fdopen(): server failed"); - send_forwarding_failure(client_fd); + if (fdopen_read_write(server_socket, &server_fd_read, + &server_fd_write) != 0) { + send_forwarding_failure(client_fd_write); goto out; } /* Connect to proxy if requested (command line option). */ - if (NULL != global_proxy_host && NULL != global_proxy_port) { - fprintf(server_fd, "CONNECT %s:%s HTTP/1.0\r\n", host, port); - fprintf(server_fd, "\r\n"); + if (global_proxy_host != NULL && global_proxy_port != NULL) { + fprintf(server_fd_write, "CONNECT %s:%s HTTP/1.0\r\n", host, port); + fprintf(server_fd_write, "\r\n"); + fflush(server_fd_write); /* Read response line from proxy server. */ - result = read_http_request(server_fd, buffer, sizeof(buffer)); - if (-1 == result) { - /* Read error. */ - LOG(LOG_WARNING, "read_http_request(): proxy read error"); - send_forwarding_failure(client_fd); + result = read_http_request(server_fd_read, buffer, sizeof(buffer)); + if (result == -1) { + LOG(WARNING, "read_http_request(): proxy read error"); + send_forwarding_failure(client_fd_write); goto out; - } else if (-2 == result) { - /* EOF */ - LOG(LOG_WARNING, "read_http_request(): proxy EOF"); - send_forwarding_failure(client_fd); + } else if (result == -2) { + LOG(WARNING, "read_http_request(): proxy EOF"); + send_forwarding_failure(client_fd_write); goto out; } /* Check response of proxy server. */ - if (0 != strncmp(buffer, "HTTP/1.0 200", 12)) { - LOG(LOG_WARNING, "bad proxy response: %s", buffer); - send_forwarding_failure(client_fd); + if (strncmp(buffer, "HTTP/1.0 200", 12)) { + LOG(WARNING, "bad proxy response: %s", buffer); + send_forwarding_failure(client_fd_write); goto out; } } - LOG(LOG_DEBUG, "connection to server established"); + LOG(DEBUG, "connection to server established"); /* If the -u option is used and we don't know this hostname's server * certificate then just pass through the connection and let the client @@ -202,24 +203,24 @@ void handle_connection(int client_socket) { char path[TLSPROXY_MAX_PATH_LENGTH]; FILE *file = NULL; - if (-2 == server_certificate_file(&file, host, path, sizeof(path))) { + if (server_certificate_file(&file, host, path, sizeof(path)) == -2) { /* We've established a connection, tell the client. */ - fprintf(client_fd, "HTTP/1.0 200 Connection established\r\n"); - fprintf(client_fd, "\r\n"); - fflush(client_fd); + fprintf(client_fd_write, "HTTP/1.0 200 Connection established\r\n"); + fprintf(client_fd_write, "\r\n"); + fflush(client_fd_write); - LOG(LOG_DEBUG, "transferring data"); + LOG(DEBUG, "transferring data"); /* Proxy data between client and server until one side is done * (EOF or error). */ transfer_data(client_socket, server_socket); - LOG(LOG_DEBUG, "finished transferring data"); + LOG(DEBUG, "finished transferring data"); goto out; } /* server_certificate_file() may have opened the file, close it. */ - if (NULL != file) { + if (file != NULL) { fclose(file); } } @@ -227,31 +228,31 @@ void handle_connection(int client_socket) { /* Initialize TLS client credentials to talk to the server. */ result = initialize_tls_session_server(server_socket, &server_session, &server_x509_cred); - if (0 != result) { - LOG(LOG_WARNING, "initialize_tls_session_server() failed"); - send_forwarding_failure(client_fd); + if (result != 0) { + LOG(WARNING, "initialize_tls_session_server() failed"); + send_forwarding_failure(client_fd_write); goto out; } server_session_init = 1; - LOG(LOG_DEBUG, "starting server TLS handshake"); + LOG(DEBUG, "starting server TLS handshake"); /* Try to establish TLS handshake between us and server. */ result = gnutls_handshake(server_session); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_WARNING, "server TLS handshake failed: %s", - gnutls_strerror(result)); - send_forwarding_failure(client_fd); + if (result != GNUTLS_E_SUCCESS) { + LOG(WARNING, "server TLS handshake failed: %s", + gnutls_strerror(result)); + send_forwarding_failure(client_fd_write); goto out; } server_session_started = 1; - LOG(LOG_DEBUG, "server TLS handshake finished"); + LOG(DEBUG, "server TLS handshake finished"); /* Make sure the server certificate is valid and known. */ - if (0 != verify_tls_connection(server_session, host)) { - LOG(LOG_ERROR, "server certificate validation failed!"); - /* We send the error message over our TLS connection to the client, + if (verify_tls_connection(server_session, host) != 0) { + LOG(ERROR, "server certificate validation failed!"); + /* We'll send the error message over our TLS connection to the client, * but with an invalid certificate. No data is transfered from/to the * target server. */ validation_failed = 1; @@ -265,31 +266,31 @@ void handle_connection(int client_socket) { : host, &client_session, &client_x509_cred); - if (0 != result) { - LOG(LOG_WARNING, "initialize_tls_session_client() failed"); - send_forwarding_failure(client_fd); + if (result != 0) { + LOG(WARNING, "initialize_tls_session_client() failed"); + send_forwarding_failure(client_fd_write); goto out; } client_session_init = 1; /* We've established a connection, tell the client. */ - fprintf(client_fd, "HTTP/1.0 200 Connection established\r\n"); - fprintf(client_fd, "\r\n"); - fflush(client_fd); + fprintf(client_fd_write, "HTTP/1.0 200 Connection established\r\n"); + fprintf(client_fd_write, "\r\n"); + fflush(client_fd_write); - LOG(LOG_DEBUG, "starting client TLS handshake"); + LOG(DEBUG, "starting client TLS handshake"); /* Try to establish TLS handshake between client and us. */ result = gnutls_handshake(client_session); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_WARNING, "client TLS handshake failed: %s", - gnutls_strerror(result)); - send_forwarding_failure(client_fd); + if (result != GNUTLS_E_SUCCESS) { + LOG(WARNING, "client TLS handshake failed: %s", + gnutls_strerror(result)); + send_forwarding_failure(client_fd_write); goto out; } client_session_started = 1; - LOG(LOG_DEBUG, "client TLS handshake finished"); + LOG(DEBUG, "client TLS handshake finished"); /* Tell the client that the verification failed. Shouldn't be necessary as * the client should terminate the connection because he received the @@ -299,29 +300,29 @@ void handle_connection(int client_socket) { goto out; } - LOG(LOG_DEBUG, "transferring TLS data"); + LOG(DEBUG, "transferring TLS data"); /* Proxy data between client and server until one side is done (EOF or * error). */ transfer_data_tls(client_socket, server_socket, client_session, server_session); - LOG(LOG_DEBUG, "finished transferring TLS data"); + LOG(DEBUG, "finished transferring TLS data"); out: /* Close TLS sessions if necessary. Use GNUTLS_SHUT_RDWR so the data is * reliable transmitted. */ - if (0 != server_session_started) { + if (server_session_started) { gnutls_bye(server_session, GNUTLS_SHUT_RDWR); } - if (0 != client_session_started) { + if (client_session_started) { gnutls_bye(client_session, GNUTLS_SHUT_RDWR); } - if (0 != server_session_init) { + if (server_session_init) { gnutls_deinit(server_session); gnutls_certificate_free_credentials(server_x509_cred); } - if (0 != client_session_init) { + if (client_session_init) { gnutls_deinit(client_session); gnutls_certificate_free_cas(client_x509_cred); gnutls_certificate_free_keys(client_x509_cred); @@ -329,21 +330,27 @@ out: } /* Close connection to server/proxy. */ - if (NULL != server_fd) { - fclose(server_fd); - } else if (-1 != server_socket) { + if (server_fd_read != NULL) { + if (server_fd_write != NULL) { + fclose(server_fd_write); + } + fclose(server_fd_read); + } else if (server_socket != -1) { close(server_socket); } - LOG(LOG_DEBUG, "connection to server closed"); + LOG(DEBUG, "connection to server closed"); /* Close connection to client. */ - if (NULL != client_fd) { - fclose(client_fd); + if (client_fd_read != NULL) { + if (client_fd_write != NULL) { + fclose(client_fd_write); + } + fclose(client_fd_read); } else { close(client_socket); } - LOG(LOG_DEBUG, "connection to client closed"); + LOG(DEBUG, "connection to client closed"); - LOG(LOG_DEBUG, "connection finished"); + LOG(DEBUG, "connection finished"); } @@ -357,20 +364,20 @@ static int initialize_tls_session_client(int peer_socket, /* The "invalid" hostname is special. If it's used we send an invalid * certificate to let the client know something is wrong. */ - use_invalid_cert = (0 == strcmp(hostname, "invalid")); + use_invalid_cert = (!strcmp(hostname, "invalid")); - if (0 != proxy_certificate_path(hostname, path, sizeof(path))) { - LOG(LOG_ERROR, - "initialize_tls_session_client(): \ -failed to get proxy certificate path"); + if (proxy_certificate_path(hostname, path, sizeof(path)) != 0) { + LOG(ERROR, + "initialize_tls_session_client(): " + "failed to get proxy certificate path"); return -1; } result = gnutls_certificate_allocate_credentials(x509_cred); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, - "initialize_tls_session_client(): \ -gnutls_certificate_allocate_credentials(): %s", + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, + "initialize_tls_session_client(): " + "gnutls_certificate_allocate_credentials(): %s", gnutls_strerror(result)); return -1; } @@ -380,8 +387,8 @@ gnutls_certificate_allocate_credentials(): %s", result = gnutls_certificate_set_x509_trust_file(*x509_cred, PROXY_CA_FILE, GNUTLS_X509_FMT_PEM); - if (0 >= result) { - LOG(LOG_ERROR, + if (result <= 0) { + LOG(ERROR, "initialize_tls_session_client(): can't read CA file: '%s'", PROXY_CA_FILE); gnutls_certificate_free_credentials(*x509_cred); @@ -404,10 +411,10 @@ gnutls_certificate_allocate_credentials(): %s", PROXY_KEY_FILE, GNUTLS_X509_FMT_PEM); } - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, - "initialize_tls_session_client(): \ -can't read server certificate ('%s') or key file ('%s'): %s", + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, + "initialize_tls_session_client(): " + "can't read server certificate ('%s') or key file ('%s'): %s", path, PROXY_KEY_FILE, gnutls_strerror(result)); gnutls_certificate_free_credentials(*x509_cred); /* Could be a missing certificate. */ @@ -417,16 +424,16 @@ can't read server certificate ('%s') or key file ('%s'): %s", gnutls_certificate_set_dh_params(*x509_cred, global_tls_dh_params); result = gnutls_init(session, GNUTLS_SERVER); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, "initialize_tls_session_client(): gnutls_init(): %s", gnutls_strerror(result)); gnutls_certificate_free_credentials(*x509_cred); return -1; } result = gnutls_priority_set(*session, global_tls_priority_cache); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, "initialize_tls_session_client(): gnutls_priority_set(): %s", gnutls_strerror(result)); gnutls_deinit(*session); @@ -435,8 +442,8 @@ can't read server certificate ('%s') or key file ('%s'): %s", } result = gnutls_credentials_set(*session, GNUTLS_CRD_CERTIFICATE, *x509_cred); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, "initialize_tls_session_client(): gnutls_credentials_set(): %s", gnutls_strerror(result)); gnutls_deinit(*session); @@ -454,25 +461,25 @@ static int initialize_tls_session_server(int peer_socket, int result; result = gnutls_certificate_allocate_credentials(x509_cred); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, - "initialize_tls_session_server(): \ -gnutls_certificate_allocate_credentials(): %s", + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, + "initialize_tls_session_server(): " + "gnutls_certificate_allocate_credentials(): %s", gnutls_strerror(result)); return -1; } result = gnutls_init(session, GNUTLS_CLIENT); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, "initialize_tls_session_server(): gnutls_init(): %s", gnutls_strerror(result)); gnutls_certificate_free_credentials(*x509_cred); return -1; } result = gnutls_priority_set(*session, global_tls_priority_cache); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, "initialize_tls_session_server(): gnutls_priority_set(): %s", gnutls_strerror(result)); gnutls_deinit(*session); @@ -481,8 +488,8 @@ gnutls_certificate_allocate_credentials(): %s", } result = gnutls_credentials_set(*session, GNUTLS_CRD_CERTIFICATE, *x509_cred); - if (GNUTLS_E_SUCCESS != result) { - LOG(LOG_ERROR, + if (result != GNUTLS_E_SUCCESS) { + LOG(ERROR, "initialize_tls_session_server(): gnutls_credentials_set(): %s", gnutls_strerror(result)); gnutls_deinit(*session); @@ -496,83 +503,107 @@ gnutls_certificate_allocate_credentials(): %s", } +static int fdopen_read_write(int socket, FILE **read_fd, FILE **write_fd) { + *read_fd = fdopen(socket, "r"); + if (*read_fd == NULL) { + LOG_PERROR(WARNING, "fdopen_read_write(): fdopen(\"r\") failed"); + return -1; + } + + *write_fd = fdopen(dup(socket), "w"); + if (*write_fd == NULL) { + LOG_PERROR(WARNING, "fdopen_read_write(): fdopen(\"w\") failed"); + fclose(*read_fd); + *read_fd = NULL; /* "tell" caller read_fd is already closed */ + return -1; + } + + return 0; +} + /* Read HTTP request line and headers (ignored). * * On success 0 is returned, -1 on client error, -2 on unexpected EOF. */ static int read_http_request(FILE *client_fd, char *request, size_t length) { char buffer[MAX_REQUEST_LINE]; + int found_proxy_authorization; - if (NULL == fgets(request, (int)length, client_fd)) { + assert(length <= INT_MAX); + if (fgets(request, (int)length, client_fd) == NULL) { if (ferror(client_fd)) { - LOG_PERROR(LOG_WARNING, "read_http_request(): fgets()"); + LOG_PERROR(WARNING, "read_http_request(): fgets()"); return -1; } - + /* EOF */ return -2; } - while (NULL != fgets(buffer, sizeof(buffer), client_fd)) { + found_proxy_authorization = 0; + while (fgets(buffer, sizeof(buffer), client_fd) != NULL) { + const char *authentication = "Proxy-Authorization: Basic "; + + if (http_digest_authorization != NULL + && !strncmp(buffer, authentication, strlen(authentication))) { + found_proxy_authorization = 1; + + /* Check if the passphrase matches. */ + strtok(buffer, "\r\n"); + if (strcmp(buffer + strlen(authentication), + http_digest_authorization)) { + return -3; + } + } + /* End of header. */ - if (0 == strcmp(buffer, "\n") || 0 == strcmp(buffer, "\r\n")) { + if (!strcmp(buffer, "\n") || !strcmp(buffer, "\r\n")) { break; } } if (ferror(client_fd)) { - LOG_PERROR(LOG_WARNING, "read_http_request(): fgets()"); + LOG_PERROR(WARNING, "read_http_request(): fgets()"); return -1; } + if (http_digest_authorization != NULL && !found_proxy_authorization) { + return -3; + } + return 0; } static void send_bad_request(FILE *client_fd) { -#define RESPONSE_ERROR "400 Bad Request" -#define RESPONSE_MSG "Your browser sent an invalid request." - fprintf(client_fd, HTTP_RESPONSE_FORMAT, - RESPONSE_ERROR, RESPONSE_ERROR, RESPONSE_ERROR, - RESPONSE_MSG); -#undef RESPONSE_ERROR -#undef RESPONSE_MSG + const char error[] = "400 Bad Request"; + const char msg[] = "Your browser sent an invalid request."; + fprintf(client_fd, HTTP_RESPONSE_FORMAT, error, "", error, error, msg); + fflush(client_fd); +} +static void send_authentication_required(FILE *client_fd) { + const char error[] = "407 Proxy Authentication Required"; + const char auth[] = "Proxy-Authenticate: Basic realm=\"tlsproxy\"\r\n"; + const char msg[] = "TODO"; + fprintf(client_fd, HTTP_RESPONSE_FORMAT, error, auth, error, error, msg); + fflush(client_fd); } static void send_forwarding_failure(FILE *client_fd) { -#define RESPONSE_ERROR "503 Forwarding failure" -#define RESPONSE_MSG "Failed to connect to server, check logs." - fprintf(client_fd, HTTP_RESPONSE_FORMAT, - RESPONSE_ERROR, RESPONSE_ERROR, RESPONSE_ERROR, - RESPONSE_MSG); -#undef RESPONSE_ERROR -#undef RESPONSE_MSG + const char error[] = "503 Forwarding failure"; + const char msg[] = "Failed to connect to server, check logs."; + fprintf(client_fd, HTTP_RESPONSE_FORMAT, error, "", error, error, msg); + fflush(client_fd); } static void tls_send_invalid_cert_message(gnutls_session_t session) { -#define RESPONSE_ERROR "500 Internal Server Error" -#define RESPONSE_MSG "Server certificate validation failed, check logs." + const char error[] = "500 Internal Server Error"; + const char msg[] = "Server certificate validation failed, check logs."; int result; - char buffer[sizeof(HTTP_RESPONSE_FORMAT) - 1 /* '\0' */ - - 4 * 2 /* four %s */ - + (sizeof(RESPONSE_ERROR) - 1 /* '\0' */) * 3 - + sizeof(RESPONSE_MSG) - 1 /* '\0' */ - + 1 /* '\0' */]; - - result = snprintf(buffer, sizeof(buffer), - HTTP_RESPONSE_FORMAT, - RESPONSE_ERROR, RESPONSE_ERROR, RESPONSE_ERROR, - RESPONSE_MSG); - if (result < 0) { - LOG_PERROR(LOG_ERROR, - "tls_send_invalid_cert_message(): snprintf failed"); - return; - } else if ((size_t)result >= sizeof(buffer)) { - LOG(LOG_ERROR, - "tls_send_invalid_cert_message(): snprintf buffer too short"); - return; - } - - gnutls_record_send(session, buffer, sizeof(buffer) - 1); - /* don't send trailing '\0' */ -#undef RESPONSE_ERROR -#undef RESPONSE_MSG + char buffer[sizeof(HTTP_RESPONSE_FORMAT) + + 3 * sizeof(error) + sizeof(msg)]; + + result = snprintf(buffer, sizeof(buffer), HTTP_RESPONSE_FORMAT, + error, "", error, error, msg); + assert(result > 0 && (size_t)result < sizeof(buffer)); + + gnutls_record_send(session, buffer, strlen(buffer)); } @@ -588,22 +619,22 @@ static void transfer_data(int client, int server) { fds[1].revents = 0; for (;;) { - int result = poll(fds, 2, -1 /* no timeout */); + int result = poll(fds, 2 /* fd count */, -1 /* no timeout */); if (result < 0) { - LOG_PERROR(LOG_ERROR, "transfer_data(): poll()"); + LOG_PERROR(ERROR, "transfer_data(): poll()"); return; } /* Data available from client. */ if (fds[0].revents & POLLIN || fds[0].revents & POLLPRI) { - if (0 != read_from_write_to(client, server)) { + if (read_from_write_to(client, server) != 0) { /* EOF (or other error) */ break; } } /* Data available from server. */ if (fds[1].revents & POLLIN || fds[1].revents & POLLPRI) { - if (0 != read_from_write_to(server, client)) { + if (read_from_write_to(server, client) != 0) { /* EOF (or other error) */ break; } @@ -627,26 +658,25 @@ static int read_from_write_to(int from, int to) { ssize_t size_written; char buffer[4096]; - LOG(LOG_DEBUG, "read_from_write_to(): %d -> %d", from, to); + LOG(DEBUG, "read_from_write_to(): %d -> %d", from, to); size_read = read(from, buffer, sizeof(buffer)); - if (0 > size_read) { - LOG_PERROR(LOG_WARNING, "read_from_write_to(): read()"); + if (size_read < 0) { + LOG_PERROR(WARNING, "read_from_write_to(): read()"); return -1; - } /* EOF */ - if (0 == size_read) { + } else if (size_read == 0) { return -1; } size_written = write(to, buffer, (size_t)size_read); - if (0 > size_written) { - LOG_PERROR(LOG_WARNING, "read_from_write_to(): write()"); + if (size_written < 0) { + LOG_PERROR(WARNING, "read_from_write_to(): write()"); return -1; } if (size_read != size_written) { - LOG(LOG_ERROR, "read_from_write_to(): only written %ld of %ld bytes!", - (long int)size_written, (long int)size_read); + LOG(ERROR, "read_from_write_to(): only written %ld of %ld bytes!", + (long int)size_written, (long int)size_read); return -1; } @@ -670,31 +700,31 @@ static void transfer_data_tls(int client, int server, /* Get maximum possible buffer size. */ buffer_size = gnutls_record_get_max_size(client_session); - if (buffer_size > gnutls_record_get_max_size(server_session)) { + if (gnutls_record_get_max_size(server_session) < buffer_size) { buffer_size = gnutls_record_get_max_size(server_session); } - LOG(LOG_DEBUG, "transfer_data_tls(): suggested buffer size: %ld", - (long int)buffer_size); + LOG(DEBUG, "transfer_data_tls(): suggested buffer size: %ld", + (long int)buffer_size); for (;;) { - int result = poll(fds, 2, -1 /* no timeout */); + int result = poll(fds, 2 /* fd count */, -1 /* no timeout */); if (result < 0) { - LOG_PERROR(LOG_ERROR, "transfer_data(): poll()"); + LOG_PERROR(ERROR, "transfer_data(): poll()"); return; } /* Data available from client. */ if (fds[0].revents & POLLIN || fds[0].revents & POLLPRI) { - if (0 != read_from_write_to_tls(client_session, server_session, - buffer_size)) { + if (read_from_write_to_tls(client_session, server_session, + buffer_size) != 0) { /* EOF (or other error) */ break; } } /* Data available from server. */ if (fds[1].revents & POLLIN || fds[1].revents & POLLPRI) { - if (0 != read_from_write_to_tls(server_session, client_session, - buffer_size)) { + if (read_from_write_to_tls(server_session, client_session, + buffer_size) != 0) { /* EOF (or other error) */ break; } @@ -722,29 +752,28 @@ static int read_from_write_to_tls(gnutls_session_t from, if (buffer_size > sizeof(buffer)) { buffer_size = sizeof(buffer); } - LOG(LOG_DEBUG, "read_from_write_to_tls(): used buffer size: %ld", - (long int)buffer_size); + LOG(DEBUG, "read_from_write_to_tls(): used buffer size: %ld", + (long int)buffer_size); size_read = gnutls_record_recv(from, buffer, buffer_size); - if (0 > size_read) { - LOG(LOG_WARNING, "read_from_write_to_tls(): gnutls_record_recv(): %s", - gnutls_strerror((int)size_read)); + if (size_read < 0) { + LOG(WARNING, "read_from_write_to_tls(): gnutls_record_recv(): %s", + gnutls_strerror((int)size_read)); return -1; - } /* EOF */ - if (0 == size_read) { + } else if (size_read == 0) { return -1; } size_written = gnutls_record_send(to, buffer, (size_t)size_read); - if (0 > size_written) { - LOG(LOG_WARNING, "read_from_write_to_tls(): gnutls_record_send(): %s", - gnutls_strerror((int)size_written)); + if (size_written < 0) { + LOG(WARNING, "read_from_write_to_tls(): gnutls_record_send(): %s", + gnutls_strerror((int)size_written)); return -1; } if (size_read != size_written) { - LOG(LOG_ERROR, "read_from_write_to_tls(): only written %ld of %ld bytes!", - (long int)size_written, (long int)size_read); + LOG(ERROR, "read_from_write_to_tls(): only written %ld of %ld bytes!", + (long int)size_written, (long int)size_read); return -1; } @@ -760,7 +789,7 @@ static int connect_to_host(const char *hostname, const char *port) { int server_socket; struct addrinfo *server; - if (NULL == hostname || NULL == port) { + if (hostname == NULL || port == NULL) { return -1; } @@ -773,35 +802,39 @@ static int connect_to_host(const char *hostname, const char *port) { | AI_ADDRCONFIG /* supported by this computer */ | AI_V4MAPPED; /* support IPv4 through IPv6 */ gai_return = getaddrinfo(hostname, port, &gai_hints, &gai_result); - if (0 != gai_return) { - LOG_PERROR(LOG_WARNING, "connect_to_host(): getaddrinfo()"); + if (gai_return != 0) { + if (gai_return == EAI_SYSTEM) { + LOG_PERROR(WARNING, "connect_to_host(): getaddrinfo()"); + } else { + LOG(WARNING, "connect_to_host(): getaddrinfo(): %s", + gai_strerror(gai_return)); + } return -1; } /* Now try to connect to each server returned by getaddrinfo(), use the * first successful connect. */ - for (server = gai_result; NULL != server; server = server->ai_next) { + for (server = gai_result; server != NULL; server = server->ai_next) { server_socket = socket(server->ai_family, server->ai_socktype, server->ai_protocol); - if (-1 == server_socket) { - LOG_PERROR(LOG_DEBUG, "connect_to_host(): socket(), trying next"); + if (server_socket < 0) { + LOG_PERROR(DEBUG, "connect_to_host(): socket(), trying next"); continue; } - if (-1 != connect(server_socket, server->ai_addr, - server->ai_addrlen)) { + if (connect(server_socket, server->ai_addr, server->ai_addrlen) == 0) { break; } - LOG_PERROR(LOG_DEBUG, "connect_to_host(): connect(), trying next"); + LOG_PERROR(DEBUG, "connect_to_host(): connect(), trying next"); close(server_socket); } /* Make sure we free the result from getaddrinfo(). */ freeaddrinfo(gai_result); - if (NULL == server) { - LOG_PERROR(LOG_WARNING, "connect_to_host(): no server found, abort"); + if (server == NULL) { + LOG_PERROR(WARNING, "connect_to_host(): no server found, abort"); return -1; } @@ -822,20 +855,19 @@ static int parse_request(const char *request, char *host, char *port, char *position; /* scanf() doesn't check spaces. */ - if (0 != strncmp(request, "CONNECT ", 8)) { + if (strncmp(request, "CONNECT ", 8)) { return -1; } /* Check request and extract data, "host:port" is not yet separated. */ - if (2 != sscanf(request, "CONNECT %s HTTP/1.%d", - host, version_minor)) { + if (sscanf(request, "CONNECT %s HTTP/1.%d", host, version_minor) != 2) { return -1; } /* Make sure ":port" is there. */ - if (NULL == (position = strchr(host, ':'))) { + if ((position = strchr(host, ':')) == NULL) { return -1; } /* Make sure port is numeric. */ - if (1 != sscanf(position + 1, "%d", &port_unused)) { + if (sscanf(position + 1, "%d", &port_unused) != 1) { return -1; } /* Store it in *port. */