summaryrefslogtreecommitdiffstats
path: root/runtime
Commit message (Collapse)AuthorAgeFilesLines
...
| * | bugfix: potential race in object loader during use/release of object interfaceRainer Gerhards2009-09-221-3/+19
| | |
* | | Merge branch 'v4-beta' into betaRainer Gerhards2009-09-142-3/+11
|\| |
| * | fixed some minor portability issuesRainer Gerhards2009-09-142-3/+11
| | |
* | | Merge branch 'v4-beta' into betaRainer Gerhards2009-09-101-3/+8
|\| | | | | | | | | | | | | | Conflicts: action.c
| * | 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-devel' into betaRainer Gerhards2009-09-032-1/+47
|\| |
| * | 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
* | | | Merge branch 'beta'Rainer Gerhards2009-08-272-4/+29
|\| | |
| * | | 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.
* | | | reduced number of debug messages a bit againRainer Gerhards2009-08-263-6/+0
| | | |
* | | | bugfix: strings improperly reusedRainer 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.
* | | | Merge branch 'beta'Rainer Gerhards2009-08-192-10/+4
|\| | |
| * | | 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.
* | | | fixed compile problem introduced by mergeRainer Gerhards2009-08-181-0/+1
| | | |
* | | | Merge branch 'beta'Rainer Gerhards2009-08-181-2/+8
|\ \ \ \
| * | | | bugfix? (unconfirmed, testing): segfault when writing asynchronouslyRainer Gerhards2009-08-181-2/+8
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I have undone a very small optimization with using pre-malloced memory, which seems to have some issues. Now I am doing mallocs and at least in test environment this seems to solve the issue. The code now needs more review. If it runs flawlessly for some time, I may try to re-enable to pre-malloc, but not necessarily: its performance benefit is very mild (aka: I don't think it justifies introducing bigger complexities).
* | | | Merge branch 'beta'Rainer Gerhards2009-08-171-22/+34
|\| | |
| * | | 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.
* | | | bugfix: discard action did not work (did not discard messages)Rainer Gerhards2009-07-303-3/+12
| | | |
* | | | bugfix: discard action caused segfaultRainer Gerhards2009-07-301-3/+5
| | | |
* | | | internal: added ability to terminate input modules not via pthread_cancel...Rainer Gerhards2009-07-205-5/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... but an alternate approach via pthread_kill. This is somewhat safer as we do not need to think about the cancel-safeness of all libraries we use. However, not all inputs can easily supported, so this now is a feature that can be requested by the input module (the most important ones request it).
* | | | cleanup & better debug message handlingRainer Gerhards2009-07-201-66/+65
| | | | | | | | | | | | | | | | | | | | | | | | the new handling will hopefully spare a few cycles, as function calls (and most importantly parameter generation!) or now only done when debug messages are actually active.
* | | | corrected some conditions & one more simplificationRainer Gerhards2009-07-201-15/+3
| | | |
* | | | simplified startup of queue DA modeRainer Gerhards2009-07-203-67/+9
| | | |
* | | | bugfix: potential race condition in debug modeRainer Gerhards2009-07-201-1/+2
| | | |
* | | | solved potential race condition and some cleanupRainer Gerhards2009-07-202-6/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | code review brought up some few places where we may have run into a race. They have most probably been introduced during the recent set of changes. But I do not look at older versions because of the changed architecture, one can not simply backport this patch.
* | | | bugfix: minor static memory leak while reading configurationRainer Gerhards2009-07-204-19/+15
| | | | | | | | | | | | | | | | | | | | This did NOT leak based on message volume. Also, did some cleanup during the commit.
* | | | enhanced worker thread pool by atomic opsRainer Gerhards2009-07-203-32/+23
| | | | | | | | | | | | | | | | | | | | ... greater performance and was able to remove a potential troublespot in a cancel cleanup handler.
* | | | architecture change: queue now always has at least one worker threadRainer Gerhards2009-07-204-19/+20
| | | | | | | | | | | | | | | | | | | | | | | | ...if not running in direct mode. Previous versions could run without any active workers. This simplifies the code at a very small expense. See v5 compatibility note document for more in-depth discussion.
* | | | more cleanup/simplification (forgot to remove one mutex lock)Rainer Gerhards2009-07-203-36/+10
| | | |
* | | | some more threading changesRainer Gerhards2009-07-203-44/+27
| | | | | | | | | | | | | | | | ... as well as some cleanup
* | | | cleanup: removed no longer needed codeRainer Gerhards2009-07-202-20/+0
| | | |
* | | | removed mutex locksRainer Gerhards2009-07-174-34/+36
| | | | | | | | | | | | | | | | | | | | ... by utilizing that we need to modify a state variable only in a sequential way during shutdown.
* | | | further code simplificationRainer Gerhards2009-07-176-176/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... could even remove one mutex by using a better algorithm. I think I also spotted some situation in which a hang could have happened. As I can't fix it in v4 and less without moving to the new engine, I make no effort in testing this out. Hangs occur during shutdown, only (if at all). The code changes should also result in some mild performance improvement. Some bug potential, but overall the bug potential should have been greatly reduced.
* | | | more code simplification, should also bring some performance enhancementRainer Gerhards2009-07-177-80/+45
| | | | | | | | | | | | | | | | reducing the number of thread cancellation state changes
* | | | Merge branch 'beta'Rainer Gerhards2009-07-162-3/+10
|\| | | | | | | | | | | | | | | | | | | | | | | Conflicts: runtime/wti.c runtime/wtp.c
| * | | calls to prctl() need to be based on configure results (cross-platform issue)Rainer Gerhards2009-07-162-3/+5
| | | | | | | | | | | | | | | | | | | | This is for another prctl() call, not present in the beta version (looks like it would make sense to stick these into a utility function)
| * | | Merge branch 'v4-beta' into v4-develRainer Gerhards2009-07-161-2/+5
| |\ \ \
| | * | | calls to prctl() need to be based on configure results (cross-platform issue)Rainer Gerhards2009-07-161-2/+5
| | | | |
* | | | | some code simplificationRainer Gerhards2009-07-164-14/+11
| | | | |
* | | | | Merge branch 'detached' into masterRainer Gerhards2009-07-154-209/+16
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: tests/nettester.c
| * | | | | further reduction in code size/complexity due to design changeRainer Gerhards2009-07-084-40/+3
| | | | | |
| * | | | | simplified worker thread handlingRainer Gerhards2009-07-084-171/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | based on now working with detached threads. This is probably the biggest patch in this series and with large bug potential.
* | | | | | some more (very mild) cleanupRainer Gerhards2009-07-151-1/+0
| | | | | |
* | | | | | some mild restart-type HUP cleanupRainer Gerhards2009-07-152-19/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... only those things that were obvious (and puzzled people looking at the code without konwing the subtle issues of HUP ;)).