From 7eb72ef77c8acbcc3d80644b9d3c5f15b44a9930 Mon Sep 17 00:00:00 2001 From: Rainer Gerhards Date: Wed, 19 Aug 2009 13:05:42 +0200 Subject: added important bug fix doc to current release the bug fix was imported from 4.5.1, but it is important enough to be highlighted in its own right. --- ChangeLog | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/ChangeLog b/ChangeLog index fdbf45cf..bb22bb91 100644 --- a/ChangeLog +++ b/ChangeLog @@ -2,6 +2,16 @@ Version 5.1.4 [DEVEL] (rgerhards), 2009-07-?? - bugfix: discard action did not work (did not discard messages) - bugfix: discard action caused segfault +- bugfix: potential segfault in output file writer (omfile) + In async write mode, we use modular arithmetic to index the output + buffer array. However, the counter variables accidently were signed, + thus resulting in negative indizes after integer overflow. That in turn + could lead to segfaults, but was depending on the memory layout of + the instance in question (which in turn depended on a number of + variables, like compile settings but also configuration). The counters + are now unsigned (as they always should have been) and so the dangling + mis-indexing does no longer happen. This bug potentially affected all + installations, even if only some may actually have seen a segfault. --------------------------------------------------------------------------- Version 5.1.3 [DEVEL] (rgerhards), 2009-07-28 - architecture change: queue now always has at least one worker thread -- cgit