This fixes a Bad file descriptor error caused by attempting to access the #36

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@jhalcrow
Contributor

This fixes a Bad file descriptor error caused by attempting to access the request after it has already been closed.

@davidthewatson

I can confirm that this quiets the bad file descriptor exception that I saw in paramiko 1.7.7.1:

Exception happened during processing of request from ('127.0.0.1', 52904)
Traceback (most recent call last):
File "/usr/lib64/python2.7/SocketServer.py", line 582, in process_request_thread
self.finish_request(request, client_address)
File "/usr/lib64/python2.7/SocketServer.py", line 323, in finish_request
self.RequestHandlerClass(request, client_address, self)
File "/usr/lib64/python2.7/SocketServer.py", line 639, in init
self.handle()
File "tunnel.py", line 38, in handle
print('Tunnel closed from %r' % (self.request.getpeername(),))
File "/usr/lib64/python2.7/socket.py", line 224, in meth
return getattr(self._sock,name)(*args)
File "/usr/lib64/python2.7/socket.py", line 170, in _dummy
raise error(EBADF, 'Bad file descriptor')
error: [Errno 9] Bad file descriptor

Thanks!

@bitprophet bitprophet added a commit that referenced this pull request Sep 26, 2012
@bitprophet bitprophet Changelog re #36
(cherry picked from commit 5b8cdd990fea9269f8b5297ad1e3e52de08f2ee6)
45c1212
@pkirk
pkirk commented May 24, 2013

I can confirm too.
My paramiko is 1.7.5 ( python-paramiko-1.7.5-2.1.el6-noarch ).

Thanks.

@bitprophet bitprophet added a commit that referenced this pull request Sep 20, 2013
@bitprophet bitprophet Changelog re #36 fa61f3c
@bitprophet
Member

Thanks! Merged back to the 1.10 line.

@bitprophet bitprophet closed this Sep 20, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment