Coverity UNINIT_CTOR - CID 10891 - src/rect/src/rect.cpp in function: CompRect::CompRect() - Non-static class member field "mRegion.size" is not initialized in this constructor nor in any functions that it calls.

Bug #957576 reported by Product Strategy Coverity Bug Uploader
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
Won't Fix
Medium
Unassigned
Compiz Core
New
Medium
Unassigned
0.9.5
New
Undecided
Unassigned

Bug Description

This bug is exported from the Coverity Integration Manager on Canonical's servers. For information on how this is done please see this website: https://wiki.ubuntu.com/CanonicalProductStrategy/Coverity
CID: 10891
Checker: UNINIT_CTOR
Category: No category available
CWE definition: http://cwe.mitre.org/data/definitions/457.html
File: /tmp/buildd/compiz-0.9.7.0~bzr3025/src/rect/src/rect.cpp
Function: CompRect::CompRect()
Code snippet:
 37 mRegion.extents.x1 = 0;
 38 mRegion.extents.x2 = 0;
 39 mRegion.extents.y1 = 0;
 40 mRegion.extents.y2 = 0;
CID 10891 - UNINIT_CTOR
Non-static class member field "mRegion.size" is not initialized in this constructor nor in any functions that it calls.
 41 }
 42
 43 CompRect::CompRect (int x, int y, int width, int height)
 44 {
 45 mRegion.rects = &mRegion.extents;
 46 mRegion.numRects = 1;

Revision history for this message
Product Strategy Coverity Bug Uploader (coverity-uploader) wrote : compiz-core-0.9.5: /tmp/buildd/compiz-0.9.7.0~bzr3025/src/rect/src/rect.cpp

Source file with Coverity annotations.

Changed in compiz-core:
importance: Undecided → Medium
Changed in compiz:
importance: Undecided → Medium
MC Return (mc-return)
summary: - Coverity UNINIT_CTOR - CID 10891
+ Coverity UNINIT_CTOR - CID 10891 - src/rect/src/rect.cpp in function:
+ CompRect::CompRect() - Non-static class member field "mRegion.size" is
+ not initialized in this constructor nor in any functions that it calls.
Changed in compiz:
milestone: none → 0.9.10.0
Changed in compiz:
milestone: 0.9.10.0 → 0.9.10.2
MC Return (mc-return)
Changed in compiz:
milestone: 0.9.10.2 → 0.9.11.0
Revision history for this message
Stephen M. Webb (bregma) wrote :

The member in question is also never used anywhere. Rewriting the CompRect structire to no use an Xregion would eliminate a small amount of bloat but is not worth nreaking the ABI.

Changed in compiz:
milestone: 0.9.11.0 → none
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.