Got rid of double-fuzz hunk.
[rsync-patches.git] / openssl-support.diff
index 7073a7bb535f8b346097ced7745e0780e7bd61d3..b1d95eb1dd2faf5ec8c9289d45539fe7f9821430 100644 (file)
@@ -1,9 +1,7 @@
 After applying this patch, run these commands for a successful build:
 
-    autoconf
-    autoheader
+    ./prepare-source
     ./configure
-    make proto
     make
 
 Casey Marshall writes:
@@ -20,7 +18,7 @@ this implementation are:
 
        #starttls
 
-     And, if the server allows SSL, it replies with
+     And, if the daemon allows SSL, it replies with
 
        @RSYNCD: starttls
 
@@ -37,8 +35,8 @@ All warnings apply; I don't do C programming all that often, so I
 can't say if I've left any cleanup/compatibility errors in the code.
 
 
---- orig/Makefile.in   2004-11-02 16:47:15
-+++ Makefile.in        2004-10-08 20:17:06
+--- old/Makefile.in
++++ new/Makefile.in
 @@ -39,7 +39,7 @@ OBJS3=progress.o pipe.o
  DAEMON_OBJ = params.o loadparm.o clientserver.o access.o connection.o authenticate.o
  popt_OBJS=popt/findme.o  popt/popt.o  popt/poptconfig.o \
@@ -48,11 +46,11 @@ can't say if I've left any cleanup/compatibility errors in the code.
  
  TLS_OBJ = tls.o syscall.o lib/compat.o lib/snprintf.o lib/permstring.o
  
---- orig/cleanup.c     2005-03-05 18:58:38
-+++ cleanup.c  2005-01-10 10:43:22
-@@ -22,6 +22,9 @@
- #include "rsync.h"
+--- old/cleanup.c
++++ new/cleanup.c
+@@ -26,6 +26,9 @@
+ extern int am_server;
+ extern int am_daemon;
  extern int io_error;
 +#if HAVE_OPENSSL
 +extern int use_ssl;
@@ -60,9 +58,9 @@ can't say if I've left any cleanup/compatibility errors in the code.
  extern int keep_partial;
  extern int log_got_error;
  extern char *partial_dir;
-@@ -97,6 +100,11 @@ void _exit_cleanup(int code, const char 
-       signal(SIGUSR1, SIG_IGN);
-       signal(SIGUSR2, SIG_IGN);
+@@ -103,6 +106,11 @@ NORETURN void _exit_cleanup(int code, co
+       in_exit_cleanup = 1;
  
 +#if HAVE_OPENSSL
 +      if (use_ssl)
@@ -71,24 +69,24 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +
        if (verbose > 3) {
                rprintf(FINFO,"_exit_cleanup(code=%d, file=%s, line=%d): entered\n",
-                       code, safe_fname(file), line);
---- orig/clientserver.c        2005-03-01 03:39:32
-+++ clientserver.c     2004-10-08 20:44:59
-@@ -45,6 +45,9 @@ extern int select_timeout;
- extern int orig_umask;
- extern int no_detach;
- extern int default_af_hint;
+                       code, file, line);
+--- old/clientserver.c
++++ new/clientserver.c
+@@ -29,6 +29,9 @@ extern int am_sender;
+ extern int am_server;
+ extern int am_daemon;
+ extern int am_root;
 +#if HAVE_OPENSSL
 +extern int use_ssl;
 +#endif
- extern char *bind_address;
- extern struct filter_list_struct server_filter_list;
- extern char *config_file;
-@@ -99,8 +102,18 @@ int start_socket_client(char *host, char
-               exit_cleanup(RERR_SOCKETIO);
+ extern int rsync_port;
+ extern int kluge_around_eof;
+ extern int daemon_over_rsh;
+@@ -105,8 +108,18 @@ int start_socket_client(char *host, char
+       set_socket_options(fd, sockopts);
  
        ret = start_inband_exchange(user, path, fd, fd, argc);
-+      if (ret < 0)
++      if (ret)
 +              return ret;
 +
 +#if HAVE_OPENSSL
@@ -99,12 +97,12 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +      }
 +#endif
  
--      return ret < 0? ret : client_run(fd, fd, -1, argc, argv);
+-      return ret ? ret : client_run(fd, fd, -1, argc, argv);
 +      return client_run(fd, fd, -1, argc, argv);
  }
  
- int start_inband_exchange(char *user, char *path, int f_in, int f_out, 
-@@ -161,6 +174,33 @@ int start_inband_exchange(char *user, ch
+ int start_inband_exchange(char *user, char *path, int f_in, int f_out,
+@@ -167,6 +180,33 @@ int start_inband_exchange(char *user, ch
        if (verbose > 1)
                print_child_argv(sargs);
  
@@ -138,7 +136,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
        p = strchr(path,'/');
        if (p) *p = 0;
        io_printf(f_out, "%s\n", path);
-@@ -189,6 +229,10 @@ int start_inband_exchange(char *user, ch
+@@ -195,6 +235,10 @@ int start_inband_exchange(char *user, ch
                         * server to terminate the listing of modules.
                         * We don't want to go on and transfer
                         * anything; just exit. */
@@ -149,7 +147,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
                        exit(0);
                }
  
-@@ -196,6 +240,10 @@ int start_inband_exchange(char *user, ch
+@@ -202,6 +246,10 @@ int start_inband_exchange(char *user, ch
                        rprintf(FERROR, "%s\n", line);
                        /* This is always fatal; the server will now
                         * close the socket. */
@@ -157,10 +155,10 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +                      if (use_ssl)
 +                              end_tls();
 +#endif
-                       return RERR_STARTCLIENT;
-               } else {
-                       rprintf(FINFO,"%s\n", line);
-@@ -549,6 +597,7 @@ static void send_listing(int fd)
+                       return -1;
+               }
+@@ -718,6 +766,7 @@ static void send_listing(int fd)
                io_printf(fd,"@RSYNCD: EXIT\n");
  }
  
@@ -168,7 +166,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
  /* this is called when a connection is established to a client
     and we want to start talking. The setup of the system is done from
     here */
-@@ -598,6 +647,9 @@ int start_daemon(int f_in, int f_out)
+@@ -776,6 +825,9 @@ int start_daemon(int f_in, int f_out)
        if (protocol_version > remote_protocol)
                protocol_version = remote_protocol;
  
@@ -178,7 +176,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
        line[0] = 0;
        if (!read_line(f_in, line, sizeof line - 1))
                return -1;
-@@ -607,6 +659,20 @@ int start_daemon(int f_in, int f_out)
+@@ -787,6 +839,20 @@ int start_daemon(int f_in, int f_out)
                return -1;
        }
  
@@ -199,10 +197,10 @@ can't say if I've left any cleanup/compatibility errors in the code.
        if (*line == '#') {
                /* it's some sort of command that I don't understand */
                io_printf(f_out, "@ERROR: Unknown command '%s'\n", line);
---- orig/configure.in  2005-03-16 02:19:29
-+++ configure.in       2004-07-03 20:22:28
-@@ -271,6 +271,21 @@ yes
-       AC_SEARCH_LIBS(getaddrinfo, inet6)
+--- old/configure.in
++++ new/configure.in
+@@ -290,6 +290,21 @@ if test x"$enable_locale" != x"no"; then
+       AC_DEFINE(CONFIG_LOCALE)
  fi
  
 +AC_ARG_ENABLE(openssl,
@@ -223,9 +221,9 @@ can't say if I've left any cleanup/compatibility errors in the code.
  AC_MSG_CHECKING([whether to call shutdown on all sockets])
  case $host_os in
        *cygwin* ) AC_MSG_RESULT(yes)
---- orig/options.c     2005-03-17 09:05:21
-+++ options.c  2005-03-01 01:34:42
-@@ -157,6 +157,14 @@ int log_format_has_o_or_i = 0;
+--- old/options.c
++++ new/options.c
+@@ -172,6 +172,14 @@ int logfile_format_has_o_or_i = 0;
  int always_checksum = 0;
  int list_only = 0;
  
@@ -240,7 +238,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
  #define MAX_BATCH_NAME_LEN 256        /* Must be less than MAXPATHLEN-13 */
  char *batch_name = NULL;
  
-@@ -181,6 +189,7 @@ static void print_rsync_version(enum log
+@@ -200,6 +208,7 @@ static void print_rsync_version(enum log
        char const *hardlinks = "no ";
        char const *links = "no ";
        char const *ipv6 = "no ";
@@ -248,7 +246,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
        STRUCT_STAT *dumstat;
  
  #ifdef HAVE_SOCKETPAIR
-@@ -203,6 +212,10 @@ static void print_rsync_version(enum log
+@@ -222,6 +231,10 @@ static void print_rsync_version(enum log
        ipv6 = "";
  #endif
  
@@ -258,47 +256,46 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +
        rprintf(f, "%s  version %s  protocol version %d\n",
                RSYNC_NAME, RSYNC_VERSION, PROTOCOL_VERSION);
-       rprintf(f,
-@@ -216,10 +229,10 @@ static void print_rsync_version(enum log
+       rprintf(f, "Copyright (C) 1996-2006 by Andrew Tridgell, Wayne Davison, and others.\n");
+@@ -234,9 +247,9 @@ static void print_rsync_version(enum log
        /* Note that this field may not have type ino_t.  It depends
         * on the complicated interaction between largefile feature
         * macros. */
--      rprintf(f, "              %sinplace, %sIPv6, %d-bit system inums, %d-bit internal inums\n",
-+      rprintf(f, "              %sinplace, %sIPv6, %d-bit system inums, %d-bit internal inums, %sssl\n",
-               have_inplace, ipv6,
+-      rprintf(f, "              %sinplace, %sIPv6, "
++      rprintf(f, "              %sinplace, %sIPv6, %sSSL, "
+               "%d-bit system inums, %d-bit internal inums\n",
+-              have_inplace, ipv6,
++              have_inplace, ipv6, ssl,
                (int) (sizeof dumstat->st_ino * 8),
--              (int) (sizeof (int64) * 8));
-+              (int) (sizeof (int64) * 8), ssl);
+               (int) (sizeof (int64) * 8));
  #ifdef MAINTAINER_MODE
-       rprintf(f, "              panic action: \"%s\"\n",
-               get_panic_action());
-@@ -349,6 +362,13 @@ void usage(enum logcode F)
+@@ -383,6 +396,13 @@ void usage(enum logcode F)
    rprintf(F," -4, --ipv4                  prefer IPv4\n");
    rprintf(F," -6, --ipv6                  prefer IPv6\n");
  #endif
 +#if HAVE_OPENSSL
 +  rprintf(F,"     --ssl                   allow socket connections to use SSL\n");
-+  rprintf(F,"     --ssl-cert=FILE         path to server's SSL certificate\n");
-+  rprintf(F,"     --ssl-key=FILE          path to server's SSL private key\n");
++  rprintf(F,"     --ssl-cert=FILE         path to daemon's SSL certificate\n");
++  rprintf(F,"     --ssl-key=FILE          path to daemon's SSL private key\n");
 +  rprintf(F,"     --ssl-key-passwd=PASS   password for PEM-encoded private key\n");
 +  rprintf(F,"     --ssl-ca-certs=FILE     path to trusted CA certificates\n");
 +#endif
-   rprintf(F," -h, --help                  show this help screen\n");
+   rprintf(F,"     --version               print version number\n");
+   rprintf(F,"(-h) --help                  show this help (-h works with no other options)\n");
  
-   rprintf(F,"\nUse \"rsync --daemon --help\" to see the daemon-mode command-line options.\n");
-@@ -359,7 +379,7 @@ void usage(enum logcode F)
- enum {OPT_VERSION = 1000, OPT_DAEMON, OPT_SENDER, OPT_EXCLUDE, OPT_EXCLUDE_FROM,
-       OPT_FILTER, OPT_COMPARE_DEST, OPT_COPY_DEST, OPT_LINK_DEST,
-       OPT_INCLUDE, OPT_INCLUDE_FROM, OPT_MODIFY_WINDOW,
--      OPT_READ_BATCH, OPT_WRITE_BATCH, OPT_TIMEOUT, OPT_MAX_SIZE,
-+      OPT_READ_BATCH, OPT_WRITE_BATCH, OPT_TIMEOUT, OPT_MAX_SIZE, OPT_USE_SSL,
-       OPT_REFUSED_BASE = 9000};
+@@ -396,7 +416,7 @@ enum {OPT_VERSION = 1000, OPT_DAEMON, OP
+       OPT_FILTER, OPT_COMPARE_DEST, OPT_COPY_DEST, OPT_LINK_DEST, OPT_HELP,
+       OPT_INCLUDE, OPT_INCLUDE_FROM, OPT_MODIFY_WINDOW, OPT_MIN_SIZE, OPT_CHMOD,
+       OPT_READ_BATCH, OPT_WRITE_BATCH, OPT_ONLY_WRITE_BATCH, OPT_MAX_SIZE,
+-      OPT_NO_D,
++      OPT_NO_D, OPT_USE_SSL,
+       OPT_SERVER, OPT_REFUSED_BASE = 9000};
  
  static struct poptOption long_options[] = {
-@@ -458,6 +478,13 @@ static struct poptOption long_options[] 
-   {"ipv4",            '4', POPT_ARG_VAL,    &default_af_hint, AF_INET, 0, 0 },
-   {"ipv6",            '6', POPT_ARG_VAL,    &default_af_hint, AF_INET6, 0, 0 },
- #endif
+@@ -541,6 +561,13 @@ static struct poptOption long_options[] 
+   {"checksum-seed",    0,  POPT_ARG_INT,    &checksum_seed, 0, 0, 0 },
+   {"server",           0,  POPT_ARG_NONE,   0, OPT_SERVER, 0, 0 },
+   {"sender",           0,  POPT_ARG_NONE,   0, OPT_SENDER, 0, 0 },
 +#if HAVE_OPENSSL
 +  {"ssl",              0,  POPT_ARG_NONE,   0, OPT_USE_SSL, 0, 0},
 +  {"ssl-cert",         0,  POPT_ARG_STRING, &ssl_cert_path, OPT_USE_SSL, 0, 0},
@@ -306,25 +303,84 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +  {"ssl-key-passwd",   0,  POPT_ARG_STRING, &ssl_key_passwd, OPT_USE_SSL, 0, 0},
 +  {"ssl-ca-certs",     0,  POPT_ARG_STRING, &ssl_ca_path, OPT_USE_SSL, 0, 0},
 +#endif
-   /* All these options switch us into daemon-mode option-parsing. */
-   {"address",          0,  POPT_ARG_STRING, 0, OPT_DAEMON, 0, 0 },
+   /* All the following options switch us into daemon-mode option-parsing. */
    {"config",           0,  POPT_ARG_STRING, 0, OPT_DAEMON, 0, 0 },
-@@ -862,6 +889,12 @@ int parse_arguments(int *argc, const cha
-                       basis_dir[basis_dir_cnt++] = (char *)arg;
-                       break;
+   {"daemon",           0,  POPT_ARG_NONE,   0, OPT_DAEMON, 0, 0 },
+@@ -568,6 +595,13 @@ static void daemon_usage(enum logcode F)
+   rprintf(F," -4, --ipv4                  prefer IPv4\n");
+   rprintf(F," -6, --ipv6                  prefer IPv6\n");
+ #endif
++#if HAVE_OPENSSL
++  rprintf(F,"     --ssl                   allow socket connections to use SSL\n");
++  rprintf(F,"     --ssl-cert=FILE         path to daemon's SSL certificate\n");
++  rprintf(F,"     --ssl-key=FILE          path to daemon's SSL private key\n");
++  rprintf(F,"     --ssl-key-passwd=PASS   password for PEM-encoded private key\n");
++  rprintf(F,"     --ssl-ca-certs=FILE     path to trusted CA certificates\n");
++#endif
+   rprintf(F,"     --help                  show this help screen\n");
  
-+              case OPT_USE_SSL:
+   rprintf(F,"\n");
+@@ -594,6 +628,13 @@ static struct poptOption long_daemon_opt
+   {"protocol",         0,  POPT_ARG_INT,    &protocol_version, 0, 0, 0 },
+   {"server",           0,  POPT_ARG_NONE,   &am_server, 0, 0, 0 },
+   {"temp-dir",        'T', POPT_ARG_STRING, &tmpdir, 0, 0, 0 },
 +#if HAVE_OPENSSL
-+                      use_ssl = 1;
++  {"ssl",              0,  POPT_ARG_NONE,   0, OPT_USE_SSL, 0, 0},
++  {"ssl-cert",         0,  POPT_ARG_STRING, &ssl_cert_path, OPT_USE_SSL, 0, 0},
++  {"ssl-key",          0,  POPT_ARG_STRING, &ssl_key_path, OPT_USE_SSL, 0, 0},
++  {"ssl-key-passwd",   0,  POPT_ARG_STRING, &ssl_key_passwd, OPT_USE_SSL, 0, 0},
++  {"ssl-ca-certs",     0,  POPT_ARG_STRING, &ssl_ca_path, OPT_USE_SSL, 0, 0},
++#endif
+   {"verbose",         'v', POPT_ARG_NONE,   0, 'v', 0, 0 },
+   {"no-verbose",       0,  POPT_ARG_VAL,    &verbose, 0, 0, 0 },
+   {"no-v",             0,  POPT_ARG_VAL,    &verbose, 0, 0, 0 },
+@@ -848,6 +889,12 @@ int parse_arguments(int *argc, const cha
+                                       verbose++;
+                                       break;
++#ifdef HAVE_OPENSSL
++                              case OPT_USE_SSL:
++                                      use_ssl = 1;
++                                      break;
++#endif
++
+                               default:
+                                       rprintf(FERROR,
+                                           "rsync: %s: %s (in daemon mode)\n",
+@@ -871,6 +918,17 @@ int parse_arguments(int *argc, const cha
+                               exit_cleanup(RERR_SYNTAX);
+                       }
++#ifdef HAVE_OPENSSL
++                      if (use_ssl) {
++                              if (init_tls()) {
++                                      snprintf(err_buf, sizeof(err_buf),
++                                               "Openssl error: %s\n",
++                                               get_ssl_error());
++                                      return 0;
++                              }
++                      }
 +#endif
++
+                       *argv = poptGetArgs(pc);
+                       *argc = count_args(*argv);
+                       am_starting_up = 0;
+@@ -1082,6 +1140,12 @@ int parse_arguments(int *argc, const cha
+                       usage(FINFO);
+                       exit_cleanup(0);
++#if HAVE_OPENSSL
++              case OPT_USE_SSL:
++                      use_ssl = 1;
 +                      break;
++#endif
 +
                default:
                        /* A large opt value means that set_refuse_options()
                         * turned this option off. */
-@@ -1116,6 +1149,17 @@ int parse_arguments(int *argc, const cha
+@@ -1358,6 +1422,17 @@ int parse_arguments(int *argc, const cha
        if (delay_updates && !partial_dir)
-               partial_dir = partialdir_for_delayupdate;
+               partial_dir = tmp_partialdir;
  
 +#if HAVE_OPENSSL
 +      if (use_ssl) {
@@ -340,10 +396,10 @@ can't say if I've left any cleanup/compatibility errors in the code.
        if (inplace) {
  #ifdef HAVE_FTRUNCATE
                if (partial_dir) {
-@@ -1483,11 +1527,28 @@ char *check_for_hostspec(char *s, char *
- {
+@@ -1775,10 +1850,27 @@ char *check_for_hostspec(char *s, char *
        char *p;
        int not_host;
+       int hostlen;
 +      int url_prefix_len = sizeof URL_PREFIX - 1;
  
 -      if (port_ptr && strncasecmp(URL_PREFIX, s, strlen(URL_PREFIX)) == 0) {
@@ -365,14 +421,13 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +      }
 +      if (url_prefix_len) {
                char *path;
-               int hostlen;
 -              s += strlen(URL_PREFIX);
 +              s += url_prefix_len;
                if ((p = strchr(s, '/')) != NULL) {
                        hostlen = p - s;
                        path = p + 1;
---- orig/rsync.h       2005-03-16 02:19:30
-+++ rsync.h    2004-10-08 21:01:33
+--- old/rsync.h
++++ new/rsync.h
 @@ -32,6 +32,7 @@
  
  #define DEFAULT_LOCK_FILE "/var/run/rsyncd.lock"
@@ -381,7 +436,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
  
  #define BACKUP_SUFFIX "~"
  
-@@ -411,6 +412,11 @@ enum msgcode {
+@@ -419,6 +420,11 @@ enum msgcode {
  # define SIZEOF_INT64 SIZEOF_OFF_T
  #endif
  
@@ -393,9 +448,9 @@ can't say if I've left any cleanup/compatibility errors in the code.
  /* Starting from protocol version 26, we always use 64-bit
   * ino_t and dev_t internally, even if this platform does not
   * allow files to have 64-bit inums.  That's because the
---- orig/ssl.c 2004-10-08 19:37:22
-+++ ssl.c      2004-10-08 19:37:22
-@@ -0,0 +1,366 @@
+--- old/ssl.c
++++ new/ssl.c
+@@ -0,0 +1,370 @@
 +/* -*- c-file-style: "linux" -*-
 + * ssl.c: operations for negotiating SSL rsync connections. 
 + *
@@ -445,6 +500,10 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +static int ssl_running;
 +static int ssl_pid = -1;
 +
++#ifdef HAVE_SIGACTION
++static struct sigaction sigact;
++#endif
++
 +/**
 + * A non-interactive callback to be passed to SSL_CTX_set_default_password_cb,
 + * which merely copies the value of ssl_key_passwd into buf. This is
@@ -633,7 +692,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +              return 0;
 +      }
 +
-+      signal(SIGUSR1, tls_sigusr1);
++      SIGACTION(SIGUSR1, tls_sigusr1);
 +      ssl = SSL_new(ssl_ctx);
 +      if (!ssl)
 +              goto closed;