Several memory leaks in strdup() from ccsGSettingsIntegratedSettingReadValue() from ccsGNOMEIntegrationBackendReadISAndSetSettingForType() from ccsGNOMEIntegrationBackendReadOptionIntoSetting()

Bug #1100564 reported by Daniel van Vugt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
Fix Released
High
Sam Spilsbury
compiz (Ubuntu)
Fix Released
High
Sam Spilsbury

Bug Description

Several memory leaks in strdup() from ccsGSettingsIntegratedSettingReadValue() from ccsGNOMEIntegrationBackendReadISAndSetSettingForType() from ccsGNOMEIntegrationBackendReadOptionIntoSetting()

==29374== 121 bytes in 12 blocks are definitely lost in loss record 1,025 of 1,293
==29374== at 0x4C2CD7B: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==29374== by 0x5688961: strdup (strdup.c:42)
==29374== by 0xAFBAF2C: ccsGSettingsIntegratedSettingReadValue (ccs_gnome_integration_gsettings_integrated_setting.c:142)
==29374== by 0xAFB9AB2: ccsGNOMEIntegrationBackendReadISAndSetSettingForType (ccs_gnome_integration.c:177)
==29374== by 0xAFB9C24: ccsGNOMEIntegrationBackendReadOptionIntoSetting (ccs_gnome_integration.c:248)
==29374== by 0xAFB8DEA: readSetting (gsettings.c:68)
==29374== by 0xA2FF52A: ccsReadPluginSettingsDefault (main.c:3493)
==29374== by 0xA2F1A50: ccsLoadPluginSettings (compiz.cpp:3254)
==29374== by 0xA2FE341: ccsContextNew (main.c:552)
==29374== by 0xA0C9AFE: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:509)
==29374== by 0xA0CBE98: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
==29374== by 0x4EB1AD5: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
==29374==
==29374== 121 bytes in 12 blocks are definitely lost in loss record 1,026 of 1,293
==29374== at 0x4C2CD7B: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==29374== by 0x5688961: strdup (strdup.c:42)
==29374== by 0xAFBAF2C: ccsGSettingsIntegratedSettingReadValue (ccs_gnome_integration_gsettings_integrated_setting.c:142)
==29374== by 0xAFB9AB2: ccsGNOMEIntegrationBackendReadISAndSetSettingForType (ccs_gnome_integration.c:177)
==29374== by 0xAFB9C24: ccsGNOMEIntegrationBackendReadOptionIntoSetting (ccs_gnome_integration.c:248)
==29374== by 0xAFB8DEA: readSetting (gsettings.c:68)
==29374== by 0xA2FBC46: ccsReadSettingsDefault (main.c:3451)
==29374== by 0xA0C9B0A: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:510)
==29374== by 0xA0CBE98: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
==29374== by 0x4EB1AD5: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
==29374== by 0x4EB23A0: CompPlugin::push(CompPlugin*) (plugin.cpp:449)
==29374== by 0x4E83E53: compiz::private_screen::PluginManager::updatePlugins(CompScreen*, std::vector<CompOption::Value, std::allocator<CompOption::Value> > const&) (screen.cpp:1120)
==29374==
==29374== 194 bytes in 10 blocks are definitely lost in loss record 1,088 of 1,293
==29374== at 0x4C2CD7B: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==29374== by 0x5688961: strdup (strdup.c:42)
==29374== by 0xAFBAF2C: ccsGSettingsIntegratedSettingReadValue (ccs_gnome_integration_gsettings_integrated_setting.c:142)
==29374== by 0xAFB9AB2: ccsGNOMEIntegrationBackendReadISAndSetSettingForType (ccs_gnome_integration.c:177)
==29374== by 0xAFB9C24: ccsGNOMEIntegrationBackendReadOptionIntoSetting (ccs_gnome_integration.c:248)
==29374== by 0xAFB8DEA: readSetting (gsettings.c:68)
==29374== by 0xA2FF52A: ccsReadPluginSettingsDefault (main.c:3493)
==29374== by 0xA2F1A50: ccsLoadPluginSettings (compiz.cpp:3254)
==29374== by 0xA2FF65C: ccsFindSettingDefault (main.c:609)
==29374== by 0xA0CA754: CcpScreen::setOptionFromContext(CompOption*, char const*) (ccp.cpp:375)
==29374== by 0xA0CB3C6: CcpScreen::initPluginForScreen(CompPlugin*) (ccp.cpp:496)
==29374== by 0x4EB0EA3: CompScreen::initPluginForScreen(CompPlugin*) (plugin.cpp:268)
==29374==

Tags: leak

Related branches

Changed in compiz (Ubuntu):
status: New → Triaged
importance: Undecided → High
Changed in compiz:
status: Triaged → In Progress
Changed in compiz (Ubuntu):
status: Triaged → In Progress
Changed in compiz (Ubuntu):
assignee: nobody → Sam Spilsbury (smspillaz)
Changed in compiz:
status: In Progress → Fix Committed
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Fix committed into lp:compiz at revision 3572, scheduled for release in Compiz 0.9.9.0

Changed in compiz (Ubuntu):
status: In Progress → Fix Committed
Changed in compiz:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package compiz - 1:0.9.9~daily13.01.25-0ubuntu1

---------------
compiz (1:0.9.9~daily13.01.25-0ubuntu1) raring; urgency=low

  [ Didier Roche ]
  * Really take default patched hsize and vsize and don't override them
    to fix (LP: #868423)

  [ sampo555 ]
  * [regression] Window resize granularity is lost when restored after
    using Grid (LP: #925867)
  * Window management - Restoring a grid-placed window by dragging the
    title bar downwards does not restore the original window width (LP:
    #878516)

  [ Daniel van Vugt ]
  * Several memory leaks in
    google::protobuf::DescriptorPool::InternalAddGeneratedFile() from
    google::protobuf::protobuf_AddDesc_*() (LP: #1102829)
  * Several leaks in g_settings_new() [g_object_new()] from
    ccsGSettingsNewNoPath() [ccs_gsettings_interface_wrapper.c:184] (LP:
    #1097649)
  * Several memory leaks in g_signal_new() ... from
    ccsGSettingsWrapperNewForSchema()
    [ccs_gsettings_interface_wrapper.c:184] from initBackend()
    [gsettings.c:468] (LP: #1102822)

  [ Sam Spilsbury ]
  * Several memory leaks in strdup() from
    ccsGSettingsIntegratedSettingReadValue() from
    ccsGNOMEIntegrationBackendReadISAndSetSettingForType() from
    ccsGNOMEIntegrationBackendReadOptionIntoSetting() (LP: #1100564)

  [ MC Return ]
  * Coverity MISSING_BREAK - CID 12468 (LP: #1101430)
  * Coverity MISSING_BREAK - CID 12466 (LP: #1101558)
  * Coverity MISSING_BREAK - CID 12467 (LP: #1101465)
  * Showmouse plugin: Possible values of the particle life setting can
    destroy the plugin's functionality (LP: #1098877)
  * Coverity MISSING_BREAK - CID 12465 (LP: #1101557)

  [ Automatic PS uploader ]
  * Automatic snapshot from revision 3582
 -- Automatic PS uploader <email address hidden> Fri, 25 Jan 2013 04:03:43 +0000

Changed in compiz (Ubuntu):
status: Fix Committed → Fix Released
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.