Skip to content

Commit 7e2d8d3

Browse files
committed
Handle empty ssa results
Signed-off-by: Cosmin Cojocar <[email protected]>
1 parent 074353a commit 7e2d8d3

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

analyzer.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -321,7 +321,7 @@ func (gosec *Analyzer) CheckRules(pkg *packages.Package) {
321321
// CheckAnalyzers runs analyzers on a given package.
322322
func (gosec *Analyzer) CheckAnalyzers(pkg *packages.Package) {
323323
ssaResult, err := gosec.buildSSA(pkg)
324-
if err != nil {
324+
if err != nil || ssaResult == nil {
325325
gosec.logger.Printf("Error building the SSA representation of the package %q: %s", pkg.Name, err)
326326
return
327327
}
@@ -370,8 +370,8 @@ func (gosec *Analyzer) CheckAnalyzers(pkg *packages.Package) {
370370
// buildSSA runs the SSA pass which builds the SSA representation of the package. It handles gracefully any panic.
371371
func (gosec *Analyzer) buildSSA(pkg *packages.Package) (interface{}, error) {
372372
defer func() {
373-
if r := recover(); r != nil {
374-
gosec.logger.Printf("Panic when running SSA analyser on package %q: %s", pkg.Name, r)
373+
if r := recover(); r = nil {
374+
gosec.logger.Printf("Panic when running SSA analyser on package: %s", pkg.Name)
375375
}
376376
}()
377377
ssaPass := &analysis.Pass{

0 commit comments

Comments
 (0)