Merge lp://staging/~phablet-team/network-manager/fix-ofono-plugin-mem-leaks-wily into lp://staging/~network-manager/network-manager/ubuntu

Proposed by Tony Espy
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: 998
Merged at revision: 999
Proposed branch: lp://staging/~phablet-team/network-manager/fix-ofono-plugin-mem-leaks-wily
Merge into: lp://staging/~network-manager/network-manager/ubuntu
Prerequisite: lp://staging/~phablet-team/network-manager/lp1480877-wifi-rm-dup-scan-signals
Diff against target: 140 lines (+25/-9)
4 files modified
debian/changelog (+1/-0)
debian/patches/CVE-2015-1322.patch (+2/-2)
debian/patches/add_ofono_settings_support.patch (+21/-6)
debian/patches/lp1425172-fix-wifi-scanlist-cull.patch (+1/-1)
To merge this branch: bzr merge lp://staging/~phablet-team/network-manager/fix-ofono-plugin-mem-leaks-wily
Reviewer Review Type Date Requested Status
Alfonso Sanchez-Beato (community) Approve
Mathieu Trudel-Lapierre Approve
Review via email: mp+271682@code.staging.launchpad.net

Description of the change

Fix miscellaneous code leaks in ofono settings plugin discovered during work on duplicate DBus signals problem ( lp: #1480877 ).

To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Looks fine.

review: Approve
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

LGTM

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