diff options
author | Lance Stout <lancestout@gmail.com> | 2011-09-01 00:50:45 -0700 |
---|---|---|
committer | Lance Stout <lancestout@gmail.com> | 2011-09-01 00:50:45 -0700 |
commit | 0af35c2224889a06e5ac25028ee5b489106d201b (patch) | |
tree | 057e5f670881dd9769c67890e97f55093458a60c /sleekxmpp/xmlstream | |
parent | 76bc0a2ba679828d17871c43bfe076b077c9b600 (diff) | |
download | slixmpp-0af35c2224889a06e5ac25028ee5b489106d201b.tar.gz slixmpp-0af35c2224889a06e5ac25028ee5b489106d201b.tar.bz2 slixmpp-0af35c2224889a06e5ac25028ee5b489106d201b.tar.xz slixmpp-0af35c2224889a06e5ac25028ee5b489106d201b.zip |
Fix memory reference bugs.
Diffstat (limited to 'sleekxmpp/xmlstream')
-rw-r--r-- | sleekxmpp/xmlstream/stanzabase.py | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/sleekxmpp/xmlstream/stanzabase.py b/sleekxmpp/xmlstream/stanzabase.py index a2826ead..1ff89554 100644 --- a/sleekxmpp/xmlstream/stanzabase.py +++ b/sleekxmpp/xmlstream/stanzabase.py @@ -39,15 +39,23 @@ def register_stanza_plugin(stanza, plugin, iterable=False, overrides=False): the parent stanza. """ tag = "{%s}%s" % (plugin.namespace, plugin.name) + + # Prevent weird memory reference gotchas by ensuring + # that the parent stanza class has its own set of + # plugin info maps and is not using the mappings from + # an ancestor class (like ElementBase). + plugin_info = ('plugin_attrib_map', 'plugin_tag_map', + 'plugin_iterables', 'plugin_overrides') + for attr in plugin_info: + info = getattr(stanza, attr) + setattr(stanza, attr, info.copy()) + stanza.plugin_attrib_map[plugin.plugin_attrib] = plugin stanza.plugin_tag_map[tag] = plugin + if iterable: - # Prevent weird memory reference gotchas. - stanza.plugin_iterables = stanza.plugin_iterables.copy() stanza.plugin_iterables.add(plugin) if overrides: - # Prevent weird memory reference gotchas. - stanza.plugin_overrides = stanza.plugin_overrides.copy() for interface in plugin.overrides: stanza.plugin_overrides[interface] = plugin.plugin_attrib |