enabler_first_clarification() memory leak

• Reporter: cazfi
• 소유자: cazfi
• Type: Bugs
• Status: Closed
• Component: General
• MileStone: 3.0.4 (closed)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: 5

Details

Valgrind reports minor leak in S3_0 that comes from the req_to_fstring() call within enabler_first_clarification(). Other functions do req_to_fstring() calls with similar pattern.

Ticket History (3/14 Histories)

2022-09-01 11:15 Updated by: cazfi

• New Ticket "enabler_first_clarification() memory leak" created

2022-09-10 18:47 Updated by: cazfi

댓글 올리기

Reply To cazfi

Other functions do req_to_fstring() calls with similar pattern.

It's not the pattern that is used in req_to_fstring() calling. It's the fact that req_to_fstring() cannot be used without leaking memory as the API is currently designed.

2022-09-11 16:12 Updated by: cazfi

• File 0058-Fix-memory-leaks-from-req_to_fstring-usage.patch (File ID: 10309) is attached

2022-09-11 16:12 Updated by: cazfi

• File 0050-Fix-memory-leaks-from-req_to_fstring-usage.patch (File ID: 10310) is attached

2022-09-11 16:12 Updated by: cazfi

• File 0027-Fix-memory-leaks-from-req_to_fstring-usage.patch (File ID: 10311) is attached

2022-09-11 16:12 Updated by: cazfi

• 소유자 Update from (None) to cazfi
• Resolution Update from None to Accepted

2022-09-20 03:39 Updated by: cazfi

• Resolution Update from Accepted to None

댓글 올리기

Changes to req_to_fstring() itself have been badly backported to S3_0 patch (lacking the astr initialization, function header updates).

2022-09-20 06:31 Updated by: cazfi

• File 0014-Fix-memory-leaks-from-req_to_fstring-usage.patch (File ID: 10341) is attached

2022-09-20 06:32 Updated by: cazfi

• Resolution Update from None to Accepted

댓글 올리기

Reply To cazfi

Changes to req_to_fstring() itself have been badly backported to S3_0 patch (lacking the astr initialization, function header updates).

- Fixed

2022-09-26 05:17 Updated by: cazfi

• Status Update from Open to Closed
• Resolution Update from Accepted to Fixed

2022-09-26 05:33 Updated by: cazfi

• Status Update from Closed to Open
• Resolution Update from Fixed to Accepted

댓글 올리기

As a server side memory leak, I want to fix these in S2_6 too. Reopening for that part...

2022-09-26 05:33 Updated by: cazfi

• File 0016-Fix-memory-leaks-from-req_to_fstring-usage.patch (File ID: 10399) is attached

2022-09-26 05:36 Updated by: cazfi

댓글 올리기

Reply To cazfi

As a server side memory leak, I want to fix these in S2_6 too. Reopening for that part...

Attached S2_6 patch has also #45720 part.

2022-09-29 12:13 Updated by: cazfi

• Status Update from Open to Closed
• Resolution Update from Accepted to Fixed