Merge pull request #22 from stephen679/set_aliasing

check: creator.curSet pointer aliasing
This commit is contained in:
Vincent Batts 2016-07-20 07:40:19 +09:00 committed by GitHub
commit 61dd456524
2 changed files with 5 additions and 2 deletions

View file

@ -41,11 +41,11 @@ func Check(root string, dh *DirectoryHierarchy, keywords []string) (*Result, err
sort.Sort(byPos(creator.DH.Entries))
var result Result
for _, e := range creator.DH.Entries {
for i, e := range creator.DH.Entries {
switch e.Type {
case SpecialType:
if e.Name == "/set" {
creator.curSet = &e
creator.curSet = &creator.DH.Entries[i]
} else if e.Name == "/unset" {
creator.curSet = nil
}

View file

@ -122,6 +122,9 @@ func TestTimeComparison(t *testing.T) {
if err := os.Chtimes(fh.Name(), epoch, epoch); err != nil {
t.Fatal(err)
}
if err := os.Chtimes(dir, epoch, epoch); err != nil {
t.Fatal(err)
}
if err := fh.Close(); err != nil {
t.Error(err)
}