]> git.ipfire.org Git - thirdparty/libarchive.git/commitdiff
Old mkisofs made wrong "SL" System User Entries of RRIP for
authorMichihiro NAKAJIMA <ggcueroad@gmail.com>
Tue, 29 Sep 2009 08:01:36 +0000 (04:01 -0400)
committerMichihiro NAKAJIMA <ggcueroad@gmail.com>
Tue, 29 Sep 2009 08:01:36 +0000 (04:01 -0400)
symbolic files.
New test iso image is made by latest mkisofs that bug is fixed.
Keep previous test data and Its test code because we need
to check that libarchive can handle that wrong iso images.

SVN-Revision: 1478

Makefile.am
libarchive/test/CMakeLists.txt
libarchive/test/Makefile
libarchive/test/test_read_format_isorr_bz2.c
libarchive/test/test_read_format_isorr_new_bz2.c [new file with mode: 0644]
libarchive/test/test_read_format_isorr_new_bz2.iso.bz2.uu [new file with mode: 0644]

index 47144aa8402f98b84d5ad4f83b8a115b664a8951..eae57919d3dfba5f6237007aa1379361536674b2 100644 (file)
@@ -259,6 +259,7 @@ libarchive_test_SOURCES=                                    \
        libarchive/test/test_read_format_iso_gz.c               \
        libarchive/test/test_read_format_isojoliet_bz2.c        \
        libarchive/test/test_read_format_isorr_bz2.c            \
+       libarchive/test/test_read_format_isorr_new_bz2.c        \
        libarchive/test/test_read_format_isozisofs_bz2.c        \
        libarchive/test/test_read_format_mtree.c                \
        libarchive/test/test_read_format_pax_bz2.c              \
@@ -340,6 +341,7 @@ libarchive_test_EXTRA_DIST=\
        libarchive/test/test_read_format_isojoliet_bz2.iso.bz2.uu       \
        libarchive/test/test_read_format_isojolietrr_bz2.iso.bz2.uu     \
        libarchive/test/test_read_format_isorr_bz2.iso.bz2.uu           \
+       libarchive/test/test_read_format_isorr_new_bz2.iso.bz2.uu       \
        libarchive/test/test_read_format_isozisofs_bz2.iso.bz2.uu       \
        libarchive/test/test_read_format_raw.data.Z.uu                  \
        libarchive/test/test_read_format_raw.data.uu                    \
index 57f81be3ab85e8d6349de3c3854fae395dde4dc3..eea45f0613b31c7083a49d2975d1139c1171c38a 100644 (file)
@@ -58,6 +58,7 @@ IF(ENABLE_TEST)
     test_read_format_iso_gz.c
     test_read_format_isojoliet_bz2.c
     test_read_format_isorr_bz2.c
+    test_read_format_isorr_new_bz2.c
     test_read_format_isozisofs_bz2.c
     test_read_format_mtree.c
     test_read_format_pax_bz2.c
index 257c9867885f2d3ecb87a50d3fec7a12a6cd1e30..73e57b94aa1139e5d4d689b07b57a94e22e850ab 100644 (file)
@@ -53,6 +53,7 @@ TESTS= \
        test_read_format_iso_gz.c               \
        test_read_format_isojoliet_bz2.c        \
        test_read_format_isorr_bz2.c            \
+       test_read_format_isorr_new_bz2.c        \
        test_read_format_isozisofs_bz2.c        \
        test_read_format_mtree.c                \
        test_read_format_pax_bz2.c              \
