Don't use "exec" in find_class(). It's slow, unnecessary, and (as AMK

gvanrossum · gvanrossum · commit 397032aa46e8 · 1999-03-25T21:58:59.000Z
points out) it doesn't work in JPython Applets.
diff --git a/Lib/pickle.py b/Lib/pickle.py
@@ -661,15 +661,14 @@ def load_global(self):
     dispatch[GLOBAL] = load_global
 
     def find_class(self, module, name):
-        env = {}
-
         try:
-            exec 'from %s import %s' % (module, name) in env
-        except ImportError:
+            __import__(module)
+            mod = sys.modules[module]
+            klass = getattr(mod, name)
+        except (ImportError, KeyError, AttributeError):
             raise SystemError, \
                   "Failed to import class %s from module %s" % \
                   (name, module)
-        klass = env[name]
         return klass
 
     def load_reduce(self):
