3 I've been hacking together a way to use rsync with OpenSSL, and have
4 attached my current patch against a recent CVS tree. The details of
5 this implementation are:
7 1. The SSL code is added as a "layer" that is forked into its own
10 2. An SSL connection is established by the client issuing the
15 And, if the daemon allows SSL, it replies with
19 At which point both sides begin negotiating the SSL connection.
20 Servers that can't or don't want to use SSL just treat it as a
21 normal unknown command.
23 3. The SSL code is meant to be unobtrusive, and when this patch is
24 applied the program may still be built with no SSL code.
26 4. There are a number of details not implemented.
28 All warnings apply; I don't do C programming all that often, so I
29 can't say if I've left any cleanup/compatibility errors in the code.
31 To use this patch, run these commands for a successful build:
33 patch -p1 <patches/openssl-support.diff
40 @@ -39,7 +39,7 @@ OBJS3=progress.o pipe.o
41 DAEMON_OBJ = params.o loadparm.o clientserver.o access.o connection.o authenticate.o
42 popt_OBJS=popt/findme.o popt/popt.o popt/poptconfig.o \
43 popt/popthelp.o popt/poptparse.o
44 -OBJS=$(OBJS1) $(OBJS2) $(OBJS3) $(DAEMON_OBJ) $(LIBOBJ) $(ZLIBOBJ) @BUILD_POPT@
45 +OBJS=$(OBJS1) $(OBJS2) $(OBJS3) $(DAEMON_OBJ) $(LIBOBJ) $(ZLIBOBJ) @BUILD_POPT@ @SSL_OBJS@
47 TLS_OBJ = tls.o syscall.o lib/compat.o lib/snprintf.o lib/permstring.o
58 extern int keep_partial;
59 extern int log_got_error;
60 extern char *partial_dir;
61 @@ -117,6 +120,14 @@ NORETURN void _exit_cleanup(int code, co
76 --- old/clientserver.c
77 +++ new/clientserver.c
78 @@ -30,6 +30,9 @@ extern int am_sender;
85 extern int rsync_port;
86 extern int kluge_around_eof;
87 extern int daemon_over_rsh;
88 @@ -106,8 +109,18 @@ int start_socket_client(char *host, char
89 set_socket_options(fd, sockopts);
91 ret = start_inband_exchange(user, path, fd, fd, argc);
95 - return ret ? ret : client_run(fd, fd, -1, argc, argv);
98 + int f_in = get_tls_rfd();
99 + int f_out = get_tls_wfd();
100 + return client_run(f_in, f_out, -1, argc, argv);
104 + return client_run(fd, fd, -1, argc, argv);
107 int start_inband_exchange(char *user, char *path, int f_in, int f_out,
108 @@ -168,6 +181,33 @@ int start_inband_exchange(char *user, ch
110 print_child_argv(sargs);
114 + io_printf(f_out, "#starttls\n");
116 + if (!read_line(f_in, line, sizeof(line)-1)) {
117 + rprintf(FERROR, "rsync: did not receive reply to #starttls\n");
120 + if (strncmp(line, "@ERROR", 6) == 0) {
121 + rprintf(FERROR, "%s\n", line);
124 + if (strcmp(line, "@RSYNCD: starttls") == 0) {
127 + rprintf(FINFO, "%s\n", line);
129 + if (start_tls(f_in, f_out)) {
130 + rprintf(FERROR, "rsync: error during SSL handshake: %s\n",
134 + f_in = get_tls_rfd();
135 + f_out = get_tls_wfd();
139 p = strchr(path,'/');
141 io_printf(f_out, "%s\n", path);
142 @@ -196,6 +236,10 @@ int start_inband_exchange(char *user, ch
143 * server to terminate the listing of modules.
144 * We don't want to go on and transfer
145 * anything; just exit. */
153 @@ -203,6 +247,10 @@ int start_inband_exchange(char *user, ch
154 rprintf(FERROR, "%s\n", line);
155 /* This is always fatal; the server will now
156 * close the socket. */
164 @@ -780,6 +828,9 @@ int start_daemon(int f_in, int f_out)
165 if (protocol_version > remote_protocol)
166 protocol_version = remote_protocol;
172 if (!read_line(f_in, line, sizeof line - 1))
174 @@ -791,6 +842,20 @@ int start_daemon(int f_in, int f_out)
179 + if (use_ssl && strcmp(line, "#starttls") == 0) {
180 + io_printf(f_out, "@RSYNCD: starttls\n");
181 + if (start_tls(f_in, f_out)) {
182 + rprintf(FLOG, "SSL connection failed: %s\n",
186 + f_in = get_tls_rfd();
187 + f_out = get_tls_wfd();
193 /* it's some sort of command that I don't understand */
194 io_printf(f_out, "@ERROR: Unknown command '%s'\n", line);
197 @@ -290,6 +290,21 @@ if test x"$enable_locale" != x"no"; then
198 AC_DEFINE(CONFIG_LOCALE)
201 +AC_ARG_ENABLE(openssl,
202 + AC_HELP_STRING([--enable-openssl], [compile SSL support with OpenSSL.]))
204 +if test "x$enable_openssl" != xno
207 + AC_CHECK_LIB(ssl, SSL_library_init, , [have_ssl=no])
208 + if test "x$have_ssl" = xyes
210 + AC_DEFINE(HAVE_OPENSSL, 1, [true if you want to use SSL.])
216 AC_MSG_CHECKING([whether to call shutdown on all sockets])
218 *cygwin* ) AC_MSG_RESULT(yes)
221 @@ -173,6 +173,14 @@ int logfile_format_has_o_or_i = 0;
222 int always_checksum = 0;
227 +char *ssl_cert_path = NULL;
228 +char *ssl_key_path = NULL;
229 +char *ssl_key_passwd = NULL;
230 +char *ssl_ca_path = NULL;
233 #define MAX_BATCH_NAME_LEN 256 /* Must be less than MAXPATHLEN-13 */
234 char *batch_name = NULL;
236 @@ -201,6 +209,7 @@ static void print_rsync_version(enum log
237 char const *hardlinks = "no ";
238 char const *links = "no ";
239 char const *ipv6 = "no ";
240 + char const *ssl = "no ";
241 STRUCT_STAT *dumstat;
243 #ifdef HAVE_SOCKETPAIR
244 @@ -223,6 +232,10 @@ static void print_rsync_version(enum log
252 rprintf(f, "%s version %s protocol version %d\n",
253 RSYNC_NAME, RSYNC_VERSION, PROTOCOL_VERSION);
254 rprintf(f, "Copyright (C) 1996-2006 by Andrew Tridgell, Wayne Davison, and others.\n");
255 @@ -235,9 +248,9 @@ static void print_rsync_version(enum log
256 /* Note that this field may not have type ino_t. It depends
257 * on the complicated interaction between largefile feature
259 - rprintf(f, " %sinplace, %sIPv6, "
260 + rprintf(f, " %sinplace, %sIPv6, %sSSL, "
261 "%d-bit system inums, %d-bit internal inums\n",
262 - have_inplace, ipv6,
263 + have_inplace, ipv6, ssl,
264 (int) (sizeof dumstat->st_ino * 8),
265 (int) (sizeof (int64) * 8));
266 #ifdef MAINTAINER_MODE
267 @@ -385,6 +398,13 @@ void usage(enum logcode F)
268 rprintf(F," -4, --ipv4 prefer IPv4\n");
269 rprintf(F," -6, --ipv6 prefer IPv6\n");
272 + rprintf(F," --ssl allow socket connections to use SSL\n");
273 + rprintf(F," --ssl-cert=FILE path to daemon's SSL certificate\n");
274 + rprintf(F," --ssl-key=FILE path to daemon's SSL private key\n");
275 + rprintf(F," --ssl-key-passwd=PASS password for PEM-encoded private key\n");
276 + rprintf(F," --ssl-ca-certs=FILE path to trusted CA certificates\n");
278 rprintf(F," --version print version number\n");
279 rprintf(F,"(-h) --help show this help (-h works with no other options)\n");
281 @@ -398,7 +418,7 @@ enum {OPT_VERSION = 1000, OPT_DAEMON, OP
282 OPT_FILTER, OPT_COMPARE_DEST, OPT_COPY_DEST, OPT_LINK_DEST, OPT_HELP,
283 OPT_INCLUDE, OPT_INCLUDE_FROM, OPT_MODIFY_WINDOW, OPT_MIN_SIZE, OPT_CHMOD,
284 OPT_READ_BATCH, OPT_WRITE_BATCH, OPT_ONLY_WRITE_BATCH, OPT_MAX_SIZE,
286 + OPT_NO_D, OPT_USE_SSL,
287 OPT_SERVER, OPT_REFUSED_BASE = 9000};
289 static struct poptOption long_options[] = {
290 @@ -545,6 +565,13 @@ static struct poptOption long_options[]
291 {"checksum-seed", 0, POPT_ARG_INT, &checksum_seed, 0, 0, 0 },
292 {"server", 0, POPT_ARG_NONE, 0, OPT_SERVER, 0, 0 },
293 {"sender", 0, POPT_ARG_NONE, 0, OPT_SENDER, 0, 0 },
295 + {"ssl", 0, POPT_ARG_NONE, 0, OPT_USE_SSL, 0, 0},
296 + {"ssl-cert", 0, POPT_ARG_STRING, &ssl_cert_path, OPT_USE_SSL, 0, 0},
297 + {"ssl-key", 0, POPT_ARG_STRING, &ssl_key_path, OPT_USE_SSL, 0, 0},
298 + {"ssl-key-passwd", 0, POPT_ARG_STRING, &ssl_key_passwd, OPT_USE_SSL, 0, 0},
299 + {"ssl-ca-certs", 0, POPT_ARG_STRING, &ssl_ca_path, OPT_USE_SSL, 0, 0},
301 /* All the following options switch us into daemon-mode option-parsing. */
302 {"config", 0, POPT_ARG_STRING, 0, OPT_DAEMON, 0, 0 },
303 {"daemon", 0, POPT_ARG_NONE, 0, OPT_DAEMON, 0, 0 },
304 @@ -572,6 +599,13 @@ static void daemon_usage(enum logcode F)
305 rprintf(F," -4, --ipv4 prefer IPv4\n");
306 rprintf(F," -6, --ipv6 prefer IPv6\n");
309 + rprintf(F," --ssl allow socket connections to use SSL\n");
310 + rprintf(F," --ssl-cert=FILE path to daemon's SSL certificate\n");
311 + rprintf(F," --ssl-key=FILE path to daemon's SSL private key\n");
312 + rprintf(F," --ssl-key-passwd=PASS password for PEM-encoded private key\n");
313 + rprintf(F," --ssl-ca-certs=FILE path to trusted CA certificates\n");
315 rprintf(F," --help show this help screen\n");
318 @@ -598,6 +632,13 @@ static struct poptOption long_daemon_opt
319 {"protocol", 0, POPT_ARG_INT, &protocol_version, 0, 0, 0 },
320 {"server", 0, POPT_ARG_NONE, &am_server, 0, 0, 0 },
321 {"temp-dir", 'T', POPT_ARG_STRING, &tmpdir, 0, 0, 0 },
323 + {"ssl", 0, POPT_ARG_NONE, 0, OPT_USE_SSL, 0, 0},
324 + {"ssl-cert", 0, POPT_ARG_STRING, &ssl_cert_path, OPT_USE_SSL, 0, 0},
325 + {"ssl-key", 0, POPT_ARG_STRING, &ssl_key_path, OPT_USE_SSL, 0, 0},
326 + {"ssl-key-passwd", 0, POPT_ARG_STRING, &ssl_key_passwd, OPT_USE_SSL, 0, 0},
327 + {"ssl-ca-certs", 0, POPT_ARG_STRING, &ssl_ca_path, OPT_USE_SSL, 0, 0},
329 {"verbose", 'v', POPT_ARG_NONE, 0, 'v', 0, 0 },
330 {"no-verbose", 0, POPT_ARG_VAL, &verbose, 0, 0, 0 },
331 {"no-v", 0, POPT_ARG_VAL, &verbose, 0, 0, 0 },
332 @@ -855,6 +896,12 @@ int parse_arguments(int *argc, const cha
344 "rsync: %s: %s (in daemon mode)\n",
345 @@ -878,6 +925,17 @@ int parse_arguments(int *argc, const cha
346 exit_cleanup(RERR_SYNTAX);
352 + snprintf(err_buf, sizeof(err_buf),
353 + "Openssl error: %s\n",
360 *argv = poptGetArgs(pc);
361 *argc = count_args(*argv);
363 @@ -1089,6 +1147,12 @@ int parse_arguments(int *argc, const cha
374 /* A large opt value means that set_refuse_options()
375 * turned this option off. */
376 @@ -1365,6 +1429,17 @@ int parse_arguments(int *argc, const cha
377 if (delay_updates && !partial_dir)
378 partial_dir = tmp_partialdir;
383 + snprintf(err_buf, sizeof(err_buf),
384 + "Openssl error: %s\n",
392 #ifdef HAVE_FTRUNCATE
394 @@ -1782,10 +1857,27 @@ char *check_for_hostspec(char *s, char *
398 + int url_prefix_len = sizeof URL_PREFIX - 1;
400 - if (port_ptr && strncasecmp(URL_PREFIX, s, strlen(URL_PREFIX)) == 0) {
402 + url_prefix_len = 0;
403 + else if (strncasecmp(URL_PREFIX, s, url_prefix_len) != 0) {
405 + url_prefix_len = sizeof SSL_URL_PREFIX - 1;
406 + if (strncasecmp(SSL_URL_PREFIX, s, url_prefix_len) != 0)
407 + url_prefix_len = 0;
414 + url_prefix_len = 0;
417 + if (url_prefix_len) {
419 - s += strlen(URL_PREFIX);
420 + s += url_prefix_len;
421 if ((p = strchr(s, '/')) != NULL) {
428 #define DEFAULT_LOCK_FILE "/var/run/rsyncd.lock"
429 #define URL_PREFIX "rsync://"
430 +#define SSL_URL_PREFIX "rsyncs://"
432 #define BACKUP_SUFFIX "~"
434 @@ -419,6 +420,11 @@ enum msgcode {
435 # define SIZEOF_INT64 SIZEOF_OFF_T
439 +#include <openssl/ssl.h>
440 +#include <openssl/err.h>
443 /* Starting from protocol version 26, we always use 64-bit
444 * ino_t and dev_t internally, even if this platform does not
445 * allow files to have 64-bit inums. That's because the
449 +/* -*- c-file-style: "linux" -*-
450 + * ssl.c: operations for negotiating SSL rsync connections.
452 + * Copyright (C) 2003 Casey Marshall <rsdio@metastatic.org>
454 + * This program is free software; you can redistribute it and/or modify
455 + * it under the terms of the GNU General Public License as published by
456 + * the Free Software Foundation; either version 2 of the License, or
457 + * (at your option) any later version.
459 + * This program is distributed in the hope that it will be useful,
460 + * but WITHOUT ANY WARRANTY; without even the implied warranty of
461 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
462 + * GNU General Public License for more details.
464 + * You should have received a copy of the GNU General Public License
465 + * along with this program; if not, write to the Free Software
466 + * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
471 +#ifdef HAVE_SYS_SELECT_H
472 +#include <sys/select.h>
474 +#include <sys/time.h>
475 +#include <sys/types.h>
480 +#define BUF_SIZE 1024
483 +extern int am_daemon;
484 +extern int am_server;
486 +extern char *ssl_cert_path;
487 +extern char *ssl_key_path;
488 +extern char *ssl_key_passwd;
489 +extern char *ssl_ca_path;
491 +static SSL_CTX *ssl_ctx;
493 +static int tls_read[2] = { -1, -1 };
494 +static int tls_write[2] = { -1, -1 };
495 +static int ssl_running;
496 +static int ssl_pid = -1;
498 +#ifdef HAVE_SIGACTION
499 +static struct sigaction sigact;
503 + * A non-interactive callback to be passed to SSL_CTX_set_default_password_cb,
504 + * which merely copies the value of ssl_key_passwd into buf. This is
505 + * used for when the private key password is supplied via an option.
507 +static int default_password_cb(char *buf, int n, UNUSED(int f), UNUSED(void *u))
509 + if (ssl_key_passwd == NULL || n < (int)strlen(ssl_key_passwd))
511 + strncpy(buf, ssl_key_passwd, n-1);
512 + return strlen(ssl_key_passwd);
516 + * If verbose, this method traces the status of the SSL handshake.
518 +static void info_callback(const SSL *ssl, int cb, int val)
525 + cbs = "SSL_CB_LOOP";
528 + cbs = "SSL_CB_EXIT";
531 + cbs = "SSL_CB_READ";
534 + cbs = "SSL_CB_WRITE";
537 + cbs = "SSL_CB_ALERT";
539 + case SSL_CB_READ_ALERT:
540 + cbs = "SSL_CB_READ_ALERT";
542 + case SSL_CB_WRITE_ALERT:
543 + cbs = "SSL_CB_WRITE_ALERT";
545 + case SSL_CB_ACCEPT_LOOP:
546 + cbs = "SSL_CB_ACCEPT_LOOP";
548 + case SSL_CB_ACCEPT_EXIT:
549 + cbs = "SSL_CB_ACCEPT_EXIT";
551 + case SSL_CB_CONNECT_LOOP:
552 + cbs = "SSL_CB_CONNECT_LOOP";
554 + case SSL_CB_CONNECT_EXIT:
555 + cbs = "SSL_CB_CONNECT_EXIT";
557 + case SSL_CB_HANDSHAKE_START:
558 + cbs = "SSL_CB_HANDSHAKE_START";
560 + case SSL_CB_HANDSHAKE_DONE:
561 + cbs = "SSL_CB_HANDSHAKE_DONE";
564 + snprintf(buf, sizeof buf, "??? (%d)", cb);
569 + rprintf(FLOG, "SSL: info_callback(%p,%s,%d)\n", ssl, cbs, val);
570 + if (cb == SSL_CB_HANDSHAKE_DONE) {
571 + SSL_CIPHER_description(SSL_get_current_cipher((SSL*)ssl),
573 + rprintf(FLOG, "SSL: cipher: %s", buf);
579 + * Initializes the SSL context for TLSv1 connections; returns zero on
586 + SSL_library_init();
587 + SSL_load_error_strings();
588 + ssl_ctx = SSL_CTX_new(TLSv1_method());
591 + SSL_CTX_set_info_callback(ssl_ctx, info_callback);
593 + /* Sets the certificate sent to the other party. */
594 + if (ssl_cert_path != NULL
595 + && SSL_CTX_use_certificate_file(ssl_ctx, ssl_cert_path,
596 + SSL_FILETYPE_PEM) != 1)
598 + /* Set up the simple non-interactive callback if the password
599 + * was supplied on the command line. */
600 + if (ssl_key_passwd != NULL)
601 + SSL_CTX_set_default_passwd_cb(ssl_ctx, default_password_cb);
602 + /* Sets the private key that matches the public certificate. */
603 + if (ssl_key_path != NULL) {
604 + if (SSL_CTX_use_PrivateKey_file(ssl_ctx, ssl_key_path,
605 + SSL_FILETYPE_PEM) != 1)
607 + if (SSL_CTX_check_private_key(ssl_ctx) != 1)
610 + if (ssl_ca_path != NULL
611 + && !SSL_CTX_load_verify_locations(ssl_ctx, ssl_ca_path, NULL))
618 + * Returns the error string for the current SSL error, if any.
620 +char *get_ssl_error(void)
622 + return ERR_error_string(ERR_get_error(), NULL);
626 + * Returns the input file descriptor for the SSL connection.
628 +int get_tls_rfd(void)
630 + return tls_read[0];
634 + * Returns the output file descriptor for the SSL connection.
636 +int get_tls_wfd(void)
638 + return tls_write[1];
642 + * Signal handler that ends the SSL connection.
644 +static RETSIGTYPE tls_sigusr1(int UNUSED(val))
655 + * Negotiates the TLS connection, creates a socket pair for communicating
656 + * with the rsync process, then forks into a new process that will handle
657 + * the communication.
659 + * 0 is returned on success.
661 +int start_tls(int f_in, int f_out)
665 + unsigned char buf1[BUF_SIZE], buf2[BUF_SIZE];
666 + int avail1 = 0, avail2 = 0, write1 = 0, write2 = 0;
669 + if (fd_pair(tls_read))
671 + if (fd_pair(tls_write))
674 + set_blocking(tls_read[0]);
675 + set_blocking(tls_read[1]);
676 + set_blocking(tls_write[0]);
677 + set_blocking(tls_write[1]);
678 + set_blocking(f_in);
679 + set_blocking(f_out);
681 + ssl_pid = do_fork();
684 + if (ssl_pid != 0) {
685 + close(tls_write[0]);
686 + close(tls_read[1]);
690 + SIGACTION(SIGUSR1, tls_sigusr1);
691 + ssl = SSL_new(ssl_ctx);
694 + if (am_daemon || am_server)
695 + SSL_set_accept_state(ssl);
697 + SSL_set_connect_state(ssl);
698 + SSL_set_rfd(ssl, f_in);
699 + SSL_set_wfd(ssl, f_out);
701 + tls_fd = SSL_get_fd(ssl);
703 + n = MAX(tls_read[1], n);
704 + n = MAX(tls_fd, n) + 1;
707 + while (ssl_running) {
710 + FD_SET(tls_write[0], &rd);
711 + FD_SET(tls_read[1], &wd);
712 + FD_SET(tls_fd, &rd);
713 + FD_SET(tls_fd, &wd);
715 + r = select(n, &rd, &wd, NULL, NULL);
717 + if (r == -1 && errno == EINTR)
719 + if (FD_ISSET(tls_write[0], &rd)) {
720 + r = read(tls_write[0], buf1+avail1, BUF_SIZE-avail1);
724 + rprintf(FERROR, "pipe read error: %s\n",
729 + if (FD_ISSET(tls_fd, &rd)) {
730 + r = SSL_read(ssl, buf2+avail2, BUF_SIZE-avail2);
734 + switch (SSL_get_error(ssl, r)) {
735 + case SSL_ERROR_ZERO_RETURN:
737 + case SSL_ERROR_WANT_READ:
738 + case SSL_ERROR_WANT_WRITE:
740 + case SSL_ERROR_SYSCALL:
742 + rprintf(FERROR, "SSL spurious EOF\n");
744 + rprintf(FERROR, "SSL I/O error: %s\n",
747 + case SSL_ERROR_SSL:
748 + rprintf(FERROR, "SSL: %s\n",
749 + ERR_error_string(ERR_get_error(), NULL));
752 + rprintf(FERROR, "unexpected ssl error %d\n", r);
757 + if (FD_ISSET(tls_read[1], &wd) && write2 < avail2) {
758 + r = write(tls_read[1], buf2+write2, avail2-write2);
762 + rprintf(FERROR, "pipe write error: %s\n",
767 + if (FD_ISSET(tls_fd, &wd) && write1 < avail1) {
768 + r = SSL_write(ssl, buf1+write1, avail1-write1);
772 + switch (SSL_get_error(ssl, r)) {
773 + case SSL_ERROR_ZERO_RETURN:
775 + case SSL_ERROR_WANT_READ:
776 + case SSL_ERROR_WANT_WRITE:
778 + case SSL_ERROR_SYSCALL:
780 + rprintf(FERROR, "SSL: spurious EOF\n");
782 + rprintf(FERROR, "SSL: I/O error: %s\n",
785 + case SSL_ERROR_SSL:
786 + rprintf(FERROR, "SSL: %s\n",
787 + ERR_error_string(ERR_get_error(), NULL));
790 + rprintf(FERROR, "unexpected ssl error %d\n", r);
795 + if (avail1 == write1)
796 + avail1 = write1 = 0;
797 + if (avail2 == write2)
798 + avail2 = write2 = 0;
801 + /* XXX I'm pretty sure that there is a lot that I am not considering
802 + here. Bugs? Yes, probably. */
804 + /* We're finished. */
806 + close(tls_read[1]);
807 + close(tls_write[0]);
812 + * Ends the TLS connection.
817 + kill(ssl_pid, SIGUSR1);