← Back to team overview

desktop-packages team mailing list archive

[Bug 1101554] Re: 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
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to compiz in Ubuntu.
https://bugs.launchpad.net/bugs/1101554

Title:
  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".

Status in Compiz:
  Fix Released
Status in Compiz 0.9.9 series:
  Won't Fix
Status in compiz package in Ubuntu:
  Fix Released

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: 12448
  Checker: FORWARD_NULL
  Category: dynamic_cast
  CWE definition: http://cwe.mitre.org/data/definitions/476.html
  File: /tmp/buildd/compiz-0.9.9~daily13.01.14/plugins/animation/src/dodge.cpp
  Function: DodgeAnim::calculateDodgeAmounts()
  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_cast<DodgeAnim *> (curAnim);
  797 
  CID 12448 - FORWARD_NULL
  Dereferencing null pointer "dodgeAnim".
  798 	    dodgeAnim->mDodgeMaxAmountX = dodgeAmountX + offsetX;
  799 	    dodgeAnim->mDodgeMaxAmountY = dodgeAmountY + offsetY;
  800 	    dodgeAnim->mDodgeDirection = dodgeDirection;
  801

To manage notifications about this bug go to:
https://bugs.launchpad.net/compiz/+bug/1101554/+subscriptions