summaryrefslogtreecommitdiffstats
path: root/runtime
Commit message (Collapse)AuthorAgeFilesLines
* several bugfixes for the property replacerRainer Gerhards2010-02-081-44/+58
| | | | | | | | | | - bugfix: property replacer returned invalid parameters under some (unusual) conditions. In extreme cases, this could lead to garbled logs and/or a system failure. - bugfix: invalid length returned (often) when using regular expressions inside the property replacer - bugfix: submatch regex in property replacer did not honor "return 0 on no match" config case
* Merge branch 'v4-stable' into v4-betaRainer Gerhards2010-02-042-3/+8
|\
| * bugfix: several smaller bugs resolved after flexelint reviewvarmojfekoj2010-02-043-4/+9
| | | | | | | | Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
| * bugfix: potential segfault on queue shutdownvarmojfekoj2010-01-261-9/+14
| | | | | | | | Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
* | did some adoptions necessary to use the bugfix with v4-beta code baseRainer Gerhards2010-01-261-6/+6
| |
* | Merge branch 'queuepatch' into v4-betanewRainer Gerhards2010-01-261-10/+15
|\ \ | | | | | | | | | | | | Conflicts: runtime/queue.c
| * | bugfixes for potential segfaults during queue shutdownvarmojfekoj2010-01-261-9/+14
| |/ | | | | | | | | | | (bugs require certain non-standard settings to appear) Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
* | bugfix: blanks inside file names did not terminate file name parsing.Rainer Gerhards2010-01-191-1/+2
| | | | | | | | | | | | This could reslult in the whole rest of a line (including comments) to be treated as file name in "write to file" actions. Thanks to Jack for reporting this issue.
* | bugfix: rsyslog hangs when writing to a named pipe which nobody was reading.Rainer Gerhards2010-01-152-4/+12
| | | | | | | | Thanks to Michael Biebl for reporting this bug.
* | bugfix: hostname accidently set to IP address for some message sourcesRainer Gerhards2009-11-301-1/+1
| | | | | | | | for example imudp. Thanks to Anton for reporting this bug.
* | bugfix (kind of): check if TCP connection is still alive if using TLSJonathan Bond-Caron2009-11-251-1/+4
| | | | | | | | Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
* | Merge branch 'v4-stable' into v4-betaRainer Gerhards2009-11-172-2/+5
|\|
| * Merge branch 'v3-stable' into v4-stableRainer Gerhards2009-11-172-2/+5
| |\ | | | | | | | | | | | | | | | Conflicts: doc/rsyslog_conf.html runtime/net.c
| | * correcting -q/-Q patch - was invalidRainer Gerhards2009-11-122-2/+5
| | | | | | | | | | | | | | | This is the correct patch. The previous one solved the segfault, but disabled the -q/Q options.
* | | Merge branch 'v4-stable' into v4-betaRainer Gerhards2009-11-121-2/+2
|\| |
| * | Merge branch 'v3-stable' into v4-stableRainer Gerhards2009-11-121-2/+2
| |\|
| | * bugfix: segfault on startup when -q or -Q option was givenRainer Gerhards2009-11-121-2/+2
| | | | | | | | | | | | | | | bug tracker: http://bugzilla.adiscon.com/show_bug.cgi?id=157 Thanks to Jonas Nogueira for reporting this bug.
* | | added a so-called "On Demand Debug" modeRainer Gerhards2009-11-101-0/+9
| | | | | | | | | | | | | | | | | | in which debug output can be generated only after the process has started, but not right from the beginning. This is assumed to be useful for hard-to-find bugs. Also improved the doc on the debug system.
* | | bugfix: named pipes did no longer work (they always got an open error)Rainer Gerhards2009-11-051-0/+1
| | | | | | | | | | | | this was a regression from the omfile rewrite in 4.5.0
* | | Merge branch 'v4-stable' into v4-betaRainer Gerhards2009-10-211-2/+0
|\| | | | | | | | | | | | | | | | | | | | | | | Conflicts: ChangeLog configure.ac doc/manual.html plugins/imrelp/imrelp.c
| * | Merge branch 'v3-stable' into v4-stableRainer Gerhards2009-10-201-2/+0
| |\|
| | * bugfix: potential hang condition on queue shutdownRainer Gerhards2009-10-201-2/+0
| | |
| * | RFC5424 formatted messages with only structured data and no MSG part were ↵Rainer Gerhards2009-10-012-1/+6
| | | | | | | | | | | | | | | | | | | | | improperly handled. This was a regression of one of the last bugfixes, so no previously released version contained this bug (thus it does not show up in the ChangeLog).
* | | Merge branch 'aaron' into v4-betaRainer Gerhards2009-10-081-1/+1
|\ \ \
| * | | bugfix: debug string larger than 1K were improperly displayed.Rainer Gerhards2009-10-081-1/+1
| | | | | | | | | | | | | | | | Max size is now 32K.
* | | | Merge branch 'aaron' into v4-betaRainer Gerhards2009-10-051-1/+1
|\| | |
| * | | bugfix[minor]: CHKiRet improperly usedRainer Gerhards2009-10-021-1/+1
| | | |
* | | | Merge branch 'aaron' into v4-betaRainer Gerhards2009-10-021-0/+1
|\| | | | | | | | | | | | | | | | | | | Conflicts: runtime/rsyslog.h
| * | | bumped version number, corrected error message code (minor nit)Rainer Gerhards2009-10-011-0/+1
| | | |
* | | | Merge branch 'v4-stable' into v4-beta & BUGFIXRainer Gerhards2009-10-014-41/+96
|\ \ \ \ | |/ / / |/| / / | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: configure.ac doc/manual.html runtime/datetime.h runtime/parser.c runtime/rsyslog.h tools/syslogd.c v4-stable had a bug with RFC5424-formatted structured data, which showed was detected by the enhanced automatted testbench of v4-beta.
| * | bugfix: invalid handling of zero-sized messagesRainer Gerhards2009-09-291-2/+4
| | | | | | | | | | | | could lead to mis-addressing and potential memory corruption/segfault
| * | bugfix: random data could be appended to message, possibly causing segfaultsRainer Gerhards2009-09-244-42/+88
| | |
* | | minor cleanup & preparation for 4.5.4v4.5.4Rainer Gerhards2009-09-291-9/+0
| | |
* | | bugfix: this morning's race patch was incomplete, completing nowRainer Gerhards2009-09-241-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | we needed to release ALL resources (including file handles!) only after the the async writer thread has terminated (else it may access them). In this case, we had a file handle leak, because the handle was sometimes only opened in the async writer, but the close was attempted before the writer even started (in some cases).
* | | (temporary?) removal of very conservative locks in stream.cRainer Gerhards2009-09-241-2/+2
| | | | | | | | | | | | | | | | | | ...after we seem to have identified the root cause of the segfault. I leave them commented out so that we can re-activate it if need arises (aka "get some practice drill first").
* | | bugfix: potential segfault in stream writer on destructionRainer Gerhards2009-09-241-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most severely affected omfile. The problem was that some buffers were freed before the asynchronous writer thread was shut down. So the writer thread accessed invalid data, which may even already be overwritten. Symptoms (with omfile) were segfaults, grabled data and files with random names placed around the file system (most prominently into the root directory). Special thanks to Aaron for helping to track this down.
* | | minor: increased buffer size to be safe in all casesRainer Gerhards2009-09-221-2/+2
| | | | | | | | | | | | | | | if the buffer was too small, we would see more API calls, but no failure, so this is no fix!
* | | bugfixes: potential problems in out file zip writer.Rainer Gerhards2009-09-221-14/+31
| | | | | | | | | | | | | | | | | | Problems could lead to abort and/or memory leak. The module is now hardened in a very conservative way, which is sub-optimal from a performance point of view. This should be improved if it has proven reliable in practice.
* | | bugfix: potential race in object loader during use/release of object interfaceRainer Gerhards2009-09-221-3/+19
| | |
* | | fixed some minor portability issuesRainer Gerhards2009-09-142-3/+11
| | |
* | | bugfix: repeated messages were incorrectly processedRainer Gerhards2009-09-101-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | this could lead to loss of the repeated message content. As a side- effect, it could probably also be possible that some segfault occurs (quite unlikely). The root cause was that some counters introduced during the malloc optimizations were not properly duplicated in MsgDup(). Note that repeated message processing is not enabled by default.
* | | Merge branch 'v4-stable' into v4-betaRainer Gerhards2009-09-032-1/+47
|\| | | | | | | | | | | | | | | | | | | | Conflicts: ChangeLog configure.ac doc/manual.html
| * | bugfix: reverse lookup reduction logic in imudp do DNS queries too oftenRainer Gerhards2009-09-032-1/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | A comparison was done between the current and the former source address. However, this was done on the full sockaddr_storage structure and not on the host address only. This has now been changed for IPv4 and IPv6. The end result of this bug could be a higher UDP message loss rate than necessary (note that UDP message loss can not totally be avoided due to the UDP spec)
| * | bugfix: invalid double-quoted PRI, among others in outgoing messagesRainer Gerhards2009-08-251-1/+1
| | | | | | | | | | | | | | | This causes grief with all receivers. Bug tracker: http://bugzilla.adiscon.com/show_bug.cgi?id=147
| * | Merge branch 'v4.3.2' (v4-beta) into v4-stablev4.4.0Rainer Gerhards2009-08-2135-448/+2346
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: ChangeLog configure.ac doc/manual.html tests/Makefile.am
* | | | bugfix: message sanitation had some issuesRainer Gerhards2009-08-262-4/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - control character DEL was not properly escaped - NUL and LF characters were not properly stripped if no control character replacement was to be done - NUL characters in the message body were silently dropped (this was a regeression introduced by some of the recent optimizations)
* | | | bugfix: strings improperly reused [backported from v5]Rainer Gerhards2009-08-211-1/+1
| | | | | | | | | | | | | | | | | | | | ... resulting in some message properties be populated with strings from previous messages. This was caused by an improper predicate check.
* | | | very minor cleanupRainer Gerhards2009-08-181-1/+0
| | | |
* | | | bugfix: potential segfault in output file writer (omfile)Rainer Gerhards2009-08-182-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | legacy syslog parser changed so that it now accepts date stamps in wrong case.Rainer Gerhards2009-08-171-22/+34
| | | | | | | | | | | | | | | | Some devices seem to create them and I do not see any harm in supporting that.