Compiz

[valgrind] Up to 520,000 bytes lost when running CCSGSettingsBackendConceptTest

Bug #1049169 reported by Sam Spilsbury on 2012-09-11

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Compiz	Fix Released	High	Sam Spilsbury	Compiz 0.9.8.4

Bug Description

==32325== LEAK SUMMARY:
==32325== definitely lost: 10,874 bytes in 246 blocks
==32325== indirectly lost: 10,220 bytes in 730 blocks
==32325== possibly lost: 512,856 bytes in 3,756 blocks
==32325== still reachable: 19,786 bytes in 182 blocks
==32325== suppressed: 1,304,127 bytes in 7,442 blocks
==32325== Reachable blocks (those to which a pointer was found) are not shown.
==32325== To see them, rerun with: --leak-check=full --show-reachable=yes

Tags:

Related branches

lp://staging/~compiz-team/compiz/compiz.tests_1041535

Superseded for merging into lp://staging/compiz/0.9.8

Daniel van Vugt: Needs Fixing on 2012-09-24

jenkins (community): Needs Fixing (continuous-integration) on 2012-09-24

Diff: 7931 lines (+5072/-1295)

76 files modified

compizconfig/CMakeLists.txt (+1/-0)
compizconfig/gsettings/CMakeLists.txt (+2/-0)
compizconfig/gsettings/gsettings_backend_shared/CMakeLists.txt (+0/-1)
compizconfig/gsettings/gsettings_backend_shared/ccs_gsettings_backend.c (+0/-23)
compizconfig/gsettings/gsettings_backend_shared/ccs_gsettings_backend_interface.c (+23/-0)
compizconfig/gsettings/mocks/CMakeLists.txt (+21/-0)
compizconfig/gsettings/tests/CMakeLists.txt (+19/-10)
compizconfig/gsettings/tests/compizconfig_ccs_gsettings_settings_env_test.cpp (+364/-0)
compizconfig/gsettings/tests/compizconfig_ccs_gsettings_settings_env_test.h (+134/-0)
compizconfig/gsettings/tests/gsettings-mock-schemas-config.h.in (+2/-0)
compizconfig/gsettings/tests/gsettings_settings_test_fixture.h (+29/-0)
compizconfig/gsettings/tests/test_gsettings_conformance.cpp (+25/-430)
compizconfig/gsettings/tests/test_gsettings_tests.h (+57/-14)
compizconfig/integration/gnome/CMakeLists.txt (+2/-0)
compizconfig/integration/gnome/gconf/src/ccs_gnome_integration_gconf_integrated_setting.c (+18/-3)
compizconfig/integration/gnome/gsettings/CMakeLists.txt (+2/-0)
compizconfig/integration/gnome/gsettings/src/ccs_gnome_integration_gsettings_integrated_setting.c (+62/-22)
compizconfig/integration/gnome/gsettings/tests/CMakeLists.txt (+31/-0)
compizconfig/integration/gnome/gsettings/tests/compizconfig_test_ccs_gnome_gsettings_integrated_setting.cpp (+388/-0)
compizconfig/integration/gnome/include/ccs_gnome_integrated_setting.h (+9/-1)
compizconfig/integration/gnome/include/ccs_gnome_integration_constants.h (+1/-0)
compizconfig/integration/gnome/include/ccs_gnome_integration_types.h (+2/-0)
compizconfig/integration/gnome/include/ccs_gnome_integration_types.h~ (+0/-26)
compizconfig/integration/gnome/src/ccs_gnome_integrated_setting.c (+18/-3)
compizconfig/integration/gnome/src/ccs_gnome_integration.c (+13/-10)
compizconfig/integration/gnome/tests/CMakeLists.txt (+47/-0)
compizconfig/integration/gnome/tests/compizconfig_ccs_mock_gnome_integrated_setting_composition.c (+222/-0)
compizconfig/integration/gnome/tests/compizconfig_ccs_mock_gnome_integrated_setting_composition.h (+42/-0)
compizconfig/integration/gnome/tests/compizconfig_test_ccs_gnome_integration.cpp (+813/-0)
compizconfig/libcompizconfig/src/main.c (+2/-2)
compizconfig/libcompizconfig/tests/CMakeLists.txt (+9/-1)
compizconfig/libcompizconfig/tests/compizconfig_test_ccs_mock_backend_conformance.cpp (+22/-10)
compizconfig/libcompizconfig/tests/compizconfig_test_ccs_settings_upgrade_internal.cpp (+13/-65)
compizconfig/mocks/libcompizconfig/CMakeLists.txt (+40/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integrated_setting_factory_mock.cpp (+70/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integrated_setting_factory_mock.h (+99/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integrated_setting_mock.cpp (+71/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integrated_setting_mock.h (+97/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integrated_setting_storage_mock.cpp (+73/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integrated_setting_storage_mock.h (+119/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integration_mock.cpp (+70/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_integration_mock.h (+129/-0)
compizconfig/tests/CMakeLists.txt (+71/-0)
compizconfig/tests/compizconfig_backend_concept_test.cpp (+381/-0)
compizconfig/tests/compizconfig_backend_concept_test.h (+144/-189)
compizconfig/tests/compizconfig_backend_concept_test_environment_interface.h (+101/-0)
compizconfig/tests/compizconfig_backend_concept_test_internal.h (+146/-0)
compizconfig/tests/compizconfig_ccs_item_in_list_matcher.cpp (+36/-0)
compizconfig/tests/compizconfig_ccs_item_in_list_matcher.h (+8/-13)
compizconfig/tests/compizconfig_ccs_list_equality.cpp (+74/-0)
compizconfig/tests/compizconfig_ccs_list_equality.h (+15/-29)
compizconfig/tests/compizconfig_ccs_list_wrapper.cpp (+109/-0)
compizconfig/tests/compizconfig_ccs_list_wrapper.h (+0/-199)
compizconfig/tests/compizconfig_ccs_setting_value_matcher.cpp (+69/-0)
compizconfig/tests/compizconfig_ccs_setting_value_matcher.h (+64/-0)
compizconfig/tests/compizconfig_ccs_setting_value_operators.cpp (+103/-0)
compizconfig/tests/compizconfig_ccs_setting_value_operators.h (+17/-52)
compizconfig/tests/compizconfig_ccs_settings_test_fixture.h (+106/-0)
compizconfig/tests/compizconfig_ccs_variant_types.h (+40/-0)
compizconfig/tests/compizconfig_test_value_combiners.cpp (+147/-0)
compizconfig/tests/compizconfig_test_value_combiners.h (+17/-82)
gtk/window-decorator/tests/CMakeLists.txt (+3/-1)
gtk/window-decorator/tests/test_gwd_metacity_decorations.cpp (+2/-3)
gtk/window-decorator/tests/test_gwd_settings.cpp (+65/-83)
plugins/composite/CMakeLists.txt (+5/-4)
plugins/decor/CMakeLists.txt (+4/-2)
plugins/expo/CMakeLists.txt (+4/-3)
plugins/grid/CMakeLists.txt (+5/-4)
plugins/place/CMakeLists.txt (+8/-6)
plugins/resize/CMakeLists.txt (+4/-2)
plugins/wall/CMakeLists.txt (+4/-2)
tests/shared/glib/glib_gsettings_memory_backend_env.h (+45/-0)
tests/shared/glib/glib_gslice_off_env.h (+43/-0)
tests/shared/gtest_shared_autodestroy.h (+20/-0)
tests/shared/gtest_shared_characterwrapper.h (+21/-0)
tests/shared/gtest_unspecified_bool_type_matcher.h (+30/-0)

lp://staging/~compiz-team/compiz/compiz.fix_1049169

Superseded for merging into lp://staging/compiz/0.9.8

jenkins (community): Needs Fixing (continuous-integration) on 2012-09-25

Compiz Maintainers: Pending requested 2012-09-25

lp://staging/~compiz-team/compiz/compiz.fix_1049169.1

Merged into lp://staging/compiz/0.9.8 at revision 3396

jenkins (community): Needs Fixing (continuous-integration) on 2012-09-26

Daniel van Vugt: Approve on 2012-09-26

lp://staging/~sil2100/compiz/ubuntu_0.9.8.4 (Merged)

Revision history for this message

Sam Spilsbury (smspillaz) wrote on 2012-09-11:

valgrind.log Edit (239.4 KiB, text/plain)

Changed in compiz:
status:	New → Confirmed
importance:	Undecided → High
assignee:	nobody → Sam Spilsbury (smspillaz)
milestone:	none → 0.9.8.4

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2012-09-12:

/me falls off chair

Revision history for this message

Sam Spilsbury (smspillaz) wrote on 2012-09-12:

To clarify, I think its just that we're leaking an object that has a lot of internal data, so its not too big a leak, and only happens in the testcase.

Daniel van Vugt (vanvugt) on 2012-09-24

Changed in compiz:
status:	Confirmed → In Progress

Unity Merger (unity-merger) on 2012-09-26

Changed in compiz:
status:	In Progress → Fix Committed

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2012-09-26:

Fix committed into lp:compiz at revision 3396

Daniel van Vugt (vanvugt) on 2012-09-28

Changed in compiz:
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

Bug attachments

valgrind.log Edit

Add attachment

Remote bug watches

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