GitHub user alanconway opened a pull request:
https://github.com/apache/qpid-dispatch/pull/240
Dispatch memory fixes
These are fixes for valgrind errors in the unit tests that I saw when
running tests on master. There are more to fix, in the system tests.
Requesting review as I have been away from dispatch for a while, don't want
to cause a regression. ctest with my changes is no worse than without - the
policy and user_id_proxy system tests are failing with timeouts for me on
master with/without these changes.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/alanconway/dispatch dispatch-mem
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/qpid-dispatch/pull/240.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #240
----
commit 5738079016236efb02554c6cd4536c4d8295f59a
Author: Alan Conway <aconway@...>
Date: 2018-01-02T15:37:58Z
DISPATCH-739: Fix memory leaks in unit tests, missing frees
Add missing frees in tests.
commit 4c98f54136c7e5c306be540d53be24423ece714e
Author: Alan Conway <aconway@...>
Date: 2018-01-02T15:39:09Z
DISPATCH-739: Fix memory leak in http server showing in unit tests
Maybe not a leak in a deployed broker but fixed it anyway to keep tests
clean.
commit 892a413f84753de859e5d84a425b241197e7efe1
Author: Alan Conway <aconway@...>
Date: 2018-01-02T16:00:06Z
DISPATCH-739: Fix invalid read valgrind error in tests.
Changed server shutdown order so that context objects are not used after
free
by logger.
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
