待辦事項 #45544

enabler_first_clarification() memory leak

啟用日期: 2022-09-01 11:15 最後更新: 2022-09-29 12:13

回報者:
負責人:
類型:
狀態:
關閉
元件:
里程碑:
優先權:
5 - 中
嚴重程度:
5 - 中
處理結果:
修正
檔案:
5

細節

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
  • 負責人 Update from (無) to cazfi
  • 處理結果 Update from to Accepted
2022-09-20 03:39 Updated by: cazfi
  • 處理結果 Update from Accepted to
評語

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:32 Updated by: cazfi
  • 處理結果 Update from 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
  • 狀態 Update from 開啟 to 關閉
  • 處理結果 Update from Accepted to 修正
2022-09-26 05:33 Updated by: cazfi
  • 狀態 Update from 關閉 to 開啟
  • 處理結果 Update from 修正 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: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
  • 狀態 Update from 開啟 to 關閉
  • 處理結果 Update from Accepted to 修正

編輯

Please login to add comment to this ticket » 登入