Coverity CHECKED_RETURN - CID 12386 - compizconfig/libcompizconfig/src/compiz.cpp - in function: loadPluginFromXMLFile(_CCSContext *, char *, char *) - Calling function "remove(pbFilePath)" without checking return value. This library function may fail and return an error code. No check of the return value of "remove(pbFilePath)".

Bug #1101617 reported by Product Strategy Coverity Bug Uploader
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
New
Medium
Unassigned
0.9.9
New
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: 12386
Checker: CHECKED_RETURN
Category: library_function
CWE definition: http://cwe.mitre.org/data/definitions/252.html
File: /tmp/buildd/compiz-0.9.9~daily13.01.14/compizconfig/libcompizconfig/src/compiz.cpp
Function: loadPluginFromXMLFile(_CCSContext *, char *, char *)
Code snippet:
2927 #ifdef USE_PROTOBUF
2928 if (usingProtobuf && xmlLoaded)
2929 {
2930 if (removePB)
CID 12386 - CHECKED_RETURN
Calling function "remove(pbFilePath)" without checking return value. This library function may fail and return an error code.
No check of the return value of "remove(pbFilePath)".
2931 remove (pbFilePath); // Attempt to remove .pb
2932 writePBFile (pbFilePath, NULL, &persistentPluginBriefPB, &xmlStat);
2933 updatePBFilePath (context, name, pbFilePath);
2934 }
2935
2936 if (pbFilePath)

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 → Medium
MC Return (mc-return)
summary: - Coverity CHECKED_RETURN - CID 12386
+ Coverity CHECKED_RETURN - CID 12386 -
+ compizconfig/libcompizconfig/src/compiz.cpp - in function:
+ loadPluginFromXMLFile(_CCSContext *, char *, char *) - Calling function
+ "remove(pbFilePath)" without checking return value. This library
+ function may fail and return an error code. No check of the return value
+ of "remove(pbFilePath)".
Changed in compiz:
milestone: none → 0.9.10.0
Changed in compiz:
milestone: 0.9.10.0 → 0.9.10.2
MC Return (mc-return)
Changed in compiz:
milestone: 0.9.10.2 → 0.9.11.0
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.