Closed Bug 1276642 Opened 3 years ago Closed 3 years ago

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

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 1362108)

Attachments

(1 file)

The Static Analysis tool Coverity added that |mStatus| is 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 8757913 [details]
MozReview Request: Bug 1276642 - initialize member variable |mStatus|. r?valentin

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/56280/diff/1-2/
Attachment #8757913 - Attachment description: MozReview Request: Bug 1276642 - initialize member variable |mStatus|. r?dbaron → MozReview Request: Bug 1276642 - initialize member variable |mStatus|. r?valentin
Attachment #8757913 - Flags: review?(dbaron) → review?(valentin.gosu)
Attachment #8757913 - Flags: review?(valentin.gosu) → review+
Comment on attachment 8757913 [details]
MozReview Request: Bug 1276642 - initialize member variable |mStatus|. r?valentin

https://reviewboard.mozilla.org/r/56280/#review52888

Ship it!
https://hg.mozilla.org/mozilla-central/rev/2dc637147b07
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.