Summary: | m_frame null checked but then dereferenced | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Darin Fisher (:fishd, Google) <fishd> | ||||
Component: | WebCore Misc. | Assignee: | Darin Fisher (:fishd, Google) <fishd> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | oliver | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Darin Fisher (:fishd, Google)
2008-10-23 17:57:40 PDT
Created attachment 24629 [details]
v1 patch: add early return
Comment on attachment 24629 [details]
v1 patch: add early return
This looks to me like what the author intended in their original code. Certainly the existing code is incorrect.
Please add a comment to your changelog when landing about how this was found using a static analysis tool and thus there are no reproduction steps for a test case. |