opts.IPVal returns an error on incorrect input

Signed-off-by: Solomon Hykes <solomon@docker.com>
This commit is contained in:
Solomon Hykes 2014-08-10 03:50:46 +00:00 committed by Vincent Demeester
parent a6487884e5
commit 1edb726c0b
1 changed files with 5 additions and 2 deletions

View File

@ -1,6 +1,7 @@
package opts package opts
import ( import (
"fmt"
"net" "net"
) )
@ -17,8 +18,10 @@ func NewIpOpt(ref *net.IP, defaultVal string) *IpOpt {
} }
func (o *IpOpt) Set(val string) error { func (o *IpOpt) Set(val string) error {
// FIXME: return a parse error if the value is not a valid IP? ip := net.ParseIP(val)
// We are not changing this now to preserve behavior while refactoring. if ip == nil {
return fmt.Errorf("incorrect IP format")
}
(*o.IP) = net.ParseIP(val) (*o.IP) = net.ParseIP(val)
return nil return nil
} }