Wrap firewall errors in a common struct

[mjholub]

This allows a little bit of more flexibility since now there is only one return value instead of two. The underlying logic stays the same.
Also unrelated, but I've noticed that you're not tracking go.sum. This isn't a good practice.

[gustavo-iniguez-goya]

there're a couple of errors here:

opensnitch/daemon/firewall/iptables/rules.go

Line 23 in bd8e6ff

if _, err4 = core.Exec(ipt.bin, rule); err4 != nil {

opensnitch/daemon/firewall/iptables/rules.go

Line 32 in bd8e6ff

if _, err6 = core.Exec(ipt.bin6, rule); err6 != nil {

err4 and err6 are not defined. Something in the line of var err4, err6 error would solve it.

Also "not enough return values" error here (func expects 2 values: nil, nil):

opensnitch/daemon/firewall/nftables/system.go

Line 138 in bd8e6ff

return nil

and "common" package is not used error:

opensnitch/daemon/firewall/nftables/system.go

Line 6 in bd8e6ff

"github.com/evilsocket/opensnitch/daemon/firewall/common"

[mjholub]

Seems like I've botched something when bisecting this with my other feature branch. It's my fault and I'll fix this. My apologies.

[luzpaz]

Any progress here ?

[mjholub]

I'll take a look

[mjholub]

Fixed. Ran the tests locally, they fail in the UI package, not sure why as I haven't touched it and they only print the cgo warninigs about deprecated dependencies