unity-scopes-api

Lots of valgrind complaints for smartscopes tests

Bug #1262496 reported by Michi Henning
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity-scopes-api
Fix Released
Undecided
Marcus Tomlinson
unity-scopes-api (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Running the tests with "make valgrind" (even after applying https://code.launchpad.net/~michihenning/unity-scopes-api/valgrind-fix/+merge/199605) reports lots of errors. (Look in build/Testing/Temporary/LastDynamicAnalysis<date>.log)

It looks like these are all Qt related. Possibly not due to anything we are doing, but it would be good to be sure. If these are not bogus, they need fixing. Otherwise, if they are harmless, we should add supressions to valgrind-suppress.

Related branches

lp:~marcustomlinson/unity-scopes-api/lp-1262496
Michi Henning (community): Approve
PS Jenkins bot (community): Approve (continuous-integration)
Diff: 28 lines (+21/-0)
1 file modified
valgrind-suppress (+21/-0)
lp:ubuntu/trusty/unity-scopes-api
lp:~marcustomlinson/unity-scopes-api/lp-1272147
Michi Henning (community): Approve
PS Jenkins bot (community): Approve (continuous-integration)
Diff: 112 lines (+32/-10)
4 files modified
include/unity/scopes/internal/smartscopes/SSRegistryObject.h (+2/-1)
src/scopes/internal/smartscopes/SSRegistryObject.cpp (+4/-2)
test/gtest/scopes/internal/smartscopes/smartscopesproxy/smartscopesproxy_test.cpp (+7/-5)
valgrind-suppress (+19/-2)
Superseded for merging into lp:unity-scopes-api
Michi Henning: Pending requested
Diff: 132 lines (+38/-12)
5 files modified
include/CMakeLists.txt (+6/-2)
include/unity/scopes/internal/smartscopes/SSRegistryObject.h (+2/-1)
src/scopes/internal/smartscopes/SSRegistryObject.cpp (+4/-2)
test/gtest/scopes/internal/smartscopes/smartscopesproxy/smartscopesproxy_test.cpp (+7/-5)
valgrind-suppress (+19/-2)
Michi Henning (michihenning)
Changed in unity-scopes-api:
assignee: nobody → Marcus Tomlinson (marcustomlinson)
Marcus Tomlinson (marcustomlinson)
Changed in unity-scopes-api:
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity-scopes-api at revision 146, scheduled for release in unity-scopes-api, milestone Unknown

Changed in unity-scopes-api:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity-scopes-api - 0.2.4+14.04.20140124-0ubuntu1

---------------
unity-scopes-api (0.2.4+14.04.20140124-0ubuntu1) trusty; urgency=low

  [ Michi Henning ]
  * Added ability for scope to push an exception (as an exception_ptr). On the client side,
    the exception is delivered as the what() string (if the exception is a std::exception) and
    as "unknown exception", otherwise.
  * Added abstract bases for all the *Object classes.
  * Remove redundant noexcept specifications from destructors.

  [ Jussi Pakkanen ]
  * Version bump due to namespace peeling.
  * Forward doxygen warnings into a file.

  [ Michal Hruby ]
  * Added preview and activation APIs.

  [ Michi Henning ]
  * Added default servants for Scope and QueryCtrl interfaces.

  [ Pawel Stolowski ]
  * Added set_category to CategorisedResult, removed Variant-based ctor.

  [ Marcus Tomlinson ]
  * Ensure that manager_ and reply_ objects are created and destroyed
    from within the same Qt thread. (LP: #1267917)
  * Added valgrind suppressions for memory leak false positives in Qt.
    (LP: #1262496)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 151
 -- Ubuntu daily release <email address hidden> Fri, 24 Jan 2014 10:03:35 +0000

Changed in unity-scopes-api (Ubuntu):
status: New → Fix Released
Revision history for this message
Michal Hruby (mhr3) wrote :

We still need a few more supressions, re-opening.

Changed in unity-scopes-api:
status: Fix Committed → Triaged
Marcus Tomlinson (marcustomlinson)
Changed in unity-scopes-api:
status: Triaged → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity-scopes-api/devel at revision 175, scheduled for release in unity-scopes-api, milestone Unknown

Changed in unity-scopes-api:
status: In Progress → Fix Committed
Michal Hruby (mhr3)
Changed in unity-scopes-api:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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