[PATCH 6/9] tests: Fix potentially referencing non existing attribute

Jonathan Afek jonathan at wizery.com
Thu May 19 06:06:47 PDT 2016


In wpaspy.py in the Ctrl object constructor there is a try/except.
In the except part the code references the s attribute of the
object. This attribute is only created later in the try part.
If an exception occurs before the attribute creation then
the except part references a non existing attribute.
Fix that by assigning None to the s attribute at the beginning
of the try part.

Signed-off-by: Jonathan Afek <jonathanx.afek at intel.com>
---
 wpaspy/wpaspy.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/wpaspy/wpaspy.py b/wpaspy/wpaspy.py
index 809b4ce..c2aace0 100644
--- a/wpaspy/wpaspy.py
+++ b/wpaspy/wpaspy.py
@@ -44,6 +44,7 @@ class Ctrl:
                 raise
         else:
             try:
+                self.s = None
                 ai_list = socket.getaddrinfo(path, port, socket.AF_INET,
                                              socket.SOCK_DGRAM)
                 for af, socktype, proto, cn, sockaddr in ai_list:
-- 
1.9.1




More information about the Hostap mailing list