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 546444 - Camel waits forever when downloading a msg from POP3 (SSL) when connection goes down
Camel waits forever when downloading a msg from POP3 (SSL) when connection go...
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: Mailer
2.24.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2008-08-05 17:24 UTC by Sergio Villar
Modified: 2013-09-14 16:52 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
Patch that fixes the neverending poll (365 bytes, patch)
2008-08-05 17:27 UTC, Sergio Villar
committed Details | Review

Description Sergio Villar 2008-08-05 17:24:55 UTC
Please describe the problem:
The problem happens when while trying to get a message from a POP3 account with SSL security, the network connection suddenly is lost. Then the Camel SSL stream will perform a PR_Poll forever, because when the timeout of that function happens it returns 0 and in this case only errno is set to some value, but the loop will go on forever.

Steps to reproduce:
1. try to open a message from a POP3 SSL account (better a big one)
2. while retrieving is going on, disconnect the machine from the network
3. camel will try to read forever

(I tested this with Modest and tinymail (camel-lite), but the code is the same in this case for both Camel and tinymail (camel-lite)


Actual results:
Camel performs a PR_Poll forever

Expected results:
Camel should exit with a connection error

Does this happen every time?
Yes

Other information:
Comment 1 Sergio Villar 2008-08-05 17:27:11 UTC
Created attachment 115920 [details] [review]
Patch that fixes the neverending poll

This patch makes the function stream_read exit whenever that connection error happen. Maybe we could set a number of retries but IMHO it makes no sense in this case because we'll be waiting too much for the timeouts.
Comment 2 Philip Van Hoof 2008-08-05 18:22:19 UTC
Reviewed and approved for Tinymail's camel-lite
Comment 3 Srinivasa Ragavan 2008-08-06 03:23:46 UTC
Commit to trunk as well. Thanks for your patch.
Comment 4 Sergio Villar 2008-08-13 15:51:16 UTC
Seems that they're taking too much time to give me an account in GNOME svn.
Comment 5 Suman Manjunath 2008-08-18 03:25:43 UTC
Patch committed to SVN trunk as r9361 
http://svn.gnome.org/viewvc/evolution-data-server?view=revision&revision=9361