diff options
| author | baude <bbaude@redhat.com> | 2019-07-08 13:20:17 -0500 |
|---|---|---|
| committer | baude <bbaude@redhat.com> | 2019-07-10 15:52:17 -0500 |
| commit | e053e0e05ecd884067125627f0006d1b6e19226e (patch) | |
| tree | 9e5bcca7703a041b23c2e3c9fcfe928e8bca2962 /pkg/firewall/firewalld.go | |
| parent | 81e722d08617ee19235bf57de6d86124e6b4574a (diff) | |
| download | podman-e053e0e05ecd884067125627f0006d1b6e19226e.tar.gz podman-e053e0e05ecd884067125627f0006d1b6e19226e.tar.bz2 podman-e053e0e05ecd884067125627f0006d1b6e19226e.zip | |
first pass of corrections for golangci-lint
Signed-off-by: baude <bbaude@redhat.com>
Diffstat (limited to 'pkg/firewall/firewalld.go')
| -rw-r--r-- | pkg/firewall/firewalld.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/pkg/firewall/firewalld.go b/pkg/firewall/firewalld.go index 32c2337a0..15e845cb7 100644 --- a/pkg/firewall/firewalld.go +++ b/pkg/firewall/firewalld.go @@ -18,6 +18,7 @@ package firewall import ( "fmt" + "github.com/sirupsen/logrus" "strings" "github.com/godbus/dbus" @@ -113,7 +114,9 @@ func (fb *fwdBackend) Del(conf *FirewallNetConf) error { // Remove firewalld rules which assigned the given source IP to the given zone firewalldObj := fb.conn.Object(firewalldName, firewalldPath) var res string - firewalldObj.Call(firewalldZoneInterface+"."+firewalldRemoveSourceMethod, 0, getFirewalldZone(conf), ipStr).Store(&res) + if err := firewalldObj.Call(firewalldZoneInterface+"."+firewalldRemoveSourceMethod, 0, getFirewalldZone(conf), ipStr).Store(&res); err != nil { + logrus.Errorf("unable to store firewallobj") + } } return nil } |