compiz team mailing list archive

[Daniel van Vugt <daniel.van.vugt@xxxxxxxxxxxxx>]

Public bug reported:

Several leaks in ccsIntegratedSettingListAppend() ... from
ccsGNOMEIntegrationBackendGetIntegratedSetting() from readSetting
(gsettings.c:375)

These look like they might be recurring at runtime. So probably more
severe than the other leaks found today...

==9349== 288 bytes in 18 blocks are definitely lost in loss record 1,151 of 1,327
==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
==9349==    by 0xA31A9AA: ccsReadPluginSettingsDefault (main.c:3493)
==9349==    by 0xA30CB4B: ccsLoadPluginSettings (compiz.cpp:3254)
==9349==    by 0xA3197B1: ccsContextNew (main.c:552)
==9349==    by 0xA0DE0FE: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:509)
==9349==    by 0xA0E03A8: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
==9349==    by 0x4EAF425: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
==9349==
==9349== 288 bytes in 18 blocks are definitely lost in loss record 1,152 of 1,327
==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
==9349==    by 0xA3170B6: ccsReadSettingsDefault (main.c:3451)
==9349==    by 0xA0DE10A: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:510)
==9349==    by 0xA0E03A8: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
==9349==    by 0x4EAF425: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
==9349==    by 0x4EAFCF0: CompPlugin::push(CompPlugin*) (plugin.cpp:449)
==9349==    by 0x4E83447: compiz::private_screen::PluginManager::updatePlugins(CompScreen*, std::vector<CompOption::Value, std::allocator<CompOption::Value> > const&) (screen.cpp:1120)

... and more severe:

==9349== 512 bytes in 32 blocks are definitely lost in loss record 1,203 of 1,327
==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
==9349==    by 0xA31A9AA: ccsReadPluginSettingsDefault (main.c:3493)
==9349==    by 0xA30CB4B: ccsLoadPluginSettings (compiz.cpp:3254)
==9349==    by 0xA31AADC: ccsFindSettingDefault (main.c:609)
==9349==    by 0xA0DEE08: CcpScreen::setOptionFromContext(CompOption*, char const*) (ccp.cpp:375)
==9349==    by 0xA0DF8D6: CcpScreen::initPluginForScreen(CompPlugin*) (ccp.cpp:496)
==9349==    by 0x4EAE7F3: CompScreen::initPluginForScreen(CompPlugin*) (plugin.cpp:268)

** Affects: compiz
     Importance: High
         Status: Triaged

** Affects: compiz/0.9.8
     Importance: High
         Status: Triaged

** Affects: compiz (Ubuntu)
     Importance: High
         Status: Triaged


** Tags: leak

** Also affects: compiz/0.9.8
   Importance: Undecided
       Status: New

** Changed in: compiz/0.9.8
       Status: New => Triaged

** Changed in: compiz/0.9.8
   Importance: Undecided => High

** Changed in: compiz/0.9.8
    Milestone: None => 0.9.8.8

** Also affects: compiz (Ubuntu)
   Importance: Undecided
       Status: New

** Changed in: compiz (Ubuntu)
       Status: New => Triaged

** Changed in: compiz (Ubuntu)
   Importance: Undecided => High

** Description changed:

  Several leaks in ccsIntegratedSettingListAppend() ... from
  ccsGNOMEIntegrationBackendGetIntegratedSetting() from readSetting
  (gsettings.c:375)
  
  These look like they might be recurring at runtime. So probably more
  severe than the other leaks found today...
  
  ==9349== 288 bytes in 18 blocks are definitely lost in loss record 1,151 of 1,327
  ==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
  ==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
  ==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
  ==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
  ==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
  ==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
  ==9349==    by 0xA31A9AA: ccsReadPluginSettingsDefault (main.c:3493)
  ==9349==    by 0xA30CB4B: ccsLoadPluginSettings (compiz.cpp:3254)
  ==9349==    by 0xA3197B1: ccsContextNew (main.c:552)
  ==9349==    by 0xA0DE0FE: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:509)
  ==9349==    by 0xA0E03A8: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
  ==9349==    by 0x4EAF425: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
