x/urfave-cli
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

strconv.ParseInt should be used instead of strconv.ParseUint when reading Int Flags from envvars.

Browse Source
This commit is contained in:
Sam Zaydel 2015-01-09 05:53:14 -08:00
parent aee58516e6
commit ef23aa6da2
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
flag.go
View File

@ -338,7 +338,7 @@ func (f IntFlag) Apply(set *flag.FlagSet) {
for _, envVar := range strings.Split(f.EnvVar, ",") { for _, envVar := range strings.Split(f.EnvVar, ",") {
envVar = strings.TrimSpace(envVar) envVar = strings.TrimSpace(envVar)
if envVal := os.Getenv(envVar); envVal != "" { if envVal := os.Getenv(envVar); envVal != "" {
envValInt, err := strconv.ParseUint(envVal, 10, 64) envValInt, err := strconv.ParseInt(envVal, 10, 64)
if err == nil { if err == nil {
f.Value = int(envValInt) f.Value = int(envValInt)
break break