Move some code from util.c to util2.c and add sum_as_hex().
authorWayne Davison <wayned@samba.org>
Tue, 11 Jun 2013 20:35:19 +0000 (13:35 -0700)
committerWayne Davison <wayned@samba.org>
Tue, 11 Jun 2013 20:36:44 +0000 (13:36 -0700)
Makefile.in
log.c
util.c
util2.c [new file with mode: 0644]

index 1b603b5367591cf825dfaff4b35e65d86e9e2c70..40abf0cfda750194804161e1154376ec6922075c 100644 (file)
@@ -34,7 +34,7 @@ LIBOBJ=lib/wildmatch.o lib/compat.o lib/snprintf.o lib/mdfour.o lib/md5.o \
 zlib_OBJS=zlib/deflate.o zlib/inffast.o zlib/inflate.o zlib/inftrees.o \
        zlib/trees.o zlib/zutil.o zlib/adler32.o zlib/compress.o zlib/crc32.o
 OBJS1=flist.o rsync.o generator.o receiver.o cleanup.o sender.o exclude.o \
-       util.o main.o checksum.o match.o syscall.o log.o backup.o delete.o
+       util.o util2.o main.o checksum.o match.o syscall.o log.o backup.o delete.o
 OBJS2=options.o io.o compat.o hlink.o token.o uidlist.o socket.o hashtable.o \
        fileio.o batch.o clientname.o chmod.o acls.o xattrs.o
 OBJS3=progress.o pipe.o
diff --git a/log.c b/log.c
index a2e5bd28cf12d59bebd03962d4f852204fd7ed87..34a013b93fcf29b6043bbb29984d2992245bb691 100644 (file)
--- a/log.c
+++ b/log.c
@@ -683,23 +683,14 @@ static void log_formatted(enum logcode code, const char *format, const char *op,
                        if (protocol_version >= 30
                         && (iflags & ITEM_TRANSFER
                          || (always_checksum && S_ISREG(file->mode)))) {
-                               int i, x1, x2;
                                const char *sum = iflags & ITEM_TRANSFER
                                                ? sender_file_sum : F_SUM(file);
-                               c = buf2 + checksum_len*2;
-                               *c = '\0';
-                               for (i = checksum_len; --i >= 0; ) {
-                                       x1 = CVAL(sum, i);
-                                       x2 = x1 >> 4;
-                                       x1 &= 0xF;
-                                       *--c = x1 <= 9 ? x1 + '0' : x1 + 'a' - 10;
-                                       *--c = x2 <= 9 ? x2 + '0' : x2 + 'a' - 10;
-                               }
+                               n = sum_as_hex(sum);
                        } else {
                                memset(buf2, ' ', checksum_len*2);
                                buf2[checksum_len*2] = '\0';
+                               n = buf2;
                        }
-                       n = buf2;
                        break;
                case 'i':
                        if (iflags & ITEM_DELETED) {
diff --git a/util.c b/util.c
index d0dee734bfa05440abe2108603127e1d9c76f049..c943ce07d61705a44a939ca87bb3d8567979159a 100644 (file)
--- a/util.c
+++ b/util.c
@@ -114,18 +114,6 @@ void print_child_argv(const char *prefix, char **cmd)
        rprintf(FCLIENT, " (%d args)\n", cnt);
 }
 
-NORETURN void out_of_memory(const char *str)
-{
-       rprintf(FERROR, "ERROR: out of memory in %s [%s]\n", str, who_am_i());
-       exit_cleanup(RERR_MALLOC);
-}
-
-NORETURN void overflow_exit(const char *str)
-{
-       rprintf(FERROR, "ERROR: buffer overflow in %s [%s]\n", str, who_am_i());
-       exit_cleanup(RERR_MALLOC);
-}
-
 /* This returns 0 for success, 1 for a symlink if symlink time-setting
  * is not possible, or -1 for any other error. */
 int set_modtime(const char *fname, time_t modtime, uint32 mod_nsec, mode_t mode)
@@ -1305,36 +1293,6 @@ char *timestring(time_t t)
        return TimeBuf;
 }
 
-/**
- * Sleep for a specified number of milliseconds.
- *
- * Always returns TRUE.  (In the future it might return FALSE if
- * interrupted.)
- **/
-int msleep(int t)
-{
-       int tdiff = 0;
-       struct timeval tval, t1, t2;
-
-       gettimeofday(&t1, NULL);
-
-       while (tdiff < t) {
-               tval.tv_sec = (t-tdiff)/1000;
-               tval.tv_usec = 1000*((t-tdiff)%1000);
-
-               errno = 0;
-               select(0,NULL,NULL, NULL, &tval);
-
-               gettimeofday(&t2, NULL);
-               if (t2.tv_sec < t1.tv_sec)
-                       t1 = t2; /* Time went backwards, so start over. */
-               tdiff = (t2.tv_sec - t1.tv_sec)*1000 +
-                       (t2.tv_usec - t1.tv_usec)/1000;
-       }
-
-       return True;
-}
-
 /* Determine if two time_t values are equivalent (either exact, or in
  * the modification timestamp window established by --modify-window).
  *
@@ -1392,24 +1350,6 @@ int _Insure_trap_error(int a1, int a2, int a3, int a4, int a5, int a6)
 }
 #endif
 
-#define MALLOC_MAX 0x40000000
-
-void *_new_array(unsigned long num, unsigned int size, int use_calloc)
-{
-       if (num >= MALLOC_MAX/size)
-               return NULL;
-       return use_calloc ? calloc(num, size) : malloc(num * size);
-}
-
-void *_realloc_array(void *ptr, unsigned int size, size_t num)
-{
-       if (num >= MALLOC_MAX/size)
-               return NULL;
-       if (!ptr)
-               return malloc(size * num);
-       return realloc(ptr, size * num);
-}
-
 /* Take a filename and filename length and return the most significant
  * filename suffix we can find.  This ignores suffixes such as "~",
  * ".bak", ".orig", ".~1~", etc. */
diff --git a/util2.c b/util2.c
new file mode 100644 (file)
index 0000000..85b1903
--- /dev/null
+++ b/util2.c
@@ -0,0 +1,109 @@
+/*
+ * Utility routines used in rsync.
+ *
+ * Copyright (C) 1996-2000 Andrew Tridgell
+ * Copyright (C) 1996 Paul Mackerras
+ * Copyright (C) 2001, 2002 Martin Pool <mbp@samba.org>
+ * Copyright (C) 2003-2013 Wayne Davison
+ *
+ * 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
+ * the Free Software Foundation; either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, visit the http://fsf.org website.
+ */
+
+#include "rsync.h"
+#include "ifuncs.h"
+#include "itypes.h"
+#include "inums.h"
+
+extern int checksum_len;
+
+/**
+ * Sleep for a specified number of milliseconds.
+ *
+ * Always returns TRUE.  (In the future it might return FALSE if
+ * interrupted.)
+ **/
+int msleep(int t)
+{
+       int tdiff = 0;
+       struct timeval tval, t1, t2;
+
+       gettimeofday(&t1, NULL);
+
+       while (tdiff < t) {
+               tval.tv_sec = (t-tdiff)/1000;
+               tval.tv_usec = 1000*((t-tdiff)%1000);
+
+               errno = 0;
+               select(0,NULL,NULL, NULL, &tval);
+
+               gettimeofday(&t2, NULL);
+               if (t2.tv_sec < t1.tv_sec)
+                       t1 = t2; /* Time went backwards, so start over. */
+               tdiff = (t2.tv_sec - t1.tv_sec)*1000 +
+                       (t2.tv_usec - t1.tv_usec)/1000;
+       }
+
+       return True;
+}
+
+#define MALLOC_MAX 0x40000000
+
+void *_new_array(unsigned long num, unsigned int size, int use_calloc)
+{
+       if (num >= MALLOC_MAX/size)
+               return NULL;
+       return use_calloc ? calloc(num, size) : malloc(num * size);
+}
+
+void *_realloc_array(void *ptr, unsigned int size, size_t num)
+{
+       if (num >= MALLOC_MAX/size)
+               return NULL;
+       if (!ptr)
+               return malloc(size * num);
+       return realloc(ptr, size * num);
+}
+
+const char *sum_as_hex(const char *sum)
+{
+       static char buf[MAX_DIGEST_LEN*2+1];
+       int i, x1, x2;
+       char *c = buf + checksum_len*2;
+
+       assert(c - buf < (int)sizeof buf);
+
+       *c = '\0';
+
+       for (i = checksum_len; --i >= 0; ) {
+               x1 = CVAL(sum, i);
+               x2 = x1 >> 4;
+               x1 &= 0xF;
+               *--c = x1 <= 9 ? x1 + '0' : x1 + 'a' - 10;
+               *--c = x2 <= 9 ? x2 + '0' : x2 + 'a' - 10;
+       }
+
+       return buf;
+}
+
+NORETURN void out_of_memory(const char *str)
+{
+       rprintf(FERROR, "ERROR: out of memory in %s [%s]\n", str, who_am_i());
+       exit_cleanup(RERR_MALLOC);
+}
+
+NORETURN void overflow_exit(const char *str)
+{
+       rprintf(FERROR, "ERROR: buffer overflow in %s [%s]\n", str, who_am_i());
+       exit_cleanup(RERR_MALLOC);
+}