123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236 |
- // Copyright (C) 2017 The Syncthing Authors.
- //
- // This Source Code Form is subject to the terms of the Mozilla Public
- // License, v. 2.0. If a copy of the MPL was not distributed with this file,
- // You can obtain one at https://mozilla.org/MPL/2.0/.
- package fs
- import (
- "fmt"
- "path/filepath"
- "testing"
- "time"
- "github.com/syncthing/syncthing/lib/build"
- )
- func TestIsInternal(t *testing.T) {
- cases := []struct {
- file string
- internal bool
- }{
- {".stfolder", true},
- {".stignore", true},
- {".stversions", true},
- {".stfolder/foo", true},
- {".stignore/foo", true},
- {".stversions/foo", true},
- {".stfolderfoo", false},
- {".stignorefoo", false},
- {".stversionsfoo", false},
- {"foo.stfolder", false},
- {"foo.stignore", false},
- {"foo.stversions", false},
- {"foo/.stfolder", false},
- {"foo/.stignore", false},
- {"foo/.stversions", false},
- }
- for _, tc := range cases {
- res := IsInternal(filepath.FromSlash(tc.file))
- if res != tc.internal {
- t.Errorf("Unexpected result: IsInternal(%q): %v should be %v", tc.file, res, tc.internal)
- }
- }
- }
- func TestCanonicalize(t *testing.T) {
- type testcase struct {
- path string
- expected string
- ok bool
- }
- cases := []testcase{
- // Valid cases
- {"/bar", "bar", true},
- {"/bar/baz", "bar/baz", true},
- {"bar", "bar", true},
- {"bar/baz", "bar/baz", true},
- // Not escape attempts, but oddly formatted relative paths
- {"", ".", true},
- {"/", ".", true},
- {"/..", ".", true},
- {"./bar", "bar", true},
- {"./bar/baz", "bar/baz", true},
- {"bar/../baz", "baz", true},
- {"/bar/../baz", "baz", true},
- {"./bar/../baz", "baz", true},
- // Results in an allowed path, but does it by probing. Disallowed.
- {"../foo", "", false},
- {"../foo/bar", "", false},
- {"../foo/bar", "", false},
- {"../../baz/foo/bar", "", false},
- {"bar/../../foo/bar", "", false},
- {"bar/../../../baz/foo/bar", "", false},
- // Escape attempts.
- {"..", "", false},
- {"../", "", false},
- {"../bar", "", false},
- {"../foobar", "", false},
- {"bar/../../quux/baz", "", false},
- }
- for _, tc := range cases {
- res, err := Canonicalize(tc.path)
- if tc.ok {
- if err != nil {
- t.Errorf("Unexpected error for Canonicalize(%q): %v", tc.path, err)
- continue
- }
- exp := filepath.FromSlash(tc.expected)
- if res != exp {
- t.Errorf("Unexpected result for Canonicalize(%q): %q != expected %q", tc.path, res, exp)
- }
- } else if err == nil {
- t.Errorf("Unexpected pass for Canonicalize(%q) => %q", tc.path, res)
- continue
- }
- }
- }
- func TestFileModeString(t *testing.T) {
- var fm FileMode = 0o777
- exp := "-rwxrwxrwx"
- if fm.String() != exp {
- t.Fatalf("Got %v, expected %v", fm.String(), exp)
- }
- }
- func TestIsParent(t *testing.T) {
- test := func(path, parent string, expected bool) {
- t.Helper()
- path = filepath.FromSlash(path)
- parent = filepath.FromSlash(parent)
- if res := IsParent(path, parent); res != expected {
- t.Errorf(`Unexpected result: IsParent("%v", "%v"): %v should be %v`, path, parent, res, expected)
- }
- }
- testBoth := func(path, parent string, expected bool) {
- t.Helper()
- test(path, parent, expected)
- if build.IsWindows {
- test("C:/"+path, "C:/"+parent, expected)
- } else {
- test("/"+path, "/"+parent, expected)
- }
- }
- // rel - abs
- for _, parent := range []string{"/", "/foo", "/foo/bar"} {
- for _, path := range []string{"", ".", "foo", "foo/bar", "bas", "bas/baz"} {
- if build.IsWindows {
- parent = "C:/" + parent
- }
- test(parent, path, false)
- test(path, parent, false)
- }
- }
- // equal
- for i, path := range []string{"/", "/foo", "/foo/bar", "", ".", "foo", "foo/bar"} {
- if i < 3 && build.IsWindows {
- path = "C:" + path
- }
- test(path, path, false)
- }
- test("", ".", false)
- test(".", "", false)
- for _, parent := range []string{"", "."} {
- for _, path := range []string{"foo", "foo/bar"} {
- test(path, parent, true)
- test(parent, path, false)
- }
- }
- for _, parent := range []string{"foo", "foo/bar"} {
- for _, path := range []string{"bar", "bar/foo"} {
- testBoth(path, parent, false)
- testBoth(parent, path, false)
- }
- }
- for _, parent := range []string{"foo", "foo/bar"} {
- for _, path := range []string{"foo/bar/baz", "foo/bar/baz/bas"} {
- testBoth(path, parent, true)
- testBoth(parent, path, false)
- if build.IsWindows {
- test("C:/"+path, "D:/"+parent, false)
- }
- }
- }
- }
- // Reproduces issue 9677:
- // The combination of caching the entire case FS and moving the case FS to be
- // the outmost layer of the FS lead to the mtime FS disappearing. This is
- // because in many places we intentionally create the filesystem without access
- // to the DB and thus without the mtime FS layer. With the case FS layer
- // outside, all the inner layers are also cached - notable without an mtime FS
- // layer. Later when we do try to create an FS with DB/mtime FS, we still get
- // the cached FS without mtime FS.
- func TestRepro9677MissingMtimeFS(t *testing.T) {
- mtimeDB := make(mapStore)
- name := "Testfile"
- nameLower := UnicodeLowercaseNormalized(name)
- testTime := time.Unix(1723491493, 123456789)
- // Create a file with an mtime FS entry
- firstFS := NewFilesystem(FilesystemTypeFake, fmt.Sprintf("%v?insens=true&timeprecisionsecond=true", t.Name()), &OptionDetectCaseConflicts{}, NewMtimeOption(mtimeDB))
- // Create a file, set its mtime and check that we get the expected mtime when stat-ing.
- file, err := firstFS.Create(name)
- if err != nil {
- t.Fatal(err)
- }
- file.Close()
- err = firstFS.Chtimes(name, testTime, testTime)
- if err != nil {
- t.Fatal(err)
- }
- checkMtime := func(fs Filesystem) {
- t.Helper()
- info, err := fs.Lstat(name)
- if err != nil {
- t.Fatal(err)
- }
- if !info.ModTime().Equal(testTime) {
- t.Errorf("Expected mtime %v for %v, got %v", testTime, name, info.ModTime())
- }
- info, err = fs.Lstat(nameLower)
- if !IsErrCaseConflict(err) {
- t.Errorf("Expected case-conflict error, got %v", err)
- }
- }
- checkMtime(firstFS)
- // Now syncthing gets upgraded (or even just restarted), which resets the
- // case FS registry as it lives in memory.
- globalCaseFilesystemRegistry = caseFilesystemRegistry{caseCaches: make(map[fskey]*caseCache)}
- // This time we first create some filesystem without a database and thus no
- // mtime-FS, which is used in various places outside of the folder code. We
- // aren't actually going to do anything, this just adds an entry to the
- // caseFS cache. And that's the crucial bit: In the broken case this test is
- // reproducing, it will add the FS without mtime-FS, so all future FSes will
- // be without mtime, even if requested:
- NewFilesystem(FilesystemTypeFake, fmt.Sprintf("%v?insens=true&timeprecisionsecond=true", t.Name()), &OptionDetectCaseConflicts{})
- newFS := NewFilesystem(FilesystemTypeFake, fmt.Sprintf("%v?insens=true&timeprecisionsecond=true", t.Name()), &OptionDetectCaseConflicts{}, NewMtimeOption(mtimeDB))
- checkMtime(newFS)
- }
|