Coverity FORWARD_NULL - CID 12451 - plugins/animation/src/extensionplugin.cpp - in function: ExtensionPluginAnimation::paintShouldSkipWindow(CompWindow *) - Dynamic cast to pointer "dynamic_cast <RestackAnim *>(aw->curAnimation())" can return NULL. Passing null pointer "dynamic_cast <RestackAnim *>(aw->curAnimation())" to function "RestackAnim::paintedElsewhere()", which dereferences it. (The dereference happens because this is a virtual function call.)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Confirmed
|
Medium
|
Unassigned | ||
0.9.9 |
Won't Fix
|
High
|
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:/
CID: 12451
Checker: FORWARD_NULL
Category: dynamic_cast
CWE definition: http://
File: /tmp/buildd/
Function: ExtensionPlugin
Code snippet:
110 data->mVisitCou
111
112 // If the window is (to be) painted somewhere other than in its
113 // original stacking order, we don't need to paint it now.
CID 12451 - FORWARD_NULL
Dynamic cast to pointer "dynamic_cast <RestackAnim *>(aw->
Passing null pointer "dynamic_cast <RestackAnim *>(aw->
114 if (aw->curAnimation ()->info ()->isRestackAnim &&
115 dynamic_
116 return true;
117 }
118 return false;
119 }
summary: |
- Coverity FORWARD_NULL - CID 12451 + Coverity FORWARD_NULL - CID 12451 - + plugins/animation/src/extensionplugin.cpp - in function: + ExtensionPluginAnimation::paintShouldSkipWindow(CompWindow *) - Dynamic + cast to pointer "dynamic_cast <RestackAnim *>(aw->curAnimation())" can + return NULL. Passing null pointer "dynamic_cast <RestackAnim + *>(aw->curAnimation())" to function "RestackAnim::paintedElsewhere()", + which dereferences it. (The dereference happens because this is a + virtual function call.) |
Changed in compiz: | |
milestone: | none → 0.9.10.0 |
Changed in compiz: | |
milestone: | 0.9.10.0 → 0.9.11.0 |
Changed in compiz: | |
milestone: | 0.9.11.0 → 0.9.12.1 |
Changed in compiz: | |
status: | New → Confirmed |
importance: | High → Medium |
Changed in compiz: | |
milestone: | 0.9.12.1 → 0.9.12.2 |
Source file with Coverity annotations.