Coverity FORWARD_NULL - CID 12448 - plugins/animation/src/dodge.cpp - in function: DodgeAnim::calculateDodgeAmounts() - Dynamic cast to pointer "dynamic_cast <DodgeAnim *>(curAnim)" can return NULL. Assigning: "dodgeAnim" = "dynamic_cast <DodgeAnim *>(curAnim)". Later dereferencing null pointer "dodgeAnim".
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Fix Released
|
High
|
Stephen M. Webb | ||
0.9.9 |
Won't Fix
|
High
|
Unassigned | ||
compiz (Ubuntu) |
Fix Released
|
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:/
CID: 12448
Checker: FORWARD_NULL
Category: dynamic_cast
CWE definition: http://
File: /tmp/buildd/
Function: DodgeAnim:
Code snippet:
792
793 Animation *curAnim = awCur->curAnimation ();
794 if (!curAnim || curAnim->info () != AnimEffectDodge)
795 continue;
CID 12448 - FORWARD_NULL
Dynamic cast to pointer "dynamic_cast <DodgeAnim *>(curAnim)" can return NULL.
Assigning: "dodgeAnim" = "dynamic_cast <DodgeAnim *>(curAnim)".
796 DodgeAnim *dodgeAnim = dynamic_
797
CID 12448 - FORWARD_NULL
Dereferencing null pointer "dodgeAnim".
798 dodgeAnim-
799 dodgeAnim-
800 dodgeAnim-
801
Related branches
- Christopher Townsend (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 40 lines (+9/-0)1 file modifiedplugins/animation/src/dodge.cpp (+9/-0)
summary: |
- Coverity FORWARD_NULL - CID 12448 + Coverity FORWARD_NULL - CID 12448 - plugins/animation/src/dodge.cpp - in + function: DodgeAnim::calculateDodgeAmounts() - Dynamic cast to pointer + "dynamic_cast <DodgeAnim *>(curAnim)" can return NULL. Assigning: + "dodgeAnim" = "dynamic_cast <DodgeAnim *>(curAnim)". Later dereferencing + null pointer "dodgeAnim". |
Changed in compiz: | |
milestone: | none → 0.9.10.0 |
Changed in compiz: | |
milestone: | 0.9.10.0 → 0.9.11.0 |
Changed in compiz: | |
status: | New → Incomplete |
status: | Incomplete → Confirmed |
milestone: | 0.9.11.0 → 0.9.12.1 |
Changed in compiz: | |
assignee: | nobody → Stephen M. Webb (bregma) |
status: | Confirmed → In Progress |
Changed in compiz: | |
status: | In Progress → Fix Committed |
Changed in compiz: | |
status: | Fix Committed → Fix Released |
Source file with Coverity annotations.