fix flag conversion to fix the security warning (#417)

mangalaman93 · web-flow · commit e62df71ba25f · 2024-11-11T12:23:47.000-08:00
Fixes https://github.com/dgraph-io/ristretto/security/code-scanning/9
diff --git a/z/flags.go b/z/flags.go
@@ -184,7 +184,7 @@ func (sf *SuperFlag) GetDuration(opt string) time.Duration {
 	}
 	if strings.Contains(val, "d") {
 		val = strings.Replace(val, "d", "", 1)
-		days, err := strconv.ParseUint(val, 0, 64)
+		days, err := strconv.ParseInt(val, 0, 64)
 		if err != nil {
 			return time.Duration(0)
 		}

Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,7 @@ func (sf *SuperFlag) GetDuration(opt string) time.Duration {`
`184`	`184`	`}`
`185`	`185`	`if strings.Contains(val, "d") {`
`186`	`186`	`val = strings.Replace(val, "d", "", 1)`
`187`		`- days, err := strconv.ParseUint(val, 0, 64)`
	`187`	`+ days, err := strconv.ParseInt(val, 0, 64)`
`188`	`188`	`if err != nil {`
`189`	`189`	`return time.Duration(0)`
`190`	`190`	`}`