Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 366989 - [Browser] WebKitGTK crashes when clearing all session cookies
Summary: [Browser] WebKitGTK crashes when clearing all session cookies
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.7   Edit
Hardware: PC Linux-GTK
: P3 major (vote)
Target Milestone: 3.7.2   Edit
Assignee: Grant Gayed CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-16 16:43 EST by Grant Gayed CLA
Modified: 2011-12-19 11:54 EST (History)
2 users (show)

See Also:
Silenio_Quarti: review+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Grant Gayed CLA 2011-12-16 16:43:00 EST 
Browser.clearSessions() crashes when WebKitGTK is the underlying renderer as a result of a cookie pointer being incorrectly freed.
Comment 1 Grant Gayed CLA 2011-12-16 17:02:09 EST 
Fixed in 3.8 stream > 20111216, the change is in http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?id=78abe5d2890d6c18bca5ab665e997506c0954a8f .

This should go into 3.7.2, SSQ can you review?  The fix for the crash is the change of g_free() to soup_cookie_free().  The g_slist_free() added below it fixes a leak.
Comment 2 Silenio Quarti CLA 2011-12-19 11:34:00 EST 
Fix is good for 3.7.2.
Comment 3 Grant Gayed CLA 2011-12-19 11:54:03 EST 
Fix released to the 3.7.2 stream > 20111219, closing as FIXED.