Closed Bug 1240819 Opened 4 years ago Closed 4 years ago

Some more trivial coverity fixes

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: nical, Assigned: nical)

References

(Blocks 1 open bug)

Details

(Keywords: coverity, Whiteboard: [CID 750089][CID 1292631])

Attachments

(1 file)

Followup of bug 1240708.
Assignee: nobody → nical.bugzilla
Keywords: coverity
Whiteboard: [CID 750089]
Whiteboard: [CID 750089] → [CID 750089][CID 1292631]
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)
Attachment #8710464 - Flags: review?(jmuizelaar) → review+
Keywords: leave-open
Status: NEW → RESOLVED
Closed: 4 years ago
Keywords: leave-open
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.