- ==9349== 
+ ==9349==
  ==9349== 288 bytes in 18 blocks are definitely lost in loss record 1,152 of 1,327
  ==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
  ==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
  ==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
  ==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
  ==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
  ==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
  ==9349==    by 0xA3170B6: ccsReadSettingsDefault (main.c:3451)
  ==9349==    by 0xA0DE10A: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:510)
  ==9349==    by 0xA0E03A8: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
  ==9349==    by 0x4EAF425: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
  ==9349==    by 0x4EAFCF0: CompPlugin::push(CompPlugin*) (plugin.cpp:449)
  ==9349==    by 0x4E83447: compiz::private_screen::PluginManager::updatePlugins(CompScreen*, std::vector<CompOption::Value, std::allocator<CompOption::Value> > const&) (screen.cpp:1120)
+ 
+ ... and more.

** Description changed:

  Several leaks in ccsIntegratedSettingListAppend() ... from
  ccsGNOMEIntegrationBackendGetIntegratedSetting() from readSetting
  (gsettings.c:375)
  
  These look like they might be recurring at runtime. So probably more
  severe than the other leaks found today...
  
  ==9349== 288 bytes in 18 blocks are definitely lost in loss record 1,151 of 1,327
  ==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
  ==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
  ==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
  ==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
  ==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
  ==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
  ==9349==    by 0xA31A9AA: ccsReadPluginSettingsDefault (main.c:3493)
  ==9349==    by 0xA30CB4B: ccsLoadPluginSettings (compiz.cpp:3254)
  ==9349==    by 0xA3197B1: ccsContextNew (main.c:552)
  ==9349==    by 0xA0DE0FE: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:509)
  ==9349==    by 0xA0E03A8: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
  ==9349==    by 0x4EAF425: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
  ==9349==
  ==9349== 288 bytes in 18 blocks are definitely lost in loss record 1,152 of 1,327
  ==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
  ==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
  ==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
  ==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
  ==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
  ==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
  ==9349==    by 0xA3170B6: ccsReadSettingsDefault (main.c:3451)
  ==9349==    by 0xA0DE10A: CcpScreen::CcpScreen(CompScreen*) (ccp.cpp:510)
  ==9349==    by 0xA0E03A8: CompPlugin::VTableForScreen<CcpScreen>::initScreen(CompScreen*) (plugin.h:282)
  ==9349==    by 0x4EAF425: CompManager::initPlugin(CompPlugin*) (plugin.cpp:233)
  ==9349==    by 0x4EAFCF0: CompPlugin::push(CompPlugin*) (plugin.cpp:449)
  ==9349==    by 0x4E83447: compiz::private_screen::PluginManager::updatePlugins(CompScreen*, std::vector<CompOption::Value, std::allocator<CompOption::Value> > const&) (screen.cpp:1120)
  
- ... and more.
+ ... and more severe:
+ 
+ ==9349== 512 bytes in 32 blocks are definitely lost in loss record 1,203 of 1,327
+ ==9349==    at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
+ ==9349==    by 0xA313905: ccsIntegratedSettingListAppend (lists.c:185)
+ ==9349==    by 0xA3149FC: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPredicate (main.c:5811)
+ ==9349==    by 0xA31DB7B: ccsIntegratedSettingsStorageDefaultFindMatchingSettingsByPluginAndSettingName (main.c:5830)
+ ==9349==    by 0xAFD03C4: ccsGNOMEIntegrationBackendGetIntegratedSetting (ccs_gnome_integration.c:118)
+ ==9349==    by 0xAFCFC88: readSetting (gsettings.c:375)
+ ==9349==    by 0xA31A9AA: ccsReadPluginSettingsDefault (main.c:3493)
+ ==9349==    by 0xA30CB4B: ccsLoadPluginSettings (compiz.cpp:3254)
+ ==9349==    by 0xA31AADC: ccsFindSettingDefault (main.c:609)
+ ==9349==    by 0xA0DEE08: CcpScreen::setOptionFromContext(CompOption*, char const*) (ccp.cpp:375)
+ ==9349==    by 0xA0DF8D6: CcpScreen::initPluginForScreen(CompPlugin*) (ccp.cpp:496)
+ ==9349==    by 0x4EAE7F3: CompScreen::initPluginForScreen(CompPlugin*) (plugin.cpp:268)

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

Title:
  Several leaks in ccsIntegratedSettingListAppend() ... from
  ccsGNOMEIntegrationBackendGetIntegratedSetting() from readSetting
  (gsettings.c:375)

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