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 576460 - No cap on number of HTTP connections
No cap on number of HTTP connections
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: http backend
1.0.x
Other Linux
: Normal major
: ---
Assigned To: gvfs-maint
gvfs-maint
Depends on:
Blocks:
 
 
Reported: 2009-03-23 18:23 UTC by Philip Withnall
Modified: 2014-07-03 19:10 UTC
See Also:
GNOME target: ---
GNOME version: 2.25/2.26


Attachments
Test case for number of connections (1.26 KB, text/plain)
2009-03-23 18:24 UTC, Philip Withnall
Details
Log of the network traffic (203.03 KB, text/plain)
2009-03-23 18:27 UTC, Philip Withnall
Details
Log of the network traffic (203.03 KB, application/x-compressed-tar)
2009-03-23 18:27 UTC, Philip Withnall
Details

Description Philip Withnall 2009-03-23 18:23:16 UTC
If a large number of g_file_load_contents_async()s (for example) are issued in a short space of time for a HTTP file, the GVFS HTTP backend will happily make all the connections simultaneously, which crashes my router. It doesn't seem to take SoupSession's max-conns or max-conns-per-host properties into account, which it should do; or have its own equivalents.

Test case and Wireshark log coming up.
Comment 1 Philip Withnall 2009-03-23 18:24:41 UTC
Created attachment 131206 [details]
Test case for number of connections

Test case. If the network traffic is logged, all the requests/connections are made within 100ms. Log coming up.
Comment 2 Philip Withnall 2009-03-23 18:27:42 UTC
Created attachment 131209 [details]
Log of the network traffic

Wireshark log of traffic on port 80.
Comment 3 Philip Withnall 2009-03-23 18:27:57 UTC
Created attachment 131210 [details]
Log of the network traffic

Wireshark log of traffic on port 80.
Comment 4 Philip Withnall 2009-03-23 18:28:47 UTC
Whoops, duplicate upload. Ignore the first one.
Comment 5 Ross Lagerwall 2014-07-03 19:10:52 UTC
This should be fixed with the use of a single SoupSession in bug 732090. Thanks.