[Static Analysis][Uninitialized scalar field] In constructor SessionMessageTask

RESOLVED FIXED in Firefox 51

Status

()

defect
RESOLVED FIXED
3 years ago
2 months ago

People

(Reporter: andi, Assigned: andi)

Tracking

(Blocks 1 bug, {coverity})

Trunk
mozilla51
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox51 fixed)

Details

(Whiteboard: CID 1372313)

Attachments

(1 attachment)

The Static Analysis tool Coverity detected that |mPid| is not initialised in the constructor of SessionMessageTask. Looking other the code i can't see where it's used so i suggest we could remove it.
Comment hidden (mozreview-request)

Comment 2

3 years ago
mozreview-review
Comment on attachment 8787105 [details]
Bug 1299749 - removed unused variable from SessionMessageTask.

https://reviewboard.mozilla.org/r/75982/#review74050
Attachment #8787105 - Flags: review?(amarchesini) → review+

Comment 3

3 years ago
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1b9a3973200d
removed unused variable from SessionMessageTask. r=baku

Comment 4

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/1b9a3973200d
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Component: DOM → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.