MantisBT - NetSurf
View Issue Details
0002664NetSurf[All Projects] Generalpublic2019-06-05 20:032020-05-27 08:20
Reporterkinnison 
Assigned Tokinnison 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformAllOSAllOS VersionN/A
Product Version3.9 
Target Version3.10Fixed in Version3.10 
Fixed in CI build #
Reported in CI build #
URL of problem page
Summary0002664: In-progress curl fetches which are active during shutdown of the browser will never be freed
DescriptionWhen we close the browser while there are in-progress fetches in cURL, we never
complete those fetches cleanly and so they leak all over the shop. This manifests often in lwc string leakage from the nsurls they hold refs to.

I'm not sure we care particularly.
Steps To ReproduceStart a fetch of something which takes a while, then close the browser window while it's fetching.

I noticed it while fetching https://httpstat.us/302 where if you close the browser the moment it renders, there's a couple of fetches outstanding.
TagsNo tags attached.
Attached Files

Notes
(0001963)
Vincent Sanders   
2019-06-13 09:24   
this is also seen if one simply stops the fetch rather than quits the browser
(0002057)
kinnison   
2019-08-02 14:59   
I believe this is now fixed.
(0002250)
Vincent Sanders   
2020-05-27 08:20   
Thankyou for reporting this issue.
We believe this is fixed in the 3.10 release.
If this is not the case please feel free to reopen the issue with additional details.

Issue History
2019-06-05 20:03kinnisonNew Issue
2019-06-13 09:24Vincent SandersStatusnew => confirmed
2019-06-13 09:24Vincent SandersProduct Version => 3.9
2019-06-13 09:24Vincent SandersNote Added: 0001963
2019-08-02 14:59kinnisonAssigned To => kinnison
2019-08-02 14:59kinnisonStatusconfirmed => resolved
2019-08-02 14:59kinnisonResolutionopen => fixed
2019-08-02 14:59kinnisonFixed in Version => 3.10
2019-08-02 14:59kinnisonTarget Version => 3.10
2019-08-02 14:59kinnisonNote Added: 0002057
2020-05-27 08:20Vincent SandersStatusresolved => closed
2020-05-27 08:20Vincent SandersNote Added: 0002250