← Back to team overview

dx-packages team mailing list archive

[Bug 1112844] Re: Coverity CHECKED_RETURN - CID 11391

 

** Changed in: indicator-appmenu
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of DX
Packages, which is subscribed to indicator-appmenu in Ubuntu.
Matching subscriptions: dx-packages
https://bugs.launchpad.net/bugs/1112844

Title:
  Coverity CHECKED_RETURN - CID 11391

Status in The Application Menu:
  Fix Released
Status in “indicator-appmenu” package in Ubuntu:
  Fix Released

Bug description:
  A Coverity bug could not be exported due to the nature of its shared
  project--I'll attach a pretty-printed report here.

  
  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: 11391
  Checker: CHECKED_RETURN
  Category: library_function
  CWE definition: http://cwe.mitre.org/data/definitions/252.html
  File: /tmp/buildd/hud-13.04.0/src/usage-tracker.c
  Function: configure_db
  Code snippet:
  210 }
  211
  212 gchar * cachedir = g_build_filename(basecachedir, "indicator-appmenu", NULL);
  213 if (!g_file_test(cachedir, G_FILE_TEST_EXISTS | G_FILE_TEST_IS_DIR)) {
  CID 11391 - CHECKED_RETURN
  Calling function "mkdir(cachedir, 448U)" without checking return value. This library function may fail and return an error code.
  No check of the return value of "mkdir(cachedir, 448U)".
  214 g_mkdir(cachedir, 1 << 6 | 1 << 7 | 1 << 8); // 700
  215 }
  216 g_free(cachedir);
  217
  218 self->priv->cachefile = g_build_filename(basecachedir, "indicator-appmenu", "hud-usage-log.sqlite", NULL);
  219 db_exists = g_file_test(self->priv->cachefile, G_FILE_TEST_EXISTS);

To manage notifications about this bug go to:
https://bugs.launchpad.net/indicator-appmenu/+bug/1112844/+subscriptions