index a8212dbb8973336f78d87e5e388029acc5e86aa1..0f34a0d8b93499382650deb6138461a3516ed101 100644 (file)
@@ -26,6 +26,9 @@
 __FBSDID("$FreeBSD: src/lib/libarchive/test/test_read_format_isorr_bz2.c,v 1.5 2008/09/01 05:38:33 kientzle Exp $");
 
 /*
+PLEASE use old cdrtools; mkisofs verion is 2.01.
+This version mkisofs made wrong "SL" System Use Entry of RRIP.
+
 Execute the following command to rebuild the data for this program:
    tail -n +32 test_read_format_isorr_bz2.c | /bin/sh
 
diff --git a/libarchive/test/test_read_format_isorr_new_bz2.c b/libarchive/test/test_read_format_isorr_new_bz2.c
new file mode 100644 (file)
index 0000000..9053202
--- /dev/null
@@ -0,0 +1,208 @@
+/*-
+ * Copyright (c) 2003-2007 Tim Kientzle
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR(S) ``AS IS'' AND ANY EXPRESS OR
+ * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+ * IN NO EVENT SHALL THE AUTHOR(S) BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+#include "test.h"
+
+/*
+PLEASE use latest cdrtools at least mkisofs version is 2.01.01a63 or later.
+Old version mkisofs made wrong "SL" System Use Entry of RRIP.
+
+Execute the following command to rebuild the data for this program:
+   tail -n +32 test_read_format_isorr_new_bz2.c | /bin/sh
+
+rm -rf /tmp/iso
+mkdir /tmp/iso
+mkdir /tmp/iso/dir
+echo "hello" >/tmp/iso/file
+dd if=/dev/zero count=1 bs=12345678 >>/tmp/iso/file
+ln /tmp/iso/file /tmp/iso/hardlink
+(cd /tmp/iso; ln -s file symlink)
+(cd /tmp/iso; ln -s /tmp/ symlink2)
+(cd /tmp/iso; ln -s /tmp/../ symlink3)
+(cd /tmp/iso; ln -s .././../tmp/ symlink4)
+(cd /tmp/iso; ln -s .///file symlink5)
+(cd /tmp/iso; ln -s /tmp//../ symlink6)
+TZ=utc touch -afhm -t 197001020000.01 /tmp/iso /tmp/iso/file /tmp/iso/dir
+TZ=utc touch -afhm -t 197001030000.02 /tmp/iso/symlink
+F=test_read_format_isorr_new_bz2.iso.bz2
+mkhybrid -R -uid 1 -gid 2 /tmp/iso | bzip2 > $F
+uuencode $F $F > $F.uu
+exit 1
+ */
+
+DEFINE_TEST(test_read_format_isorr_new_bz2)
+{
+       const char *refname = "test_read_format_isorr_new_bz2.iso.bz2";
+       struct archive_entry *ae;
+       struct archive *a;
+       const void *p;
+       size_t size;
+       off_t offset;
+       int i;
+       int r;
+
+       extract_reference_file(refname);
+       assert((a = archive_read_new()) != NULL);
+       r = archive_read_support_compression_bzip2(a);
+       if (r == ARCHIVE_WARN) {
+               skipping("bzip2 reading not fully supported on this platform");
+               assertEqualInt(0, archive_read_finish(a));
+               return;
+       }
+       assertEqualInt(0, r);
+       assertEqualInt(0, archive_read_support_format_all(a));
+       assertEqualInt(ARCHIVE_OK,
+           archive_read_open_filename(a, refname, 10240));
+
+       /* Retrieve each of the 8 files on the ISO image and
+        * verify that each one is what we expect. */
+       for (i = 0; i < 10; ++i) {
+               assertEqualInt(0, archive_read_next_header(a, &ae));
+
+               if (strcmp(".", archive_entry_pathname(ae)) == 0) {
+                       /* '.' root directory. */
+                       assertEqualInt(AE_IFDIR, archive_entry_filetype(ae));
+                       assertEqualInt(2048, archive_entry_size(ae));
+                       assertEqualInt(86401, archive_entry_mtime(ae));
+                       assertEqualInt(0, archive_entry_mtime_nsec(ae));
+                       assertEqualInt(86401, archive_entry_ctime(ae));
+                       assertEqualInt(0, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(0, archive_entry_uid(ae));
+                       assertEqualIntA(a, ARCHIVE_EOF,
+                           archive_read_data_block(a, &p, &size, &offset));
+                       assertEqualInt((int)size, 0);
+               } else if (strcmp("dir", archive_entry_pathname(ae)) == 0) {
+                       /* A directory. */
+                       assertEqualString("dir", archive_entry_pathname(ae));
+                       assertEqualInt(AE_IFDIR, archive_entry_filetype(ae));
+                       assertEqualInt(2048, archive_entry_size(ae));
+                       assertEqualInt(86401, archive_entry_mtime(ae));
+                       assertEqualInt(86401, archive_entry_atime(ae));
+                       assertEqualInt(2, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("file", archive_entry_pathname(ae)) == 0) {
+                       /* A regular file. */
+                       assertEqualString("file", archive_entry_pathname(ae));
+                       assertEqualInt(AE_IFREG, archive_entry_filetype(ae));
+                       assertEqualInt(12345684, archive_entry_size(ae));
+                       assertEqualInt(0,
+                           archive_read_data_block(a, &p, &size, &offset));
+                       assertEqualInt(0, offset);
+                       assertEqualMem(p, "hello\n", 6);
+                       assertEqualInt(86401, archive_entry_mtime(ae));
+                       assertEqualInt(86401, archive_entry_atime(ae));
+                       assertEqualInt(2, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("hardlink", archive_entry_pathname(ae)) == 0) {
+                       /* A hardlink to the regular file. */
+                       /* Note: If "hardlink" gets returned before "file",
+                        * then "hardlink" will get returned as a regular file
+                        * and "file" will get returned as the hardlink.
+                        * This test should tolerate that, since it's a
+                        * perfectly permissible thing for libarchive to do. */
+                       assertEqualString("hardlink", archive_entry_pathname(ae));
+                       assertEqualInt(AE_IFREG, archive_entry_filetype(ae));
+                       assertEqualString("file", archive_entry_hardlink(ae));
+                       assertEqualInt(0, archive_entry_size_is_set(ae));
+                       assertEqualInt(0, archive_entry_size(ae));
+                       assertEqualInt(86401, archive_entry_mtime(ae));
+                       assertEqualInt(86401, archive_entry_atime(ae));
+                       assertEqualInt(2, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("symlink", archive_entry_pathname(ae)) == 0) {
+                       /* A symlink to the regular file. */
+                       assertEqualInt(AE_IFLNK, archive_entry_filetype(ae));
+                       assertEqualString("file", archive_entry_symlink(ae));
+                       assertEqualInt(0, archive_entry_size(ae));
+                       assertEqualInt(172802, archive_entry_mtime(ae));
+                       assertEqualInt(172802, archive_entry_atime(ae));
+                       assertEqualInt(1, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("symlink2", archive_entry_pathname(ae)) == 0) {
+                       /* A symlink to /tmp/ (an absolute path) */
+                       assertEqualInt(AE_IFLNK, archive_entry_filetype(ae));
+                       assertEqualString("/tmp/", archive_entry_symlink(ae));
+                       assertEqualInt(0, archive_entry_size(ae));
+                       assertEqualInt(1, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("symlink3", archive_entry_pathname(ae)) == 0) {
+                       /* A symlink to /tmp/../ (with a ".." component) */
+                       assertEqualInt(AE_IFLNK, archive_entry_filetype(ae));
+                       assertEqualString("/tmp/../", archive_entry_symlink(ae));
+                       assertEqualInt(0, archive_entry_size(ae));
+                       assertEqualInt(1, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("symlink4", archive_entry_pathname(ae)) == 0) {
+                       /* A symlink to a path with ".." and "." components */
+                       assertEqualInt(AE_IFLNK, archive_entry_filetype(ae));
+                       assertEqualString(".././../tmp/",
+                           archive_entry_symlink(ae));
+                       assertEqualInt(0, archive_entry_size(ae));
+                       assertEqualInt(1, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("symlink5", archive_entry_pathname(ae)) == 0) {
+                       /* A symlink to the regular file with "/" components. */
+                       assertEqualInt(AE_IFLNK, archive_entry_filetype(ae));
+                       assertEqualString(".///file", archive_entry_symlink(ae));
+                       assertEqualInt(0, archive_entry_size(ae));
+                       assertEqualInt(172802, archive_entry_mtime(ae));
+                       assertEqualInt(172802, archive_entry_atime(ae));
+                       assertEqualInt(1, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else if (strcmp("symlink6", archive_entry_pathname(ae)) == 0) {
+                       /* A symlink to /tmp//../
+                        * (with "/" and ".." components) */
+                       assertEqualInt(AE_IFLNK, archive_entry_filetype(ae));
+                       assertEqualString("/tmp//../", archive_entry_symlink(ae));
+                       assertEqualInt(0, archive_entry_size(ae));
+                       assertEqualInt(1, archive_entry_stat(ae)->st_nlink);
+                       assertEqualInt(1, archive_entry_uid(ae));
+                       assertEqualInt(2, archive_entry_gid(ae));
+               } else {
+                       failure("Saw a file that shouldn't have been there");
+                       assertEqualString(archive_entry_pathname(ae), "");
+               }
+       }
+
+       /* End of archive. */
+       assertEqualInt(ARCHIVE_EOF, archive_read_next_header(a, &ae));
+
+       /* Verify archive format. */
+       assertEqualInt(archive_compression(a), ARCHIVE_COMPRESSION_BZIP2);
+       assertEqualInt(archive_format(a), ARCHIVE_FORMAT_ISO9660_ROCKRIDGE);
+
+       /* Close the archive. */
+       assertEqualInt(0, archive_read_close(a));
+       assertEqualInt(0, archive_read_finish(a));
+}
+
+
diff --git a/libarchive/test/test_read_format_isorr_new_bz2.iso.bz2.uu b/libarchive/test/test_read_format_isorr_new_bz2.iso.bz2.uu
new file mode 100644 (file)
index 0000000..8f71b21
--- /dev/null
@@ -0,0 +1,27 @@
+begin 644 test_read_format_isorr_bz2.iso.bz2
+M0EIH.3%!629366".K#H``-A__?__>_QUY_^Z/__?X*?OWF8DJU4`)```A2``
+M@`_!"-`#[PZUHUSNPI<Z&2(T$C!,U3]4--B`FFGE!ZC!H``(QJ,0PGJ-J'J'
+ME,`Q3`E$"-3)ID4\U3]%/4>D``R`-#0```-``:`!ZAH`!B43U!!H````:``!
+MH`````````!H<````````````````````R!%(D":GJ/4R:/4-/4/4T&FC1D!
+MIDR9`!H:`R#0```>IDPZ"IJ"51(^XL`L)`>(`<*0^$EX#Q>,#L:MV;5JVQV=
+M*'5(<F)0D01`PCA0\&``X\Y>F$`SB``7_P@HB`#<M26^J`7]<1:4`MLA4$1`
+M&T,`3B%8$6(ME$>N;1`C(A33(41$0!K0@8()6IU:$%H"`*U$@ASD4\7@4H:M
+M/;H$!]NVG>'?XLX(6*(:("IE4D=H`(;^!@D$0)=;MV[0(H(@#JX(V7;!>#$8
+MK(42:.-CC&QVZCLE;ZQ+F9`,,8%P%@40:8`*S)0.,C>R!,"*A_,='-,F%*QI
+ME-&I>C%@(`28!"8HE!$((`2`R"09F"02"9Q3%\ADPX2<#B(VB6E3DAM8'S14
+M!+83_"+C9]4\\#VJ:XJC1BE)&05\OI8\]C"0RD2#IJ,.1,D008)*16AD7$9!
+M0HST*KC0&A9T:"F4DRH3",`/4\1=/][@1O?OOM'D;EQ$6KD>_-5C2%&UQ&JK
+M4RHJJDN*QQYO3Y/U52T@[C0-ZT:,2E,,*8P2DW"02E(D$,$I(/X`I)=,""-U
+M*EVEF0UF2MZMQ>/R>7TDF*_;G8$%BB\&X3@T@*&<FDPJ65-+V.(01(-T;S*[
+M>GL$<13YU4-BQ*4BL9XO*#`22@]T)]S(Q0&1410C#$P`+O;PR(PG>R,J)2/S
+MEKA(_@KQY]83UZ6(JM-P`[<40P*1;U8*97D')9[4=23%Y;HRGO66:_,45O@L
+M><RTQ8X1<)*4NK<2MU2K=$YZ5J"BBCD$E+?24,V>R+2=:M&U28';7DOGQ7W0
+M?D:EQ\A?`6`TH!PZU.R>:A$PCQ*KV-2Q9H3JA\!TD4!<2-N7L'XT!'R(H^AB
+M)^3]JJ#QKK54V!EZQT@YP^Q5&+?#(K%)AH&H8Z%]!@T.OE`]>32]-%Q\QB#+
+M\',@44"HRUPP9+MYL8.>\Q[@6&<,5!MNU%9L(KK&9C[8\DR^CY9=]ED*HU+J
+M,P9Q=RA4U'R+M#7B\,S_$=MBFS*`KD%*UQRE+"S-9<I,21_X'2#=.`^1`N;8
+M'5/'6)5"#L*F/MTAAJ^EJS9+Y._D>!=DZ2419IV`S`<T(0U!S@LXGRV=A=2O
+M8"Q1A\J5X-SKA:V]%<8*7L.:$R^MS9P,_K!."^&0YA2-:R'P\ANA(I4"_0:Z
+HV(<0^$"YBD)"9%)`@D"(`V7/!UH`<DYY9,HRSH&N+N2*<*$@P1U8=```
+`
+end