doc/man/syslog-ng.8.xml | 18 2 + 16 - 0 !
1 file changed, 2 insertions(+), 16 deletions(-)

 remove outdated documentation segment
 syslog-ng loads modules on-demand the argument is gone.

modules/python/pylib/Makefile.am | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 ignore pep8 w504 warning

lib/cfg-block-generator.c | 19 15 + 4 - 0 !
lib/cfg-block-generator.h | 4 3 + 1 - 0 !
lib/cfg-lexer.c | 2 1 + 1 - 0 !
3 files changed, 19 insertions(+), 6 deletions(-)

 [patch] cfg-block: make cfgblockgenerator instances refcounted

Sometimes CfgBlock instances are constructed every time they are
referenced (e.g. app-parser() in its construct method), in other cases
the same generator instance is returned (e.g. those created by
block {} statements).

The shared ones were properly freed, but the dynamic kind were not.

This patch adds reference counting, the followup patch will fix the leak.

Signed-off-by: Balazs Scheidler <balazs.scheidler@oneidentity.com>

lib/cfg-lexer.c | 6 4 + 2 - 0 !
1 file changed, 4 insertions(+), 2 deletions(-)

 [patch] cfg-lexer: fix memory leak for dynamically allocated
 CfgBlockGenerators

This should fix a memory leak for app-parser() if that is found in the
configuration.

Signed-off-by: Balazs Scheidler <balazs.scheidler@oneidentity.com>

tests/pytest_framework/src/logger/logger.py | 5 4 + 1 - 0 !
1 file changed, 4 insertions(+), 1 deletion(-)

 [patch] logger: fix leaking file handlers

Signed-off-by: Andras Mitzki <andras.mitzki@balabit.com>

modules/afsocket/afinet-dest.c | 4 4 + 0 - 0 !
1 file changed, 4 insertions(+)

 [patch] afsocket: fix to prevent accessing freed up memory.

dest_addr and bind_addr pointers hold an address that was freed up. This results in access violation. This fix adds NULL to those pointers.

Signed-off-by: Naveen Revanna <nrevanna@purestorage.com>

CMakeLists.txt | 8 8 + 0 - 0 !
syslog-ng-config.h.in | 1 1 + 0 - 0 !
2 files changed, 9 insertions(+)

 [patch] cmake: add missing detection for o_largefile

Signed-off-by: Antal Nemes <antal.nemes@balabit.com>

lib/logthrdestdrv.c | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 [patch] threaded-dest: fix integer overflow

A negative number's modulo will be negative, which in our
case pointed to a negative entry of the workers array,
causing segfault.

Signed-off-by: Attila Szakacs <attila.szakacs@balabit.com>

lib/logthrdestdrv.c | 7 3 + 4 - 0 !
lib/logthrdestdrv.h | 1 1 + 0 - 0 !
2 files changed, 4 insertions(+), 4 deletions(-)

 [patch] threaded-dest: move last_worker to destdriver

Before this, `last_worker` was a static variable, which meant,
if we had two http destinations, one message sent to the
first would step the current worker in the second causing
uneven load between the workers.

Signed-off-by: Attila Szakacs <attila.szakacs@balabit.com>

syslog-ng-config.h.in | 3 1 + 2 - 0 !
1 file changed, 1 insertion(+), 2 deletions(-)

 [patch] cmake: fix typo in have_strnlen

Signed-off-by: Antal Nemes <antal.nemes@balabit.com>

modules/http/http.c | 1 1 + 0 - 0 !
1 file changed, 1 insertion(+)

 [patch] http: add missing free for self->body_template

Signed-off-by: Antal Nemes <antal.nemes@balabit.com>

lib/tests/test_pathutils.c | 12 9 + 3 - 0 !
1 file changed, 9 insertions(+), 3 deletions(-)

 [patch] test_pathutils: fix leak

Signed-off-by: Kokan <kokaipeter@gmail.com>

modules/affile/tests/test_file_list.c | 1 1 + 0 - 0 !
1 file changed, 1 insertion(+)

 [patch] test_file_list: fix leak

Signed-off-by: Kokan <kokaipeter@gmail.com>

lib/template/simple-function.c | 5 4 + 1 - 0 !
1 file changed, 4 insertions(+), 1 deletion(-)

 [patch] template: tf_simple_func_prepare leak fix

The state object had the array and the argc length, when the proper
cleanup function is called it checks the argc to free templates.

Signed-off-by: Kokan <kokaipeter@gmail.com>

modules/dbparser/groupingby.c | 12 12 + 0 - 0 !
1 file changed, 12 insertions(+)

 [patch] gorupingby: fix memory leak

Due to missing unref, the filter expressions were leaked.
Also, init functions were not called either.

Signed-off-by: Antal Nemes <antal.nemes@balabit.com>

modules/dbparser/groupingby.c | 61 50 + 11 - 0 !
1 file changed, 50 insertions(+), 11 deletions(-)

 [patch] groupingby: fix invalid memory access

There was an invalid memory access in groupingby. The TimerWheel
object stores all timers, however the individual timers are also
stored inside the contexts.

The original code stores contexts in persist state, however the
timer_wheel is freed during reload. When the new config starts, and a
context is fetched, groupingby will access the already freed timer.

This patch stores timerwheel in persist state too.

Signed-off-by: Antal Nemes <antal.nemes@balabit.com>