From 79b9b53dbea59627d43ec7ee4eb2dd54777460be Mon Sep 17 00:00:00 2001
From: Sunny <ratsunny@gmail.com>
Date: Sun, 1 Feb 2015 01:57:17 +0800
Subject: [PATCH] Never process empty string and prevent infinite loop

If user provide an empty string as network range, inet_pton will
treate it as an IPv6 unspecified address, it seems a bug but I
can't confirm. Then empty string will be converted to 0, 0 & 1
always be zero, so it caused dead loop.
---
 shadowsocks/common.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/shadowsocks/common.py b/shadowsocks/common.py
index 68ca025..d4a9a67 100644
--- a/shadowsocks/common.py
+++ b/shadowsocks/common.py
@@ -195,6 +195,8 @@ class IPNetwork(object):
         map(self.add_network, addrs)
 
     def add_network(self, addr):
+        if addr is "":
+            return
         block = addr.split('/')
         addr_family = is_ip(block[0])
         addr_len = IPNetwork.ADDRLENGTH[addr_family]
@@ -207,7 +209,7 @@ class IPNetwork(object):
             raise SyntaxError("Not a valid CIDR notation: %s" % addr)
         if len(block) is 1:
             prefix_size = 0
-            while ((ip & 1) == 0):
+            while (ip & 1) == 0 and ip is not 0:
                 ip >>= 1
                 prefix_size += 1
             logging.warn("You did't specify CIDR routing prefix size for %s, "