changeset:   90409:bc4eb1b3db5d
branch:      3.4
parent:      90407:604b74f9a07d
user:        Eric Snow <ericsnowcurrently@gmail.com>
date:        Sat Apr 19 00:13:23 2014 -0600
files:       Lib/pkgutil.py Lib/test/test_pkgutil.py Misc/NEWS
description:
Issue #21200: Return None from pkgutil.get_loader() when __spec__ is missing.


diff -r 604b74f9a07d -r bc4eb1b3db5d Lib/pkgutil.py
--- a/Lib/pkgutil.py	Fri Apr 18 17:00:19 2014 -0400
+++ b/Lib/pkgutil.py	Sat Apr 19 00:13:23 2014 -0600
@@ -461,6 +461,8 @@
         loader = getattr(module, '__loader__', None)
         if loader is not None:
             return loader
+        if getattr(module, '__spec__', None) is None:
+            return None
         fullname = module.__name__
     else:
         fullname = module_or_name
diff -r 604b74f9a07d -r bc4eb1b3db5d Lib/test/test_pkgutil.py
--- a/Lib/test/test_pkgutil.py	Fri Apr 18 17:00:19 2014 -0400
+++ b/Lib/test/test_pkgutil.py	Sat Apr 19 00:13:23 2014 -0600
@@ -1,4 +1,4 @@
-from test.support import run_unittest, unload, check_warnings
+from test.support import run_unittest, unload, check_warnings, CleanImport
 import unittest
 import sys
 import importlib
@@ -345,6 +345,23 @@
         finally:
             __loader__ = this_loader
 
+    def test_get_loader_handles_missing_spec_attribute(self):
+        name = 'spam'
+        mod = type(sys)(name)
+        del mod.__spec__
+        with CleanImport(name):
+            sys.modules[name] = mod
+            loader = pkgutil.get_loader(name)
+        self.assertIsNone(loader)
+
+    def test_get_loader_handles_spec_attribute_none(self):
+        name = 'spam'
+        mod = type(sys)(name)
+        mod.__spec__ = None
+        with CleanImport(name):
+            sys.modules[name] = mod
+            loader = pkgutil.get_loader(name)
+        self.assertIsNone(loader)
 
     def test_find_loader_avoids_emulation(self):
         with check_warnings() as w:
diff -r 604b74f9a07d -r bc4eb1b3db5d Misc/NEWS
--- a/Misc/NEWS	Fri Apr 18 17:00:19 2014 -0400
+++ b/Misc/NEWS	Sat Apr 19 00:13:23 2014 -0600
@@ -86,6 +86,8 @@
   :func:`tempfile.NamedTemporaryFile`, close the file descriptor if
   :func:`io.open` fails
 
+- Issue #21200: Return None from pkgutil.get_loader() when __spec__ is missing.
+
 - Issue #21013: Enhance ssl.create_default_context() when used for server side
   sockets to provide better security by default.