Package: syslog-ng / 3.19.1-5

Metadata

Package Version Patches format
syslog-ng 3.19.1-5 3.0 (quilt)

Patch series

view the series file
Patch File delta Description
0001 Remove outdated documentation.patch | (download)

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.
0007 Ignore PEP8 W504 warning.patch | (download)

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

 ignore pep8 w504 warning


0010 Fix_app parser_per_reload_memory_leak_part1.patch | (download)

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>

0011 Fix_app parser_per_reload_memory_leak_part2.patch | (download)

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>

0012 Fix_leaking_file_handlers.patch | (download)

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>

0013 DNS_memory_leak_segfault_fix.patch | (download)

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>

0014 cmake_add_missing_detection_for_O_LARGEFILE.patch | (download)

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>

0015 threaded dest_Fix_integer_overflow.patch | (download)

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>

0016 threaded dest_move_last_worker_to_DestDriver.patch | (download)

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>

0017 cmake_fix_typo_in_HAVE_STRNLEN.patch | (download)

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>

0018 http_add_missing_free_for_self body_template.patch | (download)

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>

0019 test_pathutils_fix_leak.patch | (download)

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>

0020 test_file_list_fix_leak.patch | (download)

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>

0021 template_tf_simple_func_prepare_leak_fix.patch | (download)

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>

0022 gorupingby_fix_memory_leak.patch | (download)

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>

0023 groupingby_fix_invalid_memory_access.patch | (download)

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>