Closed Bug 1276625 Opened 3 years ago Closed 3 years ago

[Static Analysis][Uninitialized pointer field] In constructor nsBinHexDecoder

Categories

(Core :: Networking, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: andi, Assigned: andi)

References

(Blocks 1 open bug)

Details

(Keywords: coverity, Whiteboard: CID 1362111 )

Attachments

(1 file)

The Static Analysis tool Coverity added that |mPosInDataBuffer| and |mRlebuf| are not initialized. As we are trying to push this patch: Bug 525063 we must prepare our base code in order to be able to integrate it.
Comment on attachment 8757881 [details]
MozReview Request: Bug 1276625 - initialize member variables |mPosInDataBuffer| and |mRlebuf|. r?valentin

https://reviewboard.mozilla.org/r/56250/#review52872

Looks good. Thanks
Attachment #8757881 - Flags: review?(valentin.gosu) → review+
https://hg.mozilla.org/mozilla-central/rev/d0337c1582a1
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Depends on: 1345004
You need to log in before you can comment on or make changes to this bug.