diff --git a/fail2ban/server/server.py b/fail2ban/server/server.py index 8dc1b761..ca06973d 100644 --- a/fail2ban/server/server.py +++ b/fail2ban/server/server.py @@ -327,9 +327,8 @@ class Server: def isAlive(self, jailnum=None): if jailnum is not None and len(self.__jails) != jailnum: return 0 - for j in self.__jails: - j = self.__jails[j] - if not j.isAlive(): + for jail in self.__jails.values(): + if not jail.isAlive(): return 0 return 1 diff --git a/fail2ban/tests/filtertestcase.py b/fail2ban/tests/filtertestcase.py index 7557f520..ab0f0462 100644 --- a/fail2ban/tests/filtertestcase.py +++ b/fail2ban/tests/filtertestcase.py @@ -1231,9 +1231,9 @@ class DNSUtilsNetworkTests(unittest.TestCase): self.assertEqual(res, []) def testIpToName(self): + unittest.F2B.SkipIfNoNetwork() res = DNSUtils.ipToName('8.8.4.4') self.assertEqual(res, 'google-public-dns-b.google.com') - unittest.F2B.SkipIfNoNetwork() # invalid ip (TEST-NET-1 according to RFC 5737) res = DNSUtils.ipToName('192.0.2.0') self.assertEqual(res, None)