[issue18528] Possible fd leak in socketmodule

Antoine Pitrou report at bugs.python.org
Mon Jul 22 15:06:35 CEST 2013


Antoine Pitrou added the comment:

What if you add "errno = 0" at the beginning of the function? Perhaps Coverity is confused by the fact that we use CHECK_ERRNO() to infer whether accept() succeeded or not, rather than simply checking the return value.

I don't like your patch, because it adds a code path that would really be a logic error and therefore should not pass silently.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue18528>
_______________________________________


More information about the Python-bugs-list mailing list