Fixed a build problem and a few bugs that crept into the
authorWayne Davison <wayned@samba.org>
Fri, 8 Oct 2004 21:04:06 +0000 (21:04 +0000)
committerWayne Davison <wayned@samba.org>
Fri, 8 Oct 2004 21:04:06 +0000 (21:04 +0000)
implementation.

openssl-support.diff

index 280a7d0..870f24c 100644 (file)
@@ -37,8 +37,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-08-13 07:18:58
-+++ Makefile.in        2004-07-03 20:22:28
+--- orig/Makefile.in   2004-10-08 20:16:26
++++ Makefile.in        2004-10-08 20:17:06
 @@ -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 \
@@ -46,7 +46,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
 -OBJS=$(OBJS1) $(OBJS2) $(OBJS3) $(DAEMON_OBJ) $(LIBOBJ) $(ZLIBOBJ) @BUILD_POPT@
 +OBJS=$(OBJS1) $(OBJS2) $(OBJS3) $(DAEMON_OBJ) $(LIBOBJ) $(ZLIBOBJ) @BUILD_POPT@ @SSL_OBJS@
  
- TLS_OBJ = tls.o syscall.o lib/permstring.o
+ TLS_OBJ = tls.o syscall.o lib/compat.o lib/permstring.o
  
 --- orig/cleanup.c     2004-07-29 16:08:03
 +++ cleanup.c  2004-07-03 20:22:28
@@ -73,7 +73,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
                rprintf(FINFO,"_exit_cleanup(code=%d, file=%s, line=%d): entered\n",
                        code, file, line);
 --- orig/clientserver.c        2004-08-02 02:29:16
-+++ clientserver.c     2004-07-03 20:22:28
++++ clientserver.c     2004-10-08 20:44:59
 @@ -46,6 +46,9 @@ extern int io_timeout;
  extern int orig_umask;
  extern int no_detach;
@@ -117,7 +117,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +                              return -1;
 +                      }
 +                      if (strncmp(line, "@ERROR", 6) == 0) {
-+                              rprintf(FERROR, "rsync: ssl connection denied\n");
++                              rprintf(FERROR, "%s\n", line);
 +                              return -1;
 +                      }
 +                      if (strcmp(line, "@RSYNCD: starttls") == 0) {
@@ -168,7 +168,17 @@ 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 */
-@@ -555,6 +604,20 @@ int start_daemon(int f_in, int f_out)
+@@ -546,6 +595,9 @@ int start_daemon(int f_in, int f_out)
+       if (protocol_version > remote_protocol)
+               protocol_version = remote_protocol;
++#if HAVE_OPENSSL
++retry:
++#endif
+       line[0] = 0;
+       if (!read_line(f_in, line, sizeof line - 1))
+               return -1;
+@@ -555,6 +607,20 @@ int start_daemon(int f_in, int f_out)
                return -1;
        }
  
@@ -182,14 +192,14 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +              }
 +              f_in = get_tls_rfd();
 +              f_out = get_tls_wfd();
-+              continue;
++              goto retry;
 +      }
 +#endif
 +
        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  2004-09-23 17:42:07
+--- orig/configure.in  2004-10-06 00:12:16
 +++ configure.in       2004-07-03 20:22:28
 @@ -271,6 +271,21 @@ yes
        AC_SEARCH_LIBS(getaddrinfo, inet6)
@@ -213,8 +223,8 @@ 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/main.c        2004-09-29 17:58:26
-+++ main.c     2004-07-15 02:40:51
+--- orig/main.c        2004-10-08 20:16:26
++++ main.c     2004-10-08 20:15:28
 @@ -56,6 +56,9 @@ extern int write_batch;
  extern int batch_fd;
  extern int batch_gen_fd;
@@ -225,73 +235,110 @@ can't say if I've left any cleanup/compatibility errors in the code.
  extern pid_t cleanup_child_pid;
  extern char *files_from;
  extern char *remote_filesfrom_file;
-@@ -771,18 +774,32 @@ static int start_client(int argc, char *
-       pid_t pid;
-       int f_in,f_out;
-       int rc;
-+      int url_prefix = strlen(URL_PREFIX);
-       /* Don't clobber argv[] so that ps(1) can still show the right
-        * command line. */
+@@ -777,33 +780,48 @@ static int start_client(int argc, char *
        if ((rc = copy_argv(argv)))
                return rc;
  
-+      if (strncasecmp(URL_PREFIX, argv[0], url_prefix) != 0 && !read_batch) {
-+#ifdef HAVE_OPENSSL
-+              url_prefix = strlen(SSL_URL_PREFIX);
-+              if (strncasecmp(SSL_URL_PREFIX, argv[0], url_prefix) != 0)
-+                      url_prefix = 0;
-+              else {
-+                      if (!use_ssl)
-+                              init_tls();
-+                      use_ssl = 1;
-+              }
-+#else
-+              url_prefix = 0;
-+#endif
-+      }
-       /* rsync:// always uses rsync server over direct socket connection */
+-      /* rsync:// always uses rsync server over direct socket connection */
 -      if (strncasecmp(URL_PREFIX, argv[0], strlen(URL_PREFIX)) == 0
 -          && !read_batch) {
-+      if (url_prefix) {
-               char *host, *path;
+-              char *host, *path;
++      if (!read_batch) { /* for read_batch, NO source is specified */
++              int url_prefix_len = sizeof URL_PREFIX - 1;
  
 -              host = argv[0] + strlen(URL_PREFIX);
-+              host = argv[0] + url_prefix;
-               p = strchr(host,'/');
-               if (p) {
-                       *p = '\0';
-@@ -835,12 +852,27 @@ static int start_client(int argc, char *
+-              p = strchr(host,'/');
+-              if (p) {
+-                      *p = '\0';
+-                      path = p+1;
+-              } else
+-                      path = "";
+-              if (*host == '[' && (p = strchr(host, ']')) != NULL) {
+-                      host++;
+-                      *p++ = '\0';
+-                      if (*p != ':')
+-                              p = NULL;
+-              } else
+-                      p = strchr(host, ':');
+-              if (p) {
+-                      rsync_port = atoi(p+1);
+-                      *p = '\0';
++              /* rsync:// always uses rsync server over direct socket connection */
++              if (strncasecmp(URL_PREFIX, argv[0], url_prefix_len) != 0) {
++#ifdef HAVE_OPENSSL
++                      url_prefix_len = sizeof SSL_URL_PREFIX - 1;
++                      if (strncasecmp(SSL_URL_PREFIX, argv[0], url_prefix_len) != 0)
++                              url_prefix_len = 0;
++                      else {
++                              if (!use_ssl)
++                                      init_tls();
++                              use_ssl = 1;
++                      }
++#else
++                      url_prefix_len = 0;
++#endif
++              }
++              if (url_prefix_len) {
++                      char *host, *path;
++
++                      host = argv[0] + url_prefix_len;
++                      p = strchr(host,'/');
++                      if (p) {
++                              *p = '\0';
++                              path = p+1;
++                      } else
++                              path = "";
++                      if (*host == '[' && (p = strchr(host, ']')) != NULL) {
++                              host++;
++                              *p++ = '\0';
++                              if (*p != ':')
++                                      p = NULL;
++                      } else
++                              p = strchr(host, ':');
++                      if (p) {
++                              rsync_port = atoi(p+1);
++                              *p = '\0';
++                      }
++                      return start_socket_client(host, path, argc-1, argv+1);
+               }
+-              return start_socket_client(host, path, argc-1, argv+1);
+-      }
+-      if (!read_batch) { /* for read_batch, NO source is specified */
+               p = find_colon(argv[0]);
+               if (p) { /* source is remote */
+                       if (remote_filesfrom_file
+@@ -835,12 +853,26 @@ static int start_client(int argc, char *
                        argv++;
                } else { /* source is local */
                        am_sender = 1;
-+                      url_prefix = strlen(URL_PREFIX);
-+                      if (strncasecmp(URL_PREFIX, argv[0], url_prefix) != 0) {
+-
++                      url_prefix_len = sizeof URL_PREFIX - 1;
+                       /* rsync:// destination uses rsync server over direct socket */
+-                      if (strncasecmp(URL_PREFIX, argv[argc-1], strlen(URL_PREFIX)) == 0) {
++                      if (strncasecmp(URL_PREFIX, argv[argc-1], url_prefix_len) != 0) {
 +#ifdef HAVE_OPENSSL
-+                              url_prefix = strlen(SSL_URL_PREFIX);
-+                              if (strncasecmp(SSL_URL_PREFIX, argv[0], url_prefix) != 0)
-+                                      url_prefix = 0;
++                              url_prefix_len = sizeof SSL_URL_PREFIX - 1;
++                              if (strncasecmp(SSL_URL_PREFIX, argv[argc-1], url_prefix_len) != 0)
++                                      url_prefix_len = 0;
 +                              else {
 +                                      if (!use_ssl)
 +                                              init_tls();
 +                                      use_ssl = 1;
 +                              }
 +#else
-+                              url_prefix = 0;
++                              url_prefix_len = 0;
 +#endif
 +                      }
-                       /* rsync:// destination uses rsync server over direct socket */
--                      if (strncasecmp(URL_PREFIX, argv[argc-1], strlen(URL_PREFIX)) == 0) {
-+                      if (url_prefix) {
++                      if (url_prefix_len) {
                                char *host, *path;
  
 -                              host = argv[argc-1] + strlen(URL_PREFIX);
-+                              host = argv[argc-1] + url_prefix;
++                              host = argv[argc-1] + url_prefix_len;
                                p = strchr(host,'/');
                                if (p) {
                                        *p = '\0';
---- orig/options.c     2004-09-23 17:42:07
+--- orig/options.c     2004-10-08 20:16:26
 +++ options.c  2004-07-16 20:19:50
 @@ -135,6 +135,14 @@ int quiet = 0;
  int always_checksum = 0;
@@ -409,7 +456,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
                bwlimit_writemax = (size_t)bwlimit * 128;
                if (bwlimit_writemax < 512)
 --- orig/rsync.h       2004-09-22 08:47:31
-+++ rsync.h    2004-07-03 20:22:28
++++ rsync.h    2004-10-08 21:01:33
 @@ -32,6 +32,7 @@
  
  #define DEFAULT_LOCK_FILE "/var/run/rsyncd.lock"
@@ -430,8 +477,8 @@ 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-07-02 21:44:19
-+++ ssl.c      2004-07-02 21:44:19
+--- orig/ssl.c 2004-10-08 19:37:22
++++ ssl.c      2004-10-08 19:37:22
 @@ -0,0 +1,366 @@
 +/* -*- c-file-style: "linux" -*-
 + * ssl.c: operations for negotiating SSL rsync connections. 
@@ -489,7 +536,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
 + */
 +static int default_password_cb(char *buf, int n, UNUSED(int f), UNUSED(void *u))
 +{
-+      if (ssl_key_passwd == NULL || n < strlen(ssl_key_passwd))
++      if (ssl_key_passwd == NULL || n < (int)strlen(ssl_key_passwd))
 +              return 0;
 +      strncpy(buf, ssl_key_passwd, n-1);
 +      return strlen(ssl_key_passwd);
@@ -498,7 +545,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +/**
 + * If verbose, this method traces the status of the SSL handshake.
 + */
-+static void info_callback(SSL *ssl, int cb, int val)
++static void info_callback(const SSL *ssl, int cb, int val)
 +{
 +      char buf[128];
 +      char *cbs;
@@ -551,7 +598,7 @@ can't say if I've left any cleanup/compatibility errors in the code.
 +      if (verbose > 2) {
 +              rprintf(FLOG, "SSL: info_callback(%p,%s,%d)\n", ssl, cbs, val);
 +              if (cb == SSL_CB_HANDSHAKE_DONE) {
-+                      SSL_CIPHER_description(SSL_get_current_cipher(ssl),
++                      SSL_CIPHER_description(SSL_get_current_cipher((SSL*)ssl),
 +                                             buf, sizeof buf);
 +                      rprintf(FLOG, "SSL: cipher: %s", buf);
 +              }