lourenco/mgmt
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

engine: resources: net: Clean up weird error handling

Browse Source 
Not sure how this happened.
This commit is contained in:
James Shubin
2023-08-08 00:03:36 -04:00
parent b87d70c71c
commit 63f05e12ca
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
engine/resources/net.go
View File

@@ -188,16 +188,15 @@ func (obj *NetRes) Init(init *engine.Init) error {
// Close cleans up when we're done. // Close cleans up when we're done.
func (obj *NetRes) Close() error { func (obj *NetRes) Close() error {
var errList error
if obj.socketFile == "/" { if obj.socketFile == "/" {
return fmt.Errorf("socket file should not be the root path") return fmt.Errorf("socket file should not be the root path")
} }
if obj.socketFile != "" { // safety if obj.socketFile != "" { // safety
errList = errwrap.Append(errList, os.Remove(obj.socketFile)) if err := os.Remove(obj.socketFile); err != nil {
return err
}
} }
return nil
return errList
} }
// Watch listens for events from the specified interface via a netlink socket. // Watch listens for events from the specified interface via a netlink socket.