Summary: | REGRESSION (r68260): Crash in PlatformCertificateInfo::~PlatformCertificateInfo when navigating away from Gmail | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Roben (:aroben) <aroben> | ||||||
Component: | WebKit2 | Assignee: | Adam Roben (:aroben) <aroben> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | sam | ||||||
Priority: | P2 | Keywords: | InRadar, PlatformOnly, Regression | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Attachments: |
|
Description
Adam Roben (:aroben)
2010-09-24 16:14:37 PDT
Luckily, I have a fix! Created attachment 68899 [details]
Don't double-free CERT_CONTEXTs when copying PlatformCertificateInfos on Windows
Comment on attachment 68899 [details] Don't double-free CERT_CONTEXTs when copying PlatformCertificateInfos on Windows View in context: https://bugs.webkit.org/attachment.cgi?id=68899&action=review > WebKit2/Shared/win/PlatformCertificateInfo.cpp:77 > +PlatformCertificateInfo& PlatformCertificateInfo::operator=(const PlatformCertificateInfo& other) > +{ > + if (m_certificateContext) > + ::CertFreeCertificateContext(m_certificateContext); > + m_certificateContext = ::CertDuplicateCertificateContext(other.m_certificateContext); > + return *this; > +} We should try and handle the self-assignment case here. Created attachment 68901 [details]
Don't double-free CERT_CONTEXTs when copying PlatformCertificateInfos on Windows
Committed r68387: <http://trac.webkit.org/changeset/68387> |