Ticket #45544

enabler_first_clarification() memory leak

Eröffnet am: 2022-09-01 11:15 Letztes Update: 2022-09-29 12:13

Auswertung:
Verantwortlicher:
Typ:
Status:
Geschlossen
Komponente:
Meilenstein:
Priorität:
5 - Mittel
Schweregrad:
5 - Mittel
Lösung:
Gefixt
Datei:
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-Verlauf (3/14 Historien)

2022-09-01 11:15 Aktualisiert von: cazfi
  • New Ticket "enabler_first_clarification() memory leak" created
2022-09-10 18:47 Aktualisiert von: cazfi
Kommentar

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 Aktualisiert von: cazfi
  • Verantwortlicher Update from (Keine) to cazfi
  • Lösung Update from Keine to Accepted
2022-09-20 03:39 Aktualisiert von: cazfi
  • Lösung Update from Accepted to Keine
Kommentar

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 Aktualisiert von: cazfi
  • Lösung Update from Keine to Accepted
Kommentar

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 Aktualisiert von: cazfi
  • Status Update from Offen to Geschlossen
  • Lösung Update from Accepted to Gefixt
2022-09-26 05:33 Aktualisiert von: cazfi
  • Status Update from Geschlossen to Offen
  • Lösung Update from Gefixt to Accepted
Kommentar

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

2022-09-26 05:36 Aktualisiert von: cazfi
Kommentar

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 Aktualisiert von: cazfi
  • Status Update from Offen to Geschlossen
  • Lösung Update from Accepted to Gefixt

Bearbeiten

Please login to add comment to this ticket » Anmelden