Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 1 commit into from

Conversation

jhalcrow
Copy link
Contributor

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

… the request after it has already been closed.
@davidthewatson
Copy link

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 added a commit that referenced this pull request Sep 26, 2012
(cherry picked from commit 5b8cdd990fea9269f8b5297ad1e3e52de08f2ee6)
@pkirk
Copy link

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 added a commit that referenced this pull request Sep 20, 2013
@bitprophet
Copy link
Member

Thanks! Merged back to the 1.10 line.

@bitprophet bitprophet closed this Sep 20, 2013
intgr pushed a commit to intgr/paramiko that referenced this pull request Nov 27, 2019
SSHConfig: more robust substitution
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants