Compiz

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

Bug #1100564 reported by Daniel van Vugt on 2013-01-17

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Compiz	Fix Released	High	Sam Spilsbury	Compiz 0.9.9.0
	compiz (Ubuntu)	Fix Released	High	Sam Spilsbury

Bug Description

==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:

Related branches

lp:~compiz-team/compiz/compiz.fix_1100564

Merged into lp:compiz/0.9.9 at revision 3572

Daniel van Vugt: Approve on 2013-01-22

PS Jenkins bot (community): Needs Fixing (continuous-integration) on 2013-01-19

lp:ubuntu/raring/compiz

Daniel van Vugt (vanvugt) on 2013-01-17

Changed in compiz (Ubuntu):
status:	New → Triaged
importance:	Undecided → High

Daniel van Vugt (vanvugt) on 2013-01-17

Changed in compiz:
status:	Triaged → In Progress
Changed in compiz (Ubuntu):
status:	Triaged → In Progress

Daniel van Vugt (vanvugt) on 2013-01-17

Changed in compiz (Ubuntu):
assignee:	nobody → Sam Spilsbury (smspillaz)

PS Jenkins bot (ps-jenkins) on 2013-01-22

Changed in compiz:
status:	In Progress → Fix Committed

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2013-01-23:

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

Daniel van Vugt (vanvugt) on 2013-01-24

Changed in compiz:
status:	Fix Committed → Fix Released

Revision history for this message

Launchpad Janitor (janitor) wrote on 2013-01-25:

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

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.