Coverity FORWARD_NULL - CID 12456 - plugins/rotate/src/rotate.cpp - in unction: RotateScreen::initiate(CompAction *, unsigned int, std::vector<CompOption, std::allocator<CompOption>> &) - Comparing "action" to null implies that "action" might be null.

Bug #1101467 reported by Product Strategy Coverity Bug Uploader
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
Triaged
Medium
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: 12456
Checker: FORWARD_NULL
Category: No category available
CWE definition: http://cwe.mitre.org/data/definitions/476.html
File: /tmp/buildd/compiz-0.9.9~daily13.01.14/plugins/rotate/src/rotate.cpp
Function: RotateScreen::initiate(CompAction *, unsigned int, std::vector<CompOption, std::allocator<CompOption>> &)
Code snippet:
408
409 /* Set the rotation state for cube - if action is non-NULL,
410 we set it to manual (as we were called from the 'Initiate
411 Rotation' binding. Otherwise, we set it to Change. */
CID 12456 - FORWARD_NULL
Comparing "action" to null implies that "action" might be null.
Comparing "action" to null implies that "action" might be null.
412 if (action)
413 cubeScreen->rotationState (CubeScreen::RotationManual);
414 else
415 cubeScreen->rotationState (CubeScreen::RotationChange);
416
417 screen->handleCompizEvent ("rotate", "start_viewport_switch", o);

Revision history for this message
Product Strategy Coverity Bug Uploader (coverity-uploader) wrote : compiz-0.9.9: /tmp/buildd/compiz-0.9.9~daily13.01.14/plugins/rotate/src/rotate.cpp

Source file with Coverity annotations.

Changed in compiz:
importance: Undecided → High
MC Return (mc-return)
summary: - Coverity FORWARD_NULL - CID 12456
+ Coverity FORWARD_NULL - CID 12456 - plugins/rotate/src/rotate.cpp - in
+ unction: RotateScreen::initiate(CompAction *, unsigned int,
+ std::vector<CompOption, std::allocator<CompOption>> &) - Comparing
+ "action" to null implies that "action" might be null.
Changed in compiz:
milestone: none → 0.9.10.0
MC Return (mc-return)
Changed in compiz:
milestone: 0.9.10.0 → 0.9.11.0
Stephen M. Webb (bregma)
Changed in compiz:
milestone: 0.9.11.0 → 0.9.12.0
no longer affects: compiz/0.9.9
Changed in compiz:
status: New → Confirmed
importance: High → Medium
status: Confirmed → Triaged
Stephen M. Webb (bregma)
Changed in compiz:
milestone: 0.9.12.0 → 0.9.12.1
Stephen M. Webb (bregma)
Changed in compiz:
milestone: 0.9.12.1 → 0.9.12.2
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.