Fix directory walker error checking

Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
Upstream-commit: 81d24e754d48ac8e9f0e4fe02befbf628179da43
Component: engine
This commit is contained in:
Tonis Tiigi
2016-05-26 14:59:38 -07:00
parent d77d575d98
commit e9a370108d
+10 -10
View File
@@ -29,6 +29,16 @@ func ValidateContextDirectory(srcPath string, excludes []string) error {
return err
}
return filepath.Walk(contextRoot, func(filePath string, f os.FileInfo, err error) error {
if err != nil {
if os.IsPermission(err) {
return fmt.Errorf("can't stat '%s'", filePath)
}
if os.IsNotExist(err) {
return nil
}
return err
}
// skip this directory/file if it's not in the path, it won't get added to the context
if relFilePath, err := filepath.Rel(contextRoot, filePath); err != nil {
return err
@@ -41,16 +51,6 @@ func ValidateContextDirectory(srcPath string, excludes []string) error {
return nil
}
if err != nil {
if os.IsPermission(err) {
return fmt.Errorf("can't stat '%s'", filePath)
}
if os.IsNotExist(err) {
return nil
}
return err
}
// skip checking if symlinks point to non-existing files, such symlinks can be useful
// also skip named pipes, because they hanging on open
if f.Mode()&(os.ModeSymlink|os.ModeNamedPipe) != 0 {