X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/blobdiff_plain/13c5b463610baf30f9af52d32a7da6de1bdbaa53..ac1d2d338450eb005abf03002f5784097caf7e19:/hlink.c

diff --git a/hlink.c b/hlink.c
index 988de701..3ee84335 100644
--- a/hlink.c
+++ b/hlink.c
@@ -22,6 +22,7 @@
 
 extern int dry_run;
 extern int verbose;
+extern int make_backups;
 
 #if SUPPORT_HARD_LINKS
 static int hlink_compare(struct file_struct **file1, struct file_struct **file2)
@@ -38,34 +39,49 @@ static int hlink_compare(struct file_struct **file1, struct file_struct **file2)
 	return file_compare(file1, file2);
 }
 
-struct file_struct **hlink_list;
-int hlink_count;
+static struct file_struct **hlink_list;
+static int hlink_count;
 
 #define LINKED(p1,p2) ((p1)->F_DEV == (p2)->F_DEV \
 		    && (p1)->F_INODE == (p2)->F_INODE)
 
 /* Analyze the data in the hlink_list[], remove items that aren't multiply
  * linked, and replace the dev+inode data with the hlindex+next linked list. */
-static void link_idev_data(void)
+static void link_idev_data(struct file_list *flist)
 {
 	struct file_struct *head;
 	int from, to, start;
 
+	alloc_pool_t hlink_pool;
+	alloc_pool_t idev_pool = flist->hlink_pool;
+
+	hlink_pool = pool_create(128 * 1024, sizeof (struct hlink),
+	    out_of_memory, POOL_INTERN);
+
 	for (from = to = 0; from < hlink_count; from++) {
 		start = from;
 		head = hlink_list[start];
 		while (from < hlink_count-1
 		    && LINKED(hlink_list[from], hlink_list[from+1])) {
+			pool_free(idev_pool, 0, hlink_list[from]->link_u.idev);
+			hlink_list[from]->link_u.links = pool_talloc(hlink_pool,
+			    struct hlink, 1, "hlink_list");
+
 			hlink_list[from]->F_HLINDEX = to;
 			hlink_list[from]->F_NEXT = hlink_list[from+1];
 			from++;
 		}
 		if (from > start) {
+			pool_free(idev_pool, 0, hlink_list[from]->link_u.idev);
+			hlink_list[from]->link_u.links = pool_talloc(hlink_pool,
+			    struct hlink, 1, "hlink_list");
+
 			hlink_list[from]->F_HLINDEX = to;
 			hlink_list[from]->F_NEXT = head;
 			hlink_list[from]->flags |= FLAG_HLINK_EOL;
 			hlink_list[to++] = head;
 		} else {
+			pool_free(idev_pool, 0, head->link_u.idev);
 			head->link_u.idev = NULL;
 		}
 	}
@@ -73,12 +89,16 @@ static void link_idev_data(void)
 	if (!to) {
 		free(hlink_list);
 		hlink_list = NULL;
+		pool_destroy(hlink_pool);
+		hlink_pool = NULL;
 	} else {
 		hlink_count = to;
 		if (!(hlink_list = realloc_array(hlink_list,
-					struct file_struct *, hlink_count)))
+		    struct file_struct *, hlink_count)))
 			out_of_memory("init_hard_links");
 	}
+	flist->hlink_pool = hlink_pool;
+	pool_destroy(idev_pool);
 }
 #endif
 
@@ -103,26 +123,26 @@ void init_hard_links(struct file_list *flist)
 	}
 
 	qsort(hlink_list, hlink_count,
-	      sizeof(hlink_list[0]), (int (*)()) hlink_compare);
+	    sizeof hlink_list[0], (int (*)()) hlink_compare);
 
 	if (!hlink_count) {
 		free(hlink_list);
 		hlink_list = NULL;
 	} else
-		link_idev_data();
+		link_idev_data(flist);
 #endif
 }
 
 int hard_link_check(struct file_struct *file, int skip)
 {
-	if (!file->link_u.links)
+	if (!hlink_list || !file->link_u.links)
 		return 0;
 	if (skip && !(file->flags & FLAG_HLINK_EOL))
 		hlink_list[file->F_HLINDEX] = file->F_NEXT;
 	if (hlink_list[file->F_HLINDEX] != file) {
 		if (verbose > 1) {
 			rprintf(FINFO, "\"%s\" is a hard link\n",
-				f_name(file));
+			    f_name(file));
 		}
 		return 1;
 	}
@@ -133,12 +153,12 @@ int hard_link_check(struct file_struct *file, int skip)
 static void hard_link_one(char *hlink1, char *hlink2)
 {
 	if (do_link(hlink1, hlink2)) {
-		if (verbose > 0) {
+		if (verbose) {
 			rprintf(FINFO, "link %s => %s failed: %s\n",
-				hlink2, hlink1, strerror(errno));
+			    hlink2, hlink1, strerror(errno));
 		}
 	}
-	else if (verbose > 0)
+	else if (verbose)
 		rprintf(FINFO, "%s => %s\n", hlink2, hlink1);
 }
 #endif
@@ -171,12 +191,15 @@ void do_hard_links(void)
 				if (st2.st_dev == st1.st_dev
 				    && st2.st_ino == st1.st_ino)
 					continue;
-				if (robust_unlink(hlink2)) {
+				if (make_backups) {
+					if (!make_backup(hlink2))
+						continue;
+				} else if (robust_unlink(hlink2)) {
 					if (verbose > 0) {
 						rprintf(FINFO,
-							"unlink %s failed: %s\n",
-							full_fname(hlink2), 
-							strerror(errno));
+						    "unlink %s failed: %s\n",
+						    full_fname(hlink2), 
+						    strerror(errno));
 					}
 					continue;
 				}