#include "connection.h"
#include "verify.h"
-/* close() */
-#include <unistd.h>
-/* getaddrinfo() */
+#include <errno.h>
#include <netdb.h>
-/* poll() */
#include <poll.h>
-/* errno */
-#include <errno.h>
+#include <unistd.h>
/* Maximum length of a HTTP request line. Longer request lines are aborted
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_forwarding_failure(FILE *client_fd);
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
LOG(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 (client_fd == NULL) {
- 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));
+ result = read_http_request(client_fd_read, buffer, sizeof(buffer));
if (result == -1) {
- /* Read error. */
LOG(LOG_WARNING, "read_http_request(): client read error");
goto out;
} else if (result == -2) {
- /* EOF */
LOG(LOG_WARNING, "read_http_request(): client EOF");
- send_bad_request(client_fd);
+ send_bad_request(client_fd_write);
goto out;
}
if (parse_request(buffer, host, port, &version_minor) != 0) {
LOG(LOG_WARNING, "bad request: %s", buffer);
- send_bad_request(client_fd);
+ send_bad_request(client_fd_write);
goto out;
}
if (server_socket == -1) {
LOG(LOG_WARNING, "failed to connect to server");
- send_forwarding_failure(client_fd);
+ send_forwarding_failure(client_fd_write);
goto out;
}
- server_fd = fdopen(server_socket, "a+");
- if (server_fd == NULL) {
- 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 (global_proxy_host != NULL && global_proxy_port != NULL) {
- fprintf(server_fd, "CONNECT %s:%s HTTP/1.0\r\n", host, port);
- fprintf(server_fd, "\r\n");
+ 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));
+ result = read_http_request(server_fd_read, buffer, sizeof(buffer));
if (result == -1) {
- /* Read error. */
LOG(LOG_WARNING, "read_http_request(): proxy read error");
- send_forwarding_failure(client_fd);
+ send_forwarding_failure(client_fd_write);
goto out;
} else if (result == -2) {
- /* EOF */
LOG(LOG_WARNING, "read_http_request(): proxy EOF");
- send_forwarding_failure(client_fd);
+ send_forwarding_failure(client_fd_write);
goto out;
}
/* Check response of proxy server. */
- if (strncmp(buffer, "HTTP/1.0 200", 12) != 0) {
+ if (strncmp(buffer, "HTTP/1.0 200", 12)) {
LOG(LOG_WARNING, "bad proxy response: %s", buffer);
- send_forwarding_failure(client_fd);
+ send_forwarding_failure(client_fd_write);
goto out;
}
}
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");
&server_x509_cred);
if (result != 0) {
LOG(LOG_WARNING, "initialize_tls_session_server() failed");
- send_forwarding_failure(client_fd);
+ send_forwarding_failure(client_fd_write);
goto out;
}
server_session_init = 1;
if (result != GNUTLS_E_SUCCESS) {
LOG(LOG_WARNING, "server TLS handshake failed: %s",
gnutls_strerror(result));
- send_forwarding_failure(client_fd);
+ send_forwarding_failure(client_fd_write);
goto out;
}
server_session_started = 1;
/* Make sure the server certificate is valid and known. */
if (verify_tls_connection(server_session, host) != 0) {
LOG(LOG_ERROR, "server certificate validation failed!");
- /* We send the error message over our TLS connection to the client,
+ /* 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;
&client_x509_cred);
if (result != 0) {
LOG(LOG_WARNING, "initialize_tls_session_client() failed");
- send_forwarding_failure(client_fd);
+ 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");
if (result != GNUTLS_E_SUCCESS) {
LOG(LOG_WARNING, "client TLS handshake failed: %s",
gnutls_strerror(result));
- send_forwarding_failure(client_fd);
+ send_forwarding_failure(client_fd_write);
goto out;
}
client_session_started = 1;
out:
/* Close TLS sessions if necessary. Use GNUTLS_SHUT_RDWR so the data is
* reliable transmitted. */
- if (server_session_started != 0) {
+ if (server_session_started) {
gnutls_bye(server_session, GNUTLS_SHUT_RDWR);
}
- if (client_session_started != 0) {
+ if (client_session_started) {
gnutls_bye(client_session, GNUTLS_SHUT_RDWR);
}
- if (server_session_init != 0) {
+ if (server_session_init) {
gnutls_deinit(server_session);
gnutls_certificate_free_credentials(server_x509_cred);
}
- if (client_session_init != 0) {
+ if (client_session_init) {
gnutls_deinit(client_session);
gnutls_certificate_free_cas(client_x509_cred);
gnutls_certificate_free_keys(client_x509_cred);
}
/* Close connection to server/proxy. */
- if (server_fd != NULL) {
- fclose(server_fd);
+ 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");
/* Close connection to client. */
- if (client_fd != NULL) {
- 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);
}
/* 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 = (strcmp(hostname, "invalid") == 0);
+ use_invalid_cert = (!strcmp(hostname, "invalid"));
if (proxy_certificate_path(hostname, path, sizeof(path)) != 0) {
LOG(LOG_ERROR,
}
+static int fdopen_read_write(int socket, FILE **read_fd, FILE **write_fd) {
+ *read_fd = fdopen(socket, "r");
+ if (*read_fd == NULL) {
+ LOG_PERROR(LOG_WARNING, "fdopen_read_write(): fdopen(\"r\") failed");
+ return -1;
+ }
+
+ *write_fd = fdopen(dup(socket), "w");
+ if (*write_fd == NULL) {
+ LOG_PERROR(LOG_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.
LOG_PERROR(LOG_WARNING, "read_http_request(): fgets()");
return -1;
}
-
+ /* EOF */
return -2;
}
while (fgets(buffer, sizeof(buffer), client_fd) != NULL) {
/* End of header. */
- if (strcmp(buffer, "\n") == 0 || strcmp(buffer, "\r\n") == 0) {
+ if (!strcmp(buffer, "\n") || !strcmp(buffer, "\r\n")) {
break;
}
}
fprintf(client_fd, HTTP_RESPONSE_FORMAT,
RESPONSE_ERROR, RESPONSE_ERROR, RESPONSE_ERROR,
RESPONSE_MSG);
+ fflush(client_fd);
#undef RESPONSE_ERROR
#undef RESPONSE_MSG
}
fprintf(client_fd, HTTP_RESPONSE_FORMAT,
RESPONSE_ERROR, RESPONSE_ERROR, RESPONSE_ERROR,
RESPONSE_MSG);
+ fflush(client_fd);
#undef RESPONSE_ERROR
#undef RESPONSE_MSG
}
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()");
return;
if (size_read < 0) {
LOG_PERROR(LOG_WARNING, "read_from_write_to(): read()");
return -1;
- }
/* EOF */
- if (size_read == 0) {
+ } else if (size_read == 0) {
return -1;
}
(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()");
return;
LOG(LOG_WARNING, "read_from_write_to_tls(): gnutls_record_recv(): %s",
gnutls_strerror((int)size_read));
return -1;
- }
/* EOF */
- if (size_read == 0) {
+ } else if (size_read == 0) {
return -1;
}
| AI_V4MAPPED; /* support IPv4 through IPv6 */
gai_return = getaddrinfo(hostname, port, &gai_hints, &gai_result);
if (gai_return != 0) {
- LOG_PERROR(LOG_WARNING, "connect_to_host(): getaddrinfo()");
+ if (gai_return == EAI_SYSTEM) {
+ LOG_PERROR(LOG_WARNING, "connect_to_host(): getaddrinfo()");
+ } else {
+ LOG(LOG_WARNING, "connect_to_host(): getaddrinfo(): %s",
+ gai_strerror(gai_return));
+ }
return -1;
}
char *position;
/* scanf() doesn't check spaces. */
- if (strncmp(request, "CONNECT ", 8) != 0) {
+ if (strncmp(request, "CONNECT ", 8)) {
return -1;
}
/* Check request and extract data, "host:port" is not yet separated. */