Closed
Bug 1240819
Opened 10 years ago
Closed 10 years ago
Some more trivial coverity fixes
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
People
(Reporter: nical, Assigned: nical)
References
(Blocks 1 open bug)
Details
(Keywords: coverity, Whiteboard: [CID 750089][CID 1292631])
Attachments
(1 file)
|
5.79 KB,
patch
|
jrmuizel
:
review+
|
Details | Diff | Splinter Review |
Followup of bug 1240708.
|
Assignee | |
Updated•10 years ago
|
|
|
Assignee | |
Updated•10 years ago
|
Whiteboard: [CID 750089] → [CID 750089][CID 1292631]
|
|
Assignee | |
Comment 1•10 years ago
|
||
Coverity figured out that we are checking whether drawTarget is null after having already dereferenced it. I checked where the DT comes from and it can never be null with the current gfxContext implementation, so I added a check for this so as to make the assumption will still hold true in the future and removed the branches.
Attachment #8710464 -
Flags: review?(jmuizelaar)
|
||
Updated•10 years ago
|
Attachment #8710464 -
Flags: review?(jmuizelaar) → review+
|
|
Assignee | |
Updated•10 years ago
|
Keywords: leave-open
|
||
Comment 3•10 years ago
|
||
| bugherder | ||
|
|
Assignee | |
Updated•10 years ago
|
|
||
Updated•7 years ago
|
Blocks: coverity-analysis
You need to log in
before you can comment on or make changes to this bug.
Description
•