From fbbf0028726dd252fab7ef00a75f6da236cd2372 Mon Sep 17 00:00:00 2001 From: Michal Wasilewski Date: Mon, 12 Jun 2023 12:14:57 +0200 Subject: [PATCH] use switch instead of multiple ifs (no semantic changes) Signed-off-by: Michal Wasilewski --- sysfs/vulnerability.go | 21 ++++++++------------- 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/sysfs/vulnerability.go b/sysfs/vulnerability.go index f6d950d5..1de8f7f1 100644 --- a/sysfs/vulnerability.go +++ b/sysfs/vulnerability.go @@ -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 }