Merge lp://staging/~vanvugt/mir/fix-1391976-0.14 into lp://staging/mir/0.14
Proposed by
Daniel van Vugt
| Status: | Merged |
|---|---|
| Merged at revision: | 2703 |
| Proposed branch: | lp://staging/~vanvugt/mir/fix-1391976-0.14 |
| Merge into: | lp://staging/mir/0.14 |
| Diff against target: |
169 lines (+96/-1) 7 files modified
debian/changelog (+2/-0) debian/mir-test-tools.install (+1/-0) src/protobuf/CMakeLists.txt (+1/-1) src/protobuf/google_protobuf_guard.cpp (+9/-0) tests/CMakeLists.txt (+1/-0) tests/loader-tests/CMakeLists.txt (+33/-0) tests/loader-tests/test_reload.c (+49/-0) |
| To merge this branch: | bzr merge lp://staging/~vanvugt/mir/fix-1391976-0.14 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Andreas Pokorny (community) | Approve | ||
| Kevin DuBois (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Backport fix for LP: #1391976 -
Loading libmirclient.so twice leads to a segfault in libmirprotobuf.so
We're going to need this fix released in libmirprotobuf.so.0 at least
once before it will ever be safe to go to libmirprotobuf.so.1 in
subsequent releases.
Description of the change
This is also the only bug fix landed in 0.15 that can be backported to 0.14
To post a comment you must log in.
Revision history for this message
| Kevin DuBois (kdub) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
havent started testing - i am fine with cherry picking that.