When all the links to a file have been dumped, don't forget

to free() the memory used to store the filename.

1 files changed, 3 insertions, 1 deletions

diff --git a/usr.bin/tar/write.c b/usr.bin/tar/write.c
index d2eed4bf4d23..62a3d2eda4c9 100644
--- a/usr.bin/tar/write.c
+++ b/usr.bin/tar/write.c
@@ -807,7 +807,7 @@ record_hardlink(struct bsdtar *bsdtar, struct archive_entry *entry,
 	 */
 	for (le = bsdtar->links_head; le != NULL; le = le->next) {
 		if (le->dev == st->st_dev && le->ino == st->st_ino) {
-			archive_entry_set_hardlink(entry, le->name);
+			archive_entry_copy_hardlink(entry, le->name);
 
 			/*
 			 * Decrement link count each time and release
@@ -820,6 +820,8 @@ record_hardlink(struct bsdtar *bsdtar, struct archive_entry *entry,
 					le->previous->next = le->next;
 				if (le->next != NULL)
 					le->next->previous = le->previous;
+				if (le->name != NULL)
+					free(le->name);
 				if (bsdtar->links_head == le)
 					bsdtar->links_head = le->next;
 				free(le);