syslog-ng--mainline: Forward port: various memory leak fixes
Link: <http://intra.balabit/cgi-bin/viewarch.cgi/devel@balabit.hu--other-1/syslog-ng--mainline--2.1--patch-7> Revision: syslog-ng--mainline--2.1--patch-7 Archive: devel@balabit.hu--other-1 Creator: Balazs Scheidler <bazsi@balabit.hu> Date: Wed Oct 25 10:13:16 CEST 2006 Standard-date: 2006-10-25 08:13:16 GMT Modified-files: ChangeLog src/affile.c src/afprog.c src/afsocket.c src/children.c src/children.h src/logreader.c src/logwriter.c src/logwriter.h src/messages.c New-patches: devel@balabit.hu--other-1/syslog-ng--mainline--2.0--patch-73 devel@balabit.hu--other-1/syslog-ng--mainline--2.1--patch-7 Summary: Forward port: various memory leak fixes Keywords: * src/affile.c (affile_dw_deinit): check if self->writer is NULL, (affile_dd_free): call log_writer_options_destroy as LogWriterOptions might contain dynamically allocated memory, * src/afprog.c (afprogram_dd_exit): removed log_pipe_unref, it is performed by the children handling framework, (afprogram_dd_init): use the new GDestroyNotify argument of children manager, (afprogram_dd_free): added log_writer_options_destroy call * src/afsocket.c (afsocket_dd_free): added log_writer_options_destroy call * src/children.c (ChildEntry): added callback_data_destroy member, (child_manager_child_entry_free): new function, frees a ChildEntry, (child_manager_sigchild): removed g_free, it is done by the GHashTable automatically, (child_manager_init): use g_hash_table_new_full and specify destroy notify callback for the data stored in the hashtable * src/logreader.c (log_reader_iterate_buf): assume that saddr is a borrowed reference and handle it as such, (log_reader_fetch_log): iterate_buf borrows the reference so it is our duty to free sockaddr, do so in all branches, * src/logwriter.c (log_writer_deinit): drop reference to self->control to break a circular reference, (log_writer_free): removed log_pipe_unref self->control, * src/messages.c (msg_send_internal_message): check if internal_msg_queue is null, (msg_deinit): free internal_msg_queue Diff stats: ChangeLog | 49 +++++++++++++++++++++++++++++++++++++++++++++++++ src/affile.c | 10 +++++++--- src/afprog.c | 6 ++---- src/afsocket.c | 1 + src/children.c | 14 +++++++++++--- src/children.h | 2 +- src/logreader.c | 10 +++++----- src/logwriter.c | 5 +++-- src/logwriter.h | 1 + src/messages.c | 5 ++++- 10 files changed, 84 insertions(+), 19 deletions(-)
participants (1)
-
bazsi@balabit.hu