search

mbenkmann / limux-gosa

GOsa² is a web based adminstration tool for user accounts, groups, servers, clients, and many other things.
18 stars 5 forks source link

Memory leak on CLMSG_save_fai_log #104

Closed mbenkmann closed 9 years ago

mbenkmann commented 9 years ago

From matthias...@gmail.com on March 25, 2013 07:45:10

CLMSG_save_fai_log must be rewritten to never create a large GC'ed memory block. To do this:

1) Basis is the rewriting of xml.Hash to use malloc-based memory management (see issue #101 )

2) Write a streaming decoder as io.Writer() for CLMSG_save_fai_log and use xml.Hash.WriteTo to send the data into it.

See related issue #75

Original issue: http://code.google.com/p/go-susi/issues/detail?id=104

mbenkmann commented 9 years ago

From matthias...@gmail.com on March 24, 2013 23:58:26

=> Type Optimization

Labels: -Type-Feature Type-Optimization

mbenkmann commented 9 years ago

From matthias...@gmail.com on April 03, 2013 11:02:51

Fixed in 65cb1db691fc. go-susi now uses malloc-ated memory for the complete CLMSG_save_fai_log code path, from handleConnection() over GosaDecrypt() to clmsg_save_fai_log().

Status: Done