Opened 6 months ago

Closed 3 months ago

#30894 closed defect (fixed)

Memory leak on invalid CSV_INTERVAL value

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: consider-backport-after-0415, 035-backport 040-backport 041-backport
Cc: Actual Points: 0.1
Parent ID: Points: 0.1
Reviewer: dgoulet Sponsor: Sponsor31-can

Description

There is a missing free in the case where we fail to parse the interval from a CSV_INTERVAL field.

Child Tickets

Change History (10)

comment:1 Changed 6 months ago by nickm

Parent ID: #30893

Found this while working on #30893.

comment:2 Changed 6 months ago by nickm

Status: assignedneeds_review

See bug30894_035 with PR at https://github.com/torproject/tor/pull/1114 .

This branch has been merged into the branch for #30893, so that the tests for #30893 will pass.

comment:3 Changed 6 months ago by dgoulet

Keywords: asn-merge added
Status: needs_reviewmerge_ready

ack.

comment:4 Changed 6 months ago by dgoulet

Reviewer: dgoulet

comment:5 Changed 6 months ago by dgoulet

Milestone: Tor: 0.4.2.x-finalTor: 0.4.1.x-final
Parent ID: #30893

Merged in parent #30893. Moving from milestone, unparenting for backport.

comment:6 Changed 6 months ago by asn

Keywords: asn-merge removed

Merged to 041 and forward. Leaving open for deeper backports.

comment:7 Changed 6 months ago by nickm

Milestone: Tor: 0.4.1.x-finalTor: 0.4.0.x-final

comment:8 Changed 4 months ago by teor

Keywords: consider-backport-after-0415 added

This PR was merged into 0.4.1.4, so we will test it until 0.4.1.5 is released, then merge.

comment:9 Changed 3 months ago by teor

Merged to 0.3.5 and later.
Merged #30894, #30871, and #31003 together.

comment:10 Changed 3 months ago by teor

Resolution: fixed
Status: merge_readyclosed
Note: See TracTickets for help on using tickets.