Bug 98157

Summary: [soup] Timeout support crashes if request already sent
Product: WebKit Reporter: Dominik Röttsches (drott) <d-r>
Component: PlatformAssignee: Dominik Röttsches (drott) <d-r>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: alex, cgarcia, danw, gustavo, mrobinson, rakuco, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 74802    
Attachments:
Description Flags
Patch none

Description Dominik Röttsches (drott) 2012-10-02 07:31:27 PDT 
After writing more tests in bug 74802 I found an issue with cleanup for timeouts in ResourceHandleSoup.
If a request was already sent out, then the timeout timer expires we start clearing the ResourceHandle - which leads to a RefPtr !m_deletionHasBegun failure when the sendRequestCallback is called. As a solution, I propose to reuse the d->m_cancelled flag for that purpose, and only destroy the handle when one of the progress callbacks is called - as before.
Comment 1 Dominik Röttsches (drott) 2012-10-02 12:26:25 PDT 
Created attachment 166732 [details]
Patch
Comment 2 Martin Robinson 2012-10-02 12:33:51 PDT 
See also: https://bugs.webkit.org/show_bug.cgi?id=98055

Is this bug a duplicate?
Comment 3 Dominik Röttsches (drott) 2012-10-02 13:27:36 PDT 
(In reply to comment #2)
> See also: https://bugs.webkit.org/show_bug.cgi?id=98055
> 
> Is this bug a duplicate?

Good catch, thanks.

*** This bug has been marked as a duplicate of bug 98055 ***