Use xopen3() instead of an after-the-fact chown.

This commit is contained in:
Rob Landley 2006-08-04 17:26:58 +00:00
parent 2c55fcac08
commit daf58ef61a

View File

@ -58,7 +58,8 @@ void data_extract_all(archive_handle_t *archive_handle)
switch(file_header->mode & S_IFMT) { switch(file_header->mode & S_IFMT) {
case S_IFREG: { case S_IFREG: {
/* Regular file */ /* Regular file */
dst_fd = xopen(file_header->name, O_WRONLY | O_CREAT | O_EXCL); dst_fd = xopen3(file_header->name, O_WRONLY | O_CREAT | O_EXCL,
file_header->mode);
bb_copyfd_size(archive_handle->src_fd, dst_fd, file_header->size); bb_copyfd_size(archive_handle->src_fd, dst_fd, file_header->size);
close(dst_fd); close(dst_fd);
break; break;
@ -93,11 +94,6 @@ void data_extract_all(archive_handle_t *archive_handle)
if (!(archive_handle->flags & ARCHIVE_NOPRESERVE_OWN)) { if (!(archive_handle->flags & ARCHIVE_NOPRESERVE_OWN)) {
lchown(file_header->name, file_header->uid, file_header->gid); lchown(file_header->name, file_header->uid, file_header->gid);
} }
if (!(archive_handle->flags & ARCHIVE_NOPRESERVE_PERM) &&
(file_header->mode & S_IFMT) != S_IFLNK)
{
chmod(file_header->name, file_header->mode);
}
if (archive_handle->flags & ARCHIVE_PRESERVE_DATE) { if (archive_handle->flags & ARCHIVE_PRESERVE_DATE) {
struct utimbuf t; struct utimbuf t;