dorian
/
paramiko
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

safe type checking (isinstance instead of type-is) 

Maintainer note: added changelog entry.
(cherry picked from commit 8917d83221)
This commit is contained in:
Jeff Forcier 2012-04-19 15:19:59 -07:00
parent ef9b251885
commit 1a033dc596
2 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
CHANGES
View File

@ -4,6 +4,9 @@ Temporary, post-Paramiko changelog while we get our sh!t together.
* #15: Implemented parameter substitution in SSHConfig, matching the
implementation of `ssh_config(5)`. Thanks to Olle Lundberg for the patch.
* #24: Switch some internal type checking to use `isinstance` to help prevent
problems with client libraries using subclasses of builtin types. Thanks to
Alex Morega for the patch.
## ssh 1.7.13 (2012-02-13)

8
paramiko/sftp_client.py
View File

@ -641,13 +641,13 @@ class SFTPClient (BaseSFTP):
msg = Message()
msg.add_int(self.request_number)
for item in arg:
if type(item) is int:
if isinstance(item, int):
msg.add_int(item)
elif type(item) is long:
elif isinstance(item, long):
msg.add_int64(item)
elif type(item) is str:
elif isinstance(item, str):
msg.add_string(item)
elif type(item) is SFTPAttributes:
elif isinstance(item, SFTPAttributes):
item._pack(msg)
else:
raise Exception('unknown type for %r type %r' % (item, type(item)))