Opened 7 days ago

Last modified 17 hours ago

#29740 needs_review defect

Fix memory leaks in shared random unit tests: simple version

Reported by: teor Owned by: teor
Priority: Very High Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Major Keywords: tor-ci, tor-test, memory-management, 029-backport, 034-backport, 035-backport, 040-backport, nickm-merge, dgoulet-merge
Cc: Actual Points: 0.3
Parent ID: Points: 0.1
Reviewer: asn Sponsor:


Please merge to the mainline branches.
I'll merge to to 0.2.9 and later after the mainline CI passes.

Child Tickets

Change History (7)

comment:1 Changed 7 days ago by teor

Reviewer: asn
Status: assignedmerge_ready

asn reviewed in #29706.

comment:2 Changed 7 days ago by teor

Keywords: nickm-merge dgoulet-merge added

comment:3 Changed 7 days ago by teor

Priority: HighVery High

comment:4 Changed 6 days ago by nickm

Resolution: fixed
Status: merge_readyclosed

#29706 is closed; closing this

comment:5 Changed 6 days ago by teor

Milestone: Tor: 0.4.1.x-finalTor: 0.3.5.x-final
Resolution: fixed
Status: closedreopened

#29706 is not closed. And this change should be backported to 0.2.9 and later.
Putting in 0.3.5, because #29706 was merged to 0.4.0 and later.

comment:6 Changed 18 hours ago by teor

Actual Points: 0.10.3
Milestone: Tor: 0.3.5.x-finalTor: 0.4.0.x-final
Parent ID: #29706
Status: reopenedneeds_review

It turns out we need to apply a9c3101 from #29706 to avoid all leaks:

The pull requests are:

0.2.9, one commit added:

0.3.4, merged, and one comment commit added:

0.4.0, comments merged, the code changes are already in 0.4.0:

(#29706 is too big to backport, so I'm un-parenting this ticket.)

comment:7 Changed 17 hours ago by teor

One appveyor test failed due to #29645. I'm going to relaunch the test to make sure there aren't any more failures.

Note: See TracTickets for help on using tickets.