>     /*
>      * Check if we are done writing to the file now.
>      */
>     if (dataCc <= 0 && tostdoutFlag == FALSE) {
>         struct utimbuf utb;
> 
>         if (close(outFd))
>             perror(outName);
> 
>         /* Set the file time */
>         utb.actime = mtime;
>         utb.modtime = mtime;
>         utime(outName, &utb);
>         /* Set the file permissions */
>         chown(outName, uid, gid);
>         chmod(outName, mode);
> 
>         outFd = -1;
>     }

I just commited a fix that gets rid of the utime(), chown(), and chmod()
calls from this part of tar.c. From what I showed, the utime, mode, and
ownership is already set once prior to this being called. Doing so again
is pointless.

Ben

-- 
 -----------=======-=-======-=========-----------=====------------=-=------
/  Ben Collins  --  ...on that fantastic voyage...  --  Debian GNU/Linux   \
`  [EMAIL PROTECTED]  --  [EMAIL PROTECTED]  --  [EMAIL PROTECTED]  '
 `---=========------=======-------------=-=-----=-===-======-------=--=---'


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to