Here is one of the failed CIs with memory leak logs: https://travis-ci.org/tlyu/tor/jobs/499009440#L4956

Here are the pull requests for review:

0.2.9 pull request: https://github.com/torproject/tor/pull/730

0.3.3 pull request (function name change): https://github.com/torproject/tor/pull/731

I also did a merge to master to test that there aren't any memory leaks in other tests:

master pull request: (CI only - no changes from 0.3.3) https://github.com/torproject/tor/pull/732

Trac:
Status: assigned to needs_review

lgtm;

Trac:
Reviewer: N/A to dgoulet
Status: needs_review to merge_ready
Keywords: N/A deleted, nickm-merge added

Merged to 0.4.0 and forward; marking for backport.

Trac:
Milestone: Tor: 0.4.1.x-final to Tor: 0.3.5.x-final

The 64-bit Windows Server 2016 builds failed in appveyor due to #29500 (moved).

These builds worked in 0.4.0 and master, so I merged pr/730 to 0.2.9 and later, and pr/731 to 0.3.3 and later.

(I accidentally pushed in-between, so there may be some temporary build failures on 0.3.3, 0.3.4, and 0.3.5.)

Trac:
Status: merge_ready to closed
Resolution: N/A to fixed
Milestone: Tor: 0.3.5.x-final to Tor: 0.2.9.x-final

LGTM!