validation: add check for namespace path

Ma Shimiao · Ma Shimiao · commit c8b8c73fbfbc · 2016-08-17T11:58:49.000+08:00
Signed-off-by: Ma Shimiao &lt;mashimiao.fnst@cn.fujitsu.com&gt;
diff --git a/cmd/ocitools/validate.go b/cmd/ocitools/validate.go
@@ -337,6 +337,13 @@ func checkLinux(spec rspec.Spec, rootfs string, hostCheck bool) (msgs []string)
 			} else if spec.Linux.Namespaces[index].Type == rspec.MountNamespace {
 				mountExists = true
 			}
+		} else if hostCheck {
+			_, err := os.Stat(spec.Linux.Namespaces[index].Path)
+			if os.IsNotExist(err) {
+				msgs = append(msgs, fmt.Sprintf("Path of %v not exist", spec.Linux.Namespaces[index].Type))
+			} else if err != nil {
+				msgs = append(msgs, fmt.Sprintf("%v is invalid: %s", spec.Linux.Namespaces[index].Type, err.Error()))
+			}
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -337,6 +337,13 @@ func checkLinux(spec rspec.Spec, rootfs string, hostCheck bool) (msgs []string)`
`337`	`337`	`} else if spec.Linux.Namespaces[index].Type == rspec.MountNamespace {`
`338`	`338`	`mountExists = true`
`339`	`339`	`}`
	`340`	`+ } else if hostCheck {`
	`341`	`+ _, err := os.Stat(spec.Linux.Namespaces[index].Path)`
	`342`	`+ if os.IsNotExist(err) {`
	`343`	`+ msgs = append(msgs, fmt.Sprintf("Path of %v not exist", spec.Linux.Namespaces[index].Type))`
	`344`	`+ } else if err != nil {`
	`345`	`+ msgs = append(msgs, fmt.Sprintf("%v is invalid: %s", spec.Linux.Namespaces[index].Type, err.Error()))`
	`346`	`+ }`
`340`	`347`	`}`
`341`	`348`	`}`
`342`	`349`