0001509: tar does not remove leading / from hard link target

Notes
(0002792) vda 09-24-07 11:44	In an empty directory with busybox's executable only: $ ln busybox busybox_z $ ./busybox tar cf ../z.tar * $ mkdir z; cd z $ ../busybox tar xf ../../z.tar $ ls -li total 1528 7340110 -rwxr-xr-x 2 vda eng 775788 Sep 24 19:28 busybox 7340110 -rwxr-xr-x 2 vda eng 775788 Sep 24 19:28 busybox_z Please provide a testcase where it fails.

(0002793) vda 09-24-07 12:06	I managed to reproduce this. Please test attached patch.

(0002794) Dwight Engen 09-24-07 14:16	Sorry I wasn't clear about the test case: you must attempt to tar giving /something as the files to tar up. For example: $ mkdir /foo $ cd /foo $ touch bar $ ln bar baz $ tar cf /tmp/test.tar /foo hexdump'ing /tmp/test.tar would show foo/baz's link target to be /foo/bar. I tested your patch and it works fine, I hadn't noticed that slash skipping loop in writeFileToTarball, much cleaner place to fix. I think the first part of the third hunk is just whitespace? Thanks for the fix.

(0004434) vda 02-13-08 08:34	Fixed sometime ago.

Relationships