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

Bug #1100564 reported by Daniel van Vugt on 2013-01-17
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
High
Sam Spilsbury
compiz (Ubuntu)
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==

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
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
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  Edit
Everyone can see this information.

Other bug subscribers