From 16d00870eff7efee2692a32bd6c5aaa56aac5e9a Mon Sep 17 00:00:00 2001 From: Derek McGowan Date: Tue, 28 Nov 2017 16:18:15 -0800 Subject: [PATCH] Add test for prefix header issue Signed-off-by: Derek McGowan --- archive/issues_test.go | 47 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 47 insertions(+) create mode 100644 archive/issues_test.go diff --git a/archive/issues_test.go b/archive/issues_test.go new file mode 100644 index 000000000..65e9f2b6c --- /dev/null +++ b/archive/issues_test.go @@ -0,0 +1,47 @@ +package archive + +import ( + "bytes" + "context" + "io/ioutil" + "os" + "path/filepath" + "strings" + "testing" + + "github.com/containerd/containerd/archive/compression" + "github.com/containerd/containerd/testutil" +) + +// TestPrefixHeaderReadable tests that files that could be created with the +// version of this package that was built with <=go17 are still readable. +func TestPrefixHeaderReadable(t *testing.T) { + testutil.RequiresRoot(t) + + // https://gist.github.com/stevvooe/e2a790ad4e97425896206c0816e1a882#file-out-go + var testFile = []byte("\x1f\x8b\x08\x08\x44\x21\x68\x59\x00\x03\x74\x2e\x74\x61\x72\x00\x4b\xcb\xcf\x67\xa0\x35\x30\x80\x00\x86\x06\x10\x47\x01\xc1\x37\x40\x00\x54\xb6\xb1\xa1\xa9\x99\x09\x48\x25\x1d\x40\x69\x71\x49\x62\x91\x02\xe5\x76\xa1\x79\x84\x21\x91\xd6\x80\x72\xaf\x8f\x82\x51\x30\x0a\x46\x36\x00\x00\xf0\x1c\x1e\x95\x00\x06\x00\x00") + + tmpDir, err := ioutil.TempDir("", "prefix-test") + if err != nil { + t.Fatal(err) + } + defer os.RemoveAll(tmpDir) + + r, err := compression.DecompressStream(bytes.NewReader(testFile)) + if err != nil { + t.Fatal(err) + } + defer r.Close() + _, err = Apply(context.Background(), tmpDir, r) + if err != nil { + t.Fatal(err) + } + + baseName := "foo" + pth := strings.Repeat("a", 100-len(baseName)) + "/" + baseName + + _, err = os.Lstat(filepath.Join(tmpDir, pth)) + if err != nil { + t.Fatal(err) + } +}