[BusyBox-cvs] busybox/archival/libunarchive data_extract_all.c, 1.13, 1.14

Glenn McGrath bug1 at busybox.net
Sat Nov 15 23:44:34 UTC 2003


Update of /var/cvs/busybox/archival/libunarchive
In directory winder:/tmp/cvs-serv4932/archival/libunarchive

Modified Files:
	data_extract_all.c 
Log Message:
Fix a bug where cpio wouldnt work unless -u was specified


Index: data_extract_all.c
===================================================================
RCS file: /var/cvs/busybox/archival/libunarchive/data_extract_all.c,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -d -r1.13 -r1.14
--- data_extract_all.c	11 Sep 2003 08:32:40 -0000	1.13
+++ data_extract_all.c	15 Nov 2003 23:44:31 -0000	1.14
@@ -49,18 +49,20 @@
 	else if (archive_handle->flags & ARCHIVE_EXTRACT_NEWER) {
 		/* Remove the existing entry if its older than the extracted entry */
 		struct stat statbuf;
-		if ((lstat(file_header->name, &statbuf) == -1) && (errno != ENOENT)) {
-			bb_perror_msg_and_die("Couldnt stat old file");
+		if (lstat(file_header->name, &statbuf) == -1) {
+			if (errno != ENOENT) {
+				bb_perror_msg_and_die("Couldnt stat old file");
+			}
 		}
-		if (statbuf.st_mtime <= file_header->mtime) {
+		else if (statbuf.st_mtime <= file_header->mtime) {
 			if (!(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)) {
 				bb_error_msg("%s not created: newer or same age file exists", file_header->name);
 			}
 			data_skip(archive_handle);
 			return;
 		}
-		if ((unlink(file_header->name) == -1) && (errno != ENOENT)) {
-			bb_perror_msg_and_die("Couldnt remove old file");
+		else if ((unlink(file_header->name) == -1) && (errno != EISDIR)) {
+			bb_perror_msg_and_die("Couldnt remove old file %s", file_header->name);
 		}
 	}
 
@@ -76,22 +78,19 @@
 		switch(file_header->mode & S_IFMT) {
 			case S_IFREG: {
 				/* Regular file */
-				unlink(file_header->name);
 				dst_fd = bb_xopen(file_header->name, O_WRONLY | O_CREAT | O_EXCL);
 				archive_copy_file(archive_handle, dst_fd);
 				close(dst_fd);
 				break;
 				}
 			case S_IFDIR:
-				unlink(file_header->name);
 				res = mkdir(file_header->name, file_header->mode);
-				if ((res == -1) && !(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)) {
+				if ((errno != EISDIR) && (res == -1) && !(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)) {
 					bb_perror_msg("extract_archive: %s", file_header->name);
 				}
 				break;
 			case S_IFLNK:
 				/* Symlink */
-				unlink(file_header->name);
 				res = symlink(file_header->link_name, file_header->name);
 				if ((res == -1) && !(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)) {
 					bb_perror_msg("Cannot create symlink from %s to '%s'", file_header->name, file_header->link_name);
@@ -101,7 +100,6 @@
 			case S_IFBLK:
 			case S_IFCHR:
 			case S_IFIFO:
-				unlink(file_header->name);
 				res = mknod(file_header->name, file_header->mode, file_header->device);
 				if ((res == -1) && !(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)) {
 					bb_perror_msg("Cannot create node %s", file_header->name);




More information about the busybox-cvs mailing list