dee

Coverity DEADCODE - CID 10034

Series 1.0
Bug #937399

Bug #937399 reported by Product Strategy Coverity Bug Uploader on 2012-02-21

This bug affects 1 person

	Status	Importance	Assigned to
BAMF	Invalid	Medium	Unassigned
Legacy	Invalid	Medium	Unassigned
Messaging Menu	Invalid	Medium	Unassigned
12.10	Invalid	Undecided	Unassigned
dee	Status tracked in 1.0
1.0	Invalid	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: 10034
Checker: DEADCODE
Category: No category available
CWE definition: http://cwe.mitre.org/data/definitions/561.html
File: /tmp/buildd/bamf-0.2.106/build/gtk2/doc/reference/libbamf/libbamf-scan.c
Function: output_object_args()
Code snippet:
1115
1116 object_class_name = g_type_name (object_type);
1117
1118 child_prop = FALSE;
CID 10034 - DEADCODE
After this line, the value of "style_prop" is equal to 0.
Assigning: "style_prop" = "0".
1119 style_prop = FALSE;
1120
1121 while (TRUE) {
1122 qsort (properties, n_properties, sizeof (GParamSpec *), compare_param_specs);
1123 for (arg = 0; arg < n_properties; arg++)
1124 {

Tags:

Revision history for this message

Product Strategy Coverity Bug Uploader (coverity-uploader) wrote on 2012-02-21: bamf-trunk: /tmp/buildd/bamf-0.2.106/build/gtk2/doc/reference/libbamf/libbamf-scan.c

bamf-trunk: /tmp/buildd/bamf-0.2.106/build/gtk2/doc/reference/libbamf/libbamf-scan.c Edit (196.8 KiB, text/html)

Source file with Coverity annotations.

Changed in bamf:
importance:	Undecided → Medium

Revision history for this message

Product Strategy Coverity Bug Uploader (coverity-uploader) wrote on 2012-02-21: dee-1.0: /tmp/buildd/dee-1.0.4/doc/reference/dee-1.0/dee-1.0-scan.c

dee-1.0: /tmp/buildd/dee-1.0.4/doc/reference/dee-1.0/dee-1.0-scan.c Edit (203.5 KiB, text/html)

Source file with Coverity annotations.

Changed in dee:
importance:	Undecided → Medium

Revision history for this message

Product Strategy Coverity Bug Uploader (coverity-uploader) wrote on 2012-12-06: indicator-messages-12.10: /tmp/buildd/indicator-messages-12.10.6/doc/reference/messaging-menu-scan.c

indicator-messages-12.10: /tmp/buildd/indicator-messages-12.10.6/doc/reference/messaging-menu-scan.c Edit (193.9 KiB, text/html)

Source file with Coverity annotations.

Changed in indicator-messages:
importance:	Undecided → Medium

Revision history for this message

Charles Kerr (charlesk) wrote on 2012-12-06:

There are two things going on here:

1. In the small, Coverity is being confused by the #ifdefs. It's true that under certain #ifdef combinations that style_prop is a dead store, but that doesn't make this a bug.

2. More importantly, this is code that's autogenerated by gtk-doc just to scan the "real" source code, so if this were a bug (which it's not) it would belong to gtk-doc, rather than bamf, dee, and the messaging menu.

I recommend we tell Coverity to ignore this issue.