Closed Bug 1144040 Opened 6 years ago Closed 6 years ago

Coverity detected two uninitialized class members in the nsHttpHandler constructor

Categories

(Core :: Networking: HTTP, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: bagder, Assigned: bagder)

References

(Blocks 1 open bug)

Details

(Keywords: coverity)

Attachments

(1 file)

Coverity pointed out two class members that aren't initialized properly in the constructor and thus MAY later be accessed with "random" content.

CID 1157357
Comment on attachment 8578510 [details] [diff] [review]
0001-Bug-1144040-Coverity-detected-uninitialized-nsHttpHa.patch

Review of attachment 8578510 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm thanks daniel
Attachment #8578510 - Flags: review?(mcmanus) → review+
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c27057bbbb33

OS X debug and b2g debug buildbusts, but seem unrelated.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/b976b2dc1a26
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.