Closed Bug 1361478 Opened 3 years ago Closed 3 years ago

coverity report: Using uninitialized value "result". Field "result.mScanSiblings" is uninitialized.

Categories

(Core :: Layout: Text and Fonts, defect, minor)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: mats, Assigned: mats)

References

(Blocks 1 open bug, )

Details

(Keywords: coverity)

Attachments

(1 file)

** CID 1405752:  Uninitialized variables  (UNINIT)
/layout/generic/nsTextFrame.cpp: 1284 in CanTextCrossFrameBoundary(nsIFrame *)()


________________________________________________________________________________________________________
*** CID 1405752:  Uninitialized variables  (UNINIT)
/layout/generic/nsTextFrame.cpp: 1284 in CanTextCrossFrameBoundary(nsIFrame *)()
1278           result.mFrameToScan = nullptr;
1279           result.mOverflowFrameToScan = nullptr;
1280           result.mTextRunCanCrossFrameBoundary = false;
1281           result.mLineBreakerCanCrossFrameBoundary = false;
1282         }
1283       }
>>>     CID 1405752:  Uninitialized variables  (UNINIT)
>>>     Using uninitialized value "result". Field "result.mScanSiblings" is uninitialized.
1284       return result;
1285     }
1286     
1287     BuildTextRunsScanner::FindBoundaryResult
1288     BuildTextRunsScanner::FindBoundaries(nsIFrame* aFrame, FindBoundaryState* aState)
1289     {
Attached patch fixSplinter Review
Attachment #8863872 - Flags: review?(jfkthame)
Attachment #8863872 - Flags: review?(jfkthame) → review+
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a80cc3b818f5
Add a ctor for FrameTextTraversal that initializes all members.  r=jfkthame
Flags: in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/a80cc3b818f5
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.