DBus Menu

Merge lp://staging/~chrisccoulson/libdbusmenu/lp1104136 into lp://staging/libdbusmenu/13.04

  1. lp1104136
  2. Merge into trunk.13.04
Proposed by Chris Coulson
Status: Merged
Approved by: Ted Gould
Approved revision: 440
Merged at revision: 440
Proposed branch: lp://staging/~chrisccoulson/libdbusmenu/lp1104136
Merge into: lp://staging/libdbusmenu/13.04
Diff against target: 88 lines (+14/-5)
2 files modified
libdbusmenu-glib/menuitem.c (+3/-1)
libdbusmenu-glib/server.c (+11/-4)
To merge this branch: bzr merge lp://staging/~chrisccoulson/libdbusmenu/lp1104136
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+144733@code.staging.launchpad.net

Description of the change

Fix multiple leaks due to improper use of g_variant_parse() (fixes bug 1104136)

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:440
http://jenkins.qa.ubuntu.com/job/dbusmenu-ci/25/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dbusmenu-ci/./build=pbuilder,distribution=raring,flavor=amd64/25/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dbusmenu-ci/./build=pbuilder,distribution=raring,flavor=i386/25/console

Click here to trigger a rebuild:
http://jenkins.qa.ubuntu.com/job/dbusmenu-ci/25//rebuild/?

review: Approve (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote :

Hate GVariants.... Can't believe this isn't a floating reference.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: