Compiz

Coverity MISSING_BREAK - CID 12464

Bug #1101549 reported by Product Strategy Coverity Bug Uploader on 2013-01-18

This bug affects 1 person

	Status	Importance	Assigned to
Compiz	Fix Released	High	MC Return
0.9.9	Fix Released	High	MC Return
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://wiki.ubuntu.com/CanonicalProductStrategy/Coverity
CID: 12464
Checker: MISSING_BREAK
Category: No category available
CWE definition: http://cwe.mitre.org/data/definitions/484.html
File: /tmp/buildd/compiz-0.9.9~daily13.01.14/plugins/annotate/src/annotate.cpp
Function: AnnoScreen::handleEvent(_XEvent *)
Code snippet:
964 switch (event->type) {
965 case MotionNotify:
966 handleMotionEvent (pointerX, pointerY);
967 case EnterNotify:
CID 12464 - MISSING_BREAK
This case (value 8) is not terminated by a 'break' statement.
968 case LeaveNotify:
969 handleMotionEvent (pointerX, pointerY);
CID 12464 - MISSING_BREAK
The above case falls through to this one.
970 default:
971 if (event->type == cScreen->damageEvent () + XDamageNotify)
972 {
973 XDamageNotifyEvent *de = (XDamageNotifyEvent *) event;

Tags:

Related branches

lp:~mc-return/compiz/compiz.merge-fix1101549-and-fix1101561-missing-break

Merged into lp:compiz/0.9.9 at revision 3589

Sam Spilsbury: Approve on 2013-01-29

MC Return: Needs Information on 2013-01-28

PS Jenkins bot: Pending (continuous-integration) requested 2013-01-30

lp:ubuntu/raring/compiz

Revision history for this message

Product Strategy Coverity Bug Uploader (coverity-uploader) wrote on 2013-01-18: compiz-0.9.9: /tmp/buildd/compiz-0.9.9~daily13.01.14/plugins/annotate/src/annotate.cpp

compiz-0.9.9: /tmp/buildd/compiz-0.9.9~daily13.01.14/plugins/annotate/src/annotate.cpp Edit (170.7 KiB, text/html)

Source file with Coverity annotations.

Changed in compiz:
importance:	Undecided → High

PS Jenkins bot (ps-jenkins) on 2013-01-30

Changed in compiz:
status:	In Progress → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2013-02-04:

This bug was fixed in the package compiz - 1:0.9.9~daily13.02.04-0ubuntu1

---------------
compiz (1:0.9.9~daily13.02.04-0ubuntu1) raring; urgency=low

  [ Marco Trevisan (Treviño) ]
  * Spread - Window glow is incorrectly painted in the screen when
    switching spreaded application (LP: #1109805)

  [ MC Return ]
  * Coverity MISSING_BREAK - CID 12463 (LP: #1101561)
  * Thumbnail plugin: Window title text is rendered into transparency
    and glow/background are too large (LP: #1099100)
  * Showmouse plugin code: Needs cleanup (LP: #1105969)
  * Coverity SECURE_CODING - CID 10019 (LP: #957582)
  * Coverity SECURE_CODING - CID 12511 (LP: #1101605)
  * Coverity SECURE_CODING - CID 12512 (LP: #1101571)
  * Keyboard shortcut overlay says Ctrl+Super+Down "minimises" the
    current window, but it doesn't (LP: #966099)
  * Coverity SECURE_CODING - CID 10020 (LP: #957587)
  * Coverity SECURE_CODING - CID 12529 (LP: #1101641)
  * Coverity MISSING_BREAK - CID 12464 (LP: #1101549)
  * Coverity SECURE_CODING - CID 12519 (LP: #1101565)
  * Coverity SECURE_CODING - CID 12516 (LP: #1101499)
  * [GLES] Showmouse plugin needs port to OpenGL|ES (LP: #1106270)