This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author nadeem.vawda
Recipients lukasz.langa, nadeem.vawda, vstinner
Date 2011-01-07.02:19:35
SpamBayes Score 7.380678e-09
Marked as misclassified No
Message-id <1294366779.15.0.0946107332291.issue10512@psf.upfronthosting.co.za>
In-reply-to
Content
Looking at the warnings from test_urllib2net, it seems that they all originate in the FTP tests:
* OtherNetworkTests.test_ftp()
* TimeoutTest.test_ftp_basic()
* TimeoutTest.test_ftp_default_timeout()

Most of these leaks seem to stem from the fact that socket.SocketIO.close() doesn't behave as documented. According to its docstring, it is meant to decrement the underlying socket's refcount, and close it if the refcount drops to zero. However, to do this job it calls socket._decref_socketios(), which is defined as follows:

    def _decref_socketios(self):
        if self._io_refs > 0:
            self._io_refs -= 1
        if self._closed:
            self.close()

Clearly, this doesn't do what the docstring describes. Changing the second conditional from "if self._closed:" to "if self._io_refs <= 0:" disposes of all but one of the ResourceWarnings, but also breaks 8 tests in test_socket. It seems that the tests expect a socket to remain open after all referring SocketIO objects have been closed, which contradicts the docstring for SocketIO.close(). I suppose I should open a separate issue for this.


The remaining warning occurs in test_ftp() when retrieving a non-existent file; I haven't yet managed to figure out what is causing it.
History
Date User Action Args
2011-01-07 02:19:39nadeem.vawdasetrecipients: + nadeem.vawda, vstinner, lukasz.langa
2011-01-07 02:19:39nadeem.vawdasetmessageid: <1294366779.15.0.0946107332291.issue10512@psf.upfronthosting.co.za>
2011-01-07 02:19:35nadeem.vawdalinkissue10512 messages
2011-01-07 02:19:35nadeem.vawdacreate