golang/delve
1
0
Fork 0
mirror of https://github.com/go-delve/delve.git synced 2025-10-29 17:56:45 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix panic when config value set 0 (#1723)

Browse Source
This commit is contained in:
TerrySolar
2019-10-22 01:40:37 +08:00
committed by Derek Parker
parent fb3941324b
commit 52c395eb64
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/terminal/config.go
View File

@ -123,6 +123,9 @@ func configureSet(t *Term, args string) error {
if err != nil { if err != nil {
return reflect.ValueOf(nil), fmt.Errorf("argument to %q must be a number", cfgname) return reflect.ValueOf(nil), fmt.Errorf("argument to %q must be a number", cfgname)
} }
if n < 0 {
return reflect.ValueOf(nil), fmt.Errorf("argument to %q must be a number greater than zero", cfgname)
}
return reflect.ValueOf(&n), nil return reflect.ValueOf(&n), nil
case reflect.Bool: case reflect.Bool:
v := rest == "true" v := rest == "true"