Merge pull request kubernetes#109256 from chendave/nullpointer

k8s-ci-robot · web-flow · commit a1a9dc066c7d · 2022-04-05T13:33:37.000-07:00
Fix the nil pointer dereference for comformance spec validation
diff --git a/test/conformance/walk.go b/test/conformance/walk.go
@@ -108,10 +108,9 @@ func main() {
 			testInfo := getTestInfo(spec)
 			if testInfo != nil {
 				testInfos = append(testInfos, testInfo)
-			}
-
-			if err := validateTestName(testInfo.CodeName); err != nil {
-				log.Fatal(err)
+				if err := validateTestName(testInfo.CodeName); err != nil {
+					log.Fatal(err)
+				}
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -108,10 +108,9 @@ func main() {`
`108`	`108`	`testInfo := getTestInfo(spec)`
`109`	`109`	`if testInfo != nil {`
`110`	`110`	`testInfos = append(testInfos, testInfo)`
`111`		`- }`
`112`		`-`
`113`		`- if err := validateTestName(testInfo.CodeName); err != nil {`
`114`		`- log.Fatal(err)`
	`111`	`+ if err := validateTestName(testInfo.CodeName); err != nil {`
	`112`	`+ log.Fatal(err)`
	`113`	`+ }`
`115`	`114`	`}`
`116`	`115`	`}`
`117`	`116`	`}`