After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 705273 - Leaked sockets after failed authentication
Leaked sockets after failed authentication
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: Mailer
3.8.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2013-08-01 09:51 UTC by Milan Crha
Modified: 2013-08-01 11:21 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
eds patch (2.57 KB, patch)
2013-08-01 11:17 UTC, Milan Crha
committed Details | Review

Description Milan Crha 2013-08-01 09:51:28 UTC
Moving this from a downstream bug report:
https://bugzilla.redhat.com/show_bug.cgi?id=989219

The evolution crashed on "out of file handles", with many opened sockets. After investigation with the downstream reporter it turned out that POP3 leaks opened connection after failed authentication. he uses also imapx and ews; I was able to reproduce the file handle leak with POP3.
Comment 1 Milan Crha 2013-08-01 11:15:01 UTC
This was partly introduced by my changes from bug #692904, and I realized that SMTP and POP3 provider were affected. The thing is that the change from bug #692904 was correct, it's only that I overlooked the leaking part after it.
Comment 2 Milan Crha 2013-08-01 11:17:58 UTC
Created attachment 250606 [details] [review]
eds patch

for evolution-data-server;

As I said above, this is the fix for POP3 and SMTP providers.
Comment 3 Milan Crha 2013-08-01 11:21:22 UTC
Created commit 223f0bb in eds master (3.9.90+)
Created commit 1a8c442 in eds gnome-3-8 (3.8.5+)