Bug 11194
| Summary: | SpinneretWebHost doesn't set refcount properly | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Krzysztof Kowalczyk <kkowalczyk> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Blocker | ||
| Priority: | P2 | ||
| Version: | 420+ | ||
| Hardware: | Mac | ||
| OS: | OS X 10.4 | ||
Krzysztof Kowalczyk
SpinneretWebHost doesn't have a constructor and its m_refCount field is not set properly. You can verify this by setting a brakpoint in debug version in Spinerette.cpp at line ~187:
gWebHost = new SpinneretWebHost();
<-- in debug here gWebHost->m_refCount is 0xcdcdcdcd, which means "uninitialized"
gWebHost->AddRef();
No patch but the fix should be trivial: supply
SpinneretWebHost::SpinneretWebHost() : m_refCount(0) {}
(unless for some reason SpinneretWebHost is not supposed to have a constructor)
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Krzysztof Kowalczyk
Has been fixed in another checkin.