fix: check validity of id.Obj pointer before dereference (#22)

trixnz · web-flow · commit 93b73ef69e74 · 2024-06-18T20:00:34.000-04:00
diff --git a/spancheck.go b/spancheck.go
@@ -389,7 +389,7 @@ func usesCall(
 				// Selector (End, SetStatus, RecordError) hit.
 				if n.Sel.Name == selName {
 					id, ok := n.X.(*ast.Ident)
-					found = ok && id.Obj.Decl == sv.id.Obj.Decl
+					found = ok && id.Obj != nil && id.Obj.Decl == sv.id.Obj.Decl
 				}
 
 				// Check if an ignore signature matches.

Original file line number	Diff line number	Diff line change
`@@ -389,7 +389,7 @@ func usesCall(`
`389`	`389`	`// Selector (End, SetStatus, RecordError) hit.`
`390`	`390`	`if n.Sel.Name == selName {`
`391`	`391`	`id, ok := n.X.(*ast.Ident)`
`392`		`- found = ok && id.Obj.Decl == sv.id.Obj.Decl`
	`392`	`+ found = ok && id.Obj != nil && id.Obj.Decl == sv.id.Obj.Decl`
`393`	`393`	`}`
`394`	`394`
`395`	`395`	`// Check if an ignore signature matches.`