Skip to content

Commit a328999

Browse files
author
youngifif
committed
fix_err_not_nil_should_return_otherwise_panic
1 parent 6976237 commit a328999

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

ip.go

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ func linkShow() {
6969
links, err := netlink.LinkList()
7070
if err != nil {
7171
fmt.Println(err)
72+
return
7273
}
7374
for _, link := range links {
7475
fmt.Println(link.Attrs())
@@ -79,10 +80,12 @@ func addrAdd(ifaddr string, ifname string) {
7980
link, err := netlink.LinkByName(ifname)
8081
if err != nil {
8182
fmt.Println(err)
83+
return
8284
}
8385
addr, err := netlink.ParseAddr(ifaddr)
8486
if err != nil {
8587
fmt.Println(err)
88+
return
8689
}
8790
err = netlink.AddrAdd(link, addr)
8891
if err != nil {
@@ -94,6 +97,7 @@ func routeShow() {
9497
links, err := netlink.LinkList()
9598
if err != nil {
9699
fmt.Println(err)
100+
return
97101
}
98102
for _, link := range links {
99103
routes, _ := netlink.RouteList(link, netlink.FAMILY_V4)
@@ -107,6 +111,7 @@ func routeAdd(dst string, gateway string, ifname string, src string) {
107111
link, err := netlink.LinkByName(ifname)
108112
if err != nil {
109113
fmt.Println(err)
114+
return
110115
}
111116
route := netlink.Route{LinkIndex: link.Attrs().Index}
112117
if dst != "default" {
@@ -129,6 +134,10 @@ func routeAdd(dst string, gateway string, ifname string, src string) {
129134

130135
func linkSetUp(ifname string) {
131136
link, err := netlink.LinkByName(ifname)
137+
if err != nil {
138+
fmt.Println(err)
139+
return
140+
}
132141
netlink.LinkSetUp(link)
133142
if err != nil {
134143
fmt.Println(err)
@@ -137,6 +146,10 @@ func linkSetUp(ifname string) {
137146

138147
func linkSetDown(ifname string) {
139148
link, err := netlink.LinkByName(ifname)
149+
if err != nil {
150+
fmt.Println(err)
151+
return
152+
}
140153
netlink.LinkSetDown(link)
141154
if err != nil {
142155
fmt.Println(err)

0 commit comments

Comments
 (0)