X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/blobdiff_plain/bd6abc4939621c455f931905f0530115eb8c443c..5c6fc4a6a3328d9995377efdc05a90987b96d9a2:/sender.c

diff --git a/sender.c b/sender.c
index e73ae62f..a1cd3f41 100644
--- a/sender.c
+++ b/sender.c
@@ -110,7 +110,7 @@ void send_files(struct file_list *flist, int f_out, int f_in)
 {
 	int fd = -1;
 	struct sum_struct *s;
-	struct map_struct *buf = NULL;
+	struct map_struct *mbuf = NULL;
 	STRUCT_STAT st;
 	char fname[MAXPATHLEN];
 	int i;
@@ -157,10 +157,11 @@ void send_files(struct file_list *flist, int f_out, int f_in)
 		stats.num_transferred_files++;
 		stats.total_transferred_size += file->length;
 
-		fname[0] = 0;
 		if (file->basedir) {
-			offset = stringjoin(fname, sizeof fname,
-					    file->basedir, "/", NULL);
+			/* N.B. We're sure that this fits, so offset is OK. */
+			offset = strlcpy(fname, file->basedir, sizeof fname);
+			if (!offset || fname[offset-1] != '/')
+				fname[offset++] = '/';
 		} else
 			offset = 0;
 		f_name_to(file, fname + offset);
@@ -216,15 +217,12 @@ void send_files(struct file_list *flist, int f_out, int f_in)
 				return;
 			}
 
-			if (st.st_size > 0) {
-				buf = map_file(fd, st.st_size);
-			} else {
-				buf = NULL;
-			}
+			mbuf = st.st_size ? map_file(fd, st.st_size) : NULL;
 
-			if (verbose > 2)
+			if (verbose > 2) {
 				rprintf(FINFO, "send_files mapped %s of size %.0f\n",
 					fname, (double)st.st_size);
+			}
 
 			write_int(f_out, i);
 
@@ -270,7 +268,7 @@ void send_files(struct file_list *flist, int f_out, int f_in)
 							}
 						}
 					}  /* end while  */
-					read_batch_delta_file( buff, MD4_SUM_LENGTH);
+					read_batch_delta_file(buff, MD4_SUM_LENGTH);
 					write_buf(f_out, buff, MD4_SUM_LENGTH);
 
 				}  /* j=i */
@@ -280,13 +278,13 @@ void send_files(struct file_list *flist, int f_out, int f_in)
 				continue;
 			}
 		} else  {
-			match_sums(f_out, s, buf, st.st_size);
+			match_sums(f_out, s, mbuf, st.st_size);
 			log_send(file, &initial_stats);
 		}
 
 		if (!read_batch) {
-			if (buf) {
-				j = unmap_file(buf);
+			if (mbuf) {
+				j = unmap_file(mbuf);
 				if (j) {
 					io_error |= IOERR_GENERAL;
 					rprintf(FERROR,