desktop-packages team mailing list archive

["Stephen M. Webb" <stephen.webb@xxxxxxxxxxxxx>]

** Changed in: compiz/0.9.9
       Status: New => Won't Fix

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to compiz in Ubuntu.
https://bugs.launchpad.net/bugs/1101405

Title:
  Coverity UNINIT - CID 12557- src/session.cpp - pointer_deref_read -
  Declaring variable "args" without initializer in
  setCloneRestartCommands(_SmcConn *)

Status in Compiz:
  Fix Released
Status in Compiz 0.9.9 series:
  Won't Fix
Status in “compiz” package in Ubuntu:
  Fix Released

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: 12557
  Checker: UNINIT
  Category: pointer_deref_read
  CWE definition: http://cwe.mitre.org/data/definitions/457.html
  File: /tmp/buildd/compiz-0.9.9~daily13.01.14/src/session.cpp
  Function: setCloneRestartCommands(_SmcConn *)
  Code snippet:
   85 
   86 static void
   87 setCloneRestartCommands (SmcConn connection)
   88 {
  CID 12557 - UNINIT
  Declaring variable "args" without initializer.
   89     const char **args;
   90 
   91     /* at maximum, we pass our old arguments + our new client id
   92        to the SM, so allocate for that case */
  CID 12557 - UNINIT
  Calling allocator "malloc(size_t)".
  Assigning: "args" = "(char const **)malloc((programArgc + 2) * 8UL)", which is allocated but not initialized.
   93     args = (const char **) malloc ((programArgc + 2) * sizeof (char *));
   94     if (!args)

To manage notifications about this bug go to:
https://bugs.launchpad.net/compiz/+bug/1101405/+subscriptions