Skip to content

Commit

Permalink
use switch instead of multiple ifs (no semantic changes)
Browse files Browse the repository at this point in the history
Signed-off-by: Michal Wasilewski <mwasilewski@gmx.com>
  • Loading branch information
mwasilew2 committed Jun 15, 2023
1 parent 3096943 commit fbbf002
Showing 1 changed file with 8 additions and 13 deletions.
21 changes: 8 additions & 13 deletions sysfs/vulnerability.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,29 +81,24 @@ func parseVulnerability(name, rawContent string) (*Vulnerability, error) {
v := &Vulnerability{CodeName: name}
rawContent = strings.TrimSpace(rawContent)
rawContentLower := strings.ToLower(rawContent)

if strings.HasPrefix(rawContentLower, notAffected) {
switch {
case strings.HasPrefix(rawContentLower, notAffected):
v.State = VulnerabilityStateNotAffected
return v, nil
}

if strings.HasPrefix(rawContentLower, vulnerable) {
case strings.HasPrefix(rawContentLower, vulnerable):
v.State = VulnerabilityStateVulnerable
m := strings.Fields(rawContent)
if len(m) > 1 {
v.Mitigation = strings.Join(m[1:], " ")
}
return v, nil
}

if strings.HasPrefix(rawContentLower, mitigation) {
case strings.HasPrefix(rawContentLower, mitigation):
v.State = VulnerabilityStateMitigation
m := strings.Fields(rawContent)
if len(m) > 1 {
v.Mitigation = strings.Join(m[1:], " ")
}
return v, nil
}
default:
return nil, fmt.Errorf("unknown vulnerability state for %s: %s", name, rawContent)

return nil, fmt.Errorf("unknown vulnerability state for %s: %s", name, rawContent)
}
return v, nil
}

0 comments on commit fbbf002

Please sign in to comment.