Open
Bug 1528567
Opened 6 years ago
Updated 2 years ago
Potential nullptr dereference in nsIFrame::PeekOffsetParagraph
Categories
(Core :: Layout, defect, P5)
Core
Layout
Tracking
()
NEW
People
(Reporter: ehsan.akhgari, Unassigned)
References
(Blocks 1 open bug)
Details
Coverity reports potential nullptr dereferences here: https://searchfox.org/mozilla-central/rev/e6520f0a4dd5d7474c32a1164744953ea59be0d0/layout/generic/nsFrame.cpp#7948,7974
In both locations frame might be null from these assignments: https://searchfox.org/mozilla-central/rev/e6520f0a4dd5d7474c32a1164744953ea59be0d0/layout/generic/nsFrame.cpp#7965,7991
|
Reporter | |
Updated•6 years ago
|
Blocks: coverity-analysis
|
||
Comment 1•6 years ago
|
||
There's always a containing block, so there's no way we should be able to get there if those lines are executed. Can we exclude those somehow?
|
||
Comment 2•6 years ago
|
||
Sounds like this isn't a real issue, though it would be nice to silence the warnings.
Priority: -- → P5
|
||
Comment 3•6 years ago
|
||
A MOZ_ASSERT(frame) would make coverity happy there.
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•