Re: [Mailman-Developers] Bugs in Utils.py and pythonlib/smtplib.py (Mailman 1.1)
[Dan A. Dickey]
Here are a couple of patches
Thanks!
to fix file descriptor leaks in Utils.py,
The message delivery stuff has changed a lot since 1.1, so there's no longer any TrySMTPDelivery() function in Utils.py.
and pythonlib/smtplib.py.
The modules under Mailman/pythonlib in the distribution are merely copies of the current version of various modules that accompany python itself. Thus, if there are bugs in these modules, they should be reported to the python maintainers.
In this case, though, I don't think there is a bug per se in the smtplib library -- rather, it's Mailman that doesn't take enough care in closing up file descriptors.
Below is a quick patch against current CVS Mailman for those cron scripts I have seen run out of available file descriptors (when there's a lot of lists to iterate over). I don't actually run CVS Mailman at the moment, so the patch has not been tested.
-- Harald
participants (1)
-
Harald Meland