Coverity UNINIT_CTOR - CID 12566 - plugins/animation/src/grid.cpp - in function: GridAnim::GridAnim(CompWindow *, WindowEvent, float, AnimEffectInfo *, const CompRect &) - Non-static class members "mGridWidth" and "mGridHeight" are not initialized in this constructor nor in any functions that it calls.

Bug #1101618 reported by Product Strategy Coverity Bug Uploader on 2013-01-18
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
Medium
MC Return
0.9.9
Medium
Unassigned
compiz (Ubuntu)
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: 12566
Checker: UNINIT_CTOR
Category: No category available
CWE definition: http://cwe.mitre.org/data/definitions/457.html
File: /tmp/buildd/compiz-0.9.9~daily13.01.14/plugins/animation/src/grid.cpp
Function: GridAnim::GridAnim(CompWindow *, WindowEvent, float, AnimEffectInfo *, const CompRect &)
Code snippet:
188 Animation::Animation (w, curWindowEvent, duration, info, icon),
189 mModel (NULL),
190 mUseQTexCoord (false)
191 {
CID 12566 - UNINIT_CTOR
Non-static class member "mGridWidth" is not initialized in this constructor nor in any functions that it calls.
Non-static class member "mGridHeight" is not initialized in this constructor nor in any functions that it calls.
192 }
193
194 void
195 GridAnim::init ()
196 {
197 initGrid ();

Related branches

MC Return (mc-return) on 2013-07-06
summary: - Coverity UNINIT_CTOR - CID 12566
+ Coverity UNINIT_CTOR - CID 12566 - plugins/animation/src/grid.cpp - in
+ function: GridAnim::GridAnim(CompWindow *, WindowEvent, float,
+ AnimEffectInfo *, const CompRect &) - Non-static class members
+ "mGridWidth" and "mGridHeight" are not initialized in this constructor
+ nor in any functions that it calls.
Changed in compiz:
milestone: none → 0.9.10.0
status: New → Triaged
assignee: nobody → MC Return (mc-return)
MC Return (mc-return) on 2013-07-07
Changed in compiz:
status: Triaged → In Progress
MC Return (mc-return) on 2013-07-07
Changed in compiz:
status: In Progress → New
status: New → Triaged
Changed in compiz:
milestone: 0.9.10.0 → 0.9.10.2
MC Return (mc-return) on 2013-07-24
Changed in compiz:
milestone: 0.9.10.2 → 0.9.11.0
MC Return (mc-return) on 2013-07-24
Changed in compiz:
status: Triaged → In Progress
MC Return (mc-return) on 2013-07-24
Changed in compiz:
status: In Progress → Triaged
MC Return (mc-return) on 2013-07-24
Changed in compiz:
status: Triaged → In Progress
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:compiz at revision None, scheduled for release in compiz, milestone 0.9.11.0

Changed in compiz:
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :
Download full text (44.6 KiB)

This bug was fixed in the package compiz - 1:0.9.11+14.04.20140214-0ubuntu1

---------------
compiz (1:0.9.11+14.04.20140214-0ubuntu1) trusty; urgency=low

  [ Timo Jyrinki ]
  * Bump version to 0.9.11

  [ Marco Trevisan (Treviño) ]
  * debian/00_remove_decor_in_unity_session.py: add migration script
    to avoid to load the decor plugin on compiz startup when using unity.
  * debian/compiz-gnome.gconf-defaults: disable decor plugin on unity session

  [ Sebastien Bacher ]
  * debian/compiz-gnome.links: lists keybinding in unity-control-center
  * typo fix in the previous commit. (LP: #1271710)

  [ Iven Hsu ]
  * Opacify: Only dim the windows above the active window.(LP:
    #1189374). (LP: #1189374)
  * KWD: Fix compile errors with KDE 4.11. The KWin developers made
    kdecorationbridge.h private. See:
    http://lists.freedesktop.org/archives/compiz/2013-March/003479.html
    (LP: #1193792). (LP: #1193792)

  [ Nikolay Martynov ]
  * When static switcher is enabled and has an option to show
    application icon turned on the icons are expected to be ~1/3 of a
    thumbnail (48px). Instead they are displayed in 512px size and
    completely cover everything. This change addresses this issue. See
    LP #1173914. (LP: #1173914, #1186426)

  [ BryanFRitt ]
  * Fixed the non-working Annotate 'Clear' Button. Moved this option's
    CCSM position upwards to keep the button shortcuts together. (LP:
    #1202907). (LP: #1202907)

  [ CI bot ]
  * Flush trunk to Ubuntu

  [ William Hua ]
  * Replace <Primary> with <Control> in CCSM. Fixes
    https://bugs.launchpad.net/compiz/+bug/1069121. (LP: #1069121)
  * Tweak support of key bindings of the form
    '<Modifier>Modifier_KeySym'. We tweak a bit the behaviour of key
    bindings such as '<Control>Shift_L' and '<Alt>Alt_R'. 1. We ignore
    the order of key pressing and releasing, so tapping
    '<Shift>Control_L' is the same as '<Control>Shift_L'. 2. We properly
    handle the double modifiers case, for example '<Control>Control_R'.
    3. We also parse key bindings with '<Primary>' being equivalent to
    '<Control>'.
  * Fix GSettings tests with extra slash.
  * Add an interface for plugins to provide non-option key actions that
    can be triggered.

  [ Eleni Maria Stea ]
  * It fixes the bug #1245886. In DecorScreen::handleEvent compiz
    shouldn't try to handle any events if there's no active window yet.
    (LP: #1245886)
  * Compiz static analysis shows that some compiz classes have virtual
    methods but not virtual destructors. Added the virtual destructors
    to get rid of warnings and potential memory leaks.
  * fixed cmake syntax errors.
  * CMake considered compiz a C++ project and couldn't find some
    dependencies like pthreads. Defined compiz as a C, CXX project to
    fix the issue.

  [ Povilas Kanapickas ]
  * Opacify: Properly initialize window drawing for new windows in
    Opacify plugin. (LP: #787814, part 2). (LP: #787814)
  * Opacify: Fix damage generation in the Opacify plugin. When setting
    opacity to some value, non-opacified windows need to be damaged
    regardless of opacity, whereas opacified windows need to be damaged
    only if opacity changes. Remove u...

Changed in compiz (Ubuntu):
status: New → Fix Released
Stephen M. Webb (bregma) on 2014-11-03
Changed in compiz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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