Coverity FORWARD_NULL - CID 12444 - compizconfig/libcompizconfig/src/compiz.cpp - in function: loadPluginFromXMLFile(_CCSContext *, char *, char *) - Passing null pointer "pbFilePath" to function "writePBFile(char *, PluginMetadata *, PluginBriefMetadata *, stat *)", which dereferences it. Passing null pointer "pbFilePath" to function "updatePBFilePath(CCSContext *, char *, char *)", which dereferences it. (2x FORWARD_NULL)

Bug #1101527 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: 12444
Checker: FORWARD_NULL
Category: deref_constant_null
CWE definition: http://cwe.mitre.org/data/definitions/476.html
File: /tmp/buildd/compiz-0.9.9~daily13.01.14/compizconfig/libcompizconfig/src/compiz.cpp
Function: loadPluginFromXMLFile(_CCSContext *, char *, char *)
Code snippet:
2829 }
2830
2831 #ifdef USE_PROTOBUF
2832 char *name = NULL;
CID 12444 - FORWARD_NULL
Assigning: "pbFilePath" = "NULL".
Assigning: "pbFilePath" = "NULL".
2833 char *pbFilePath = NULL;
2834 struct stat xmlStat;
2835 Bool removePB = FALSE;
2836
2837 if (usingProtobuf)
2838 {

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/compizconfig/libcompizconfig/src/compiz.cpp

Source file with Coverity annotations.

Changed in compiz:
importance: Undecided → High
MC Return (mc-return)
summary: - Coverity FORWARD_NULL - CID 12444
+ Coverity FORWARD_NULL - CID 12444 -
+ compizconfig/libcompizconfig/src/compiz.cpp - in function:
+ loadPluginFromXMLFile(_CCSContext *, char *, char *) - Passing null
+ pointer "pbFilePath" to function "writePBFile(char *, PluginMetadata *,
+ PluginBriefMetadata *, stat *)", which dereferences it. Passing null
+ pointer "pbFilePath" to function "updatePBFilePath(CCSContext *, char *,
+ char *)", which dereferences it. (2x FORWARD_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)
no longer affects: compiz/0.9.9
Changed in compiz:
importance: High → Medium
milestone: 0.9.11.0 → 0.9.12.0
status: New → 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.