From 681af253efc1d916530cf41c4b045a89e07077c8 Mon Sep 17 00:00:00 2001
From: Adam Ierymenko <adam.ierymenko@gmail.com>
Date: Tue, 1 Jul 2014 09:27:33 -0700
Subject: [PATCH] Fix for GitHub issue #88 - actually do in the code what the
 web UI expects it to do.

---
 node/NetworkConfig.hpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/node/NetworkConfig.hpp b/node/NetworkConfig.hpp
index 28db83fe4..a6f668aac 100644
--- a/node/NetworkConfig.hpp
+++ b/node/NetworkConfig.hpp
@@ -105,11 +105,11 @@ public:
 	inline bool permitsEtherType(unsigned int etherType) const
 		throw()
 	{
-		if ((!etherType)||(etherType > 0xffff)) // sanity checks
+		if ((etherType <= 0)||(etherType > 0xffff)) // sanity checks
 			return false;
-		else if ((_etWhitelist[0] & 1)) // prsence of 0 in set inverts sense: whitelist becomes blacklist
-			return ((_etWhitelist[etherType >> 3] & (1 << (etherType & 7))) == 0);
-		else return ((_etWhitelist[etherType >> 3] & (1 << (etherType & 7))) != 0);
+		if ((_etWhitelist[0] & 1)) // presence of 0 means allow all
+			return true;
+		return ((_etWhitelist[etherType >> 3] & (1 << (etherType & 7))) != 0);
 	}
 
 	std::set<unsigned int> allowedEtherTypes() const;