summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* finished adding race condition fix (mostly done by merge)Rainer Gerhards2009-01-301-0/+3
|
* Merge branch 'debian_lenny' into v3-stableRainer Gerhards2009-01-301-12/+7
|\
| * fixed copy&paste errorRainer Gerhards2009-01-281-3/+3
| | | | | | | | | | ... one should at least compile before comitting - sorry, been so eager to push that out.
| * bugfix: proper message locking on message destructRainer Gerhards2009-01-271-12/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | It looks like a race was introduced by not locking the message mutex in msgDestruct(). In theory, I thought, the decrement should be atomic, but the whole operation may be reordered. Also it has potential for task switches. If so, that would lead to a too-early destruction and thus a potential double free - exactly what we have seen from time to time. So I think this fix addresses the issue. I have also removed anything that looks like atomic operations are supported in this version - they are not. This was very late added, found to be non-portable and pulled from that release.
* | bugfix: invalid mutex access in msg.cRainer Gerhards2009-01-304-3/+10
| |
* | prepared for 3.20.3 releasev3.20.3Rainer Gerhards2009-01-193-2/+4
| |
* | Merge branch 'debian_lenny' into v3-stableRainer Gerhards2009-01-122-0/+38
|\| | | | | | | | | Conflicts: ChangeLog
| * fixed a potential segfault condition with $AllowedSender directiveRainer Gerhards2009-01-122-0/+33
| | | | | | | | | | | | On HUP, the root pointers were not properly cleaned up. Thanks to Michael Biebel, olgoat, and Juha Koho for reporting and analyzing the bug.
* | Merge branch 'debian_lenny' into v3-stableRainer Gerhards2008-12-183-29/+38
|\|
| * Merge branch 'v2-stable' into debian_lennyRainer Gerhards2008-12-183-22/+33
| |\
| | * bugfix: "$CreateDirs off" also disabled file creationRainer Gerhards2008-12-182-20/+27
| | | | | | | | | | | | | | | Thanks to William Tisater for analyzing this bug and providing a patch. The actual code change is heavily based on William's patch.
| * | doc bugfix: some spelling errors in man pages correctedGeoff Simmons2008-12-182-9/+11
| | | | | | | | | | | | Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
* | | Merge branch 'debian_lenny' into v3-stableRainer Gerhards2008-12-102-3/+14
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: ChangeLog configure.ac doc/manual.html net.h plugins/imtcp/imtcp.c plugins/imudp/imudp.c runtime/net.c runtime/rsyslog.h tcps_sess.c tcpsrv.c
| * | some legacy options were not correctly processedvarmojfekoj2008-12-102-3/+6
| | | | | | | | | | | | Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
| * | backport of $AllowedSender security fixv3.18.6Rainer Gerhards2008-12-0811-54/+90
| | | | | | | | | | | | | | | | | | | | | - security bugfix: $AllowedSender was not honored, all senders were permitted instead (see http://www.rsyslog.com/Article322.phtml) (backport from v3-stable, v3.20.9) - minor bugfix: dual close() call on tcp session closure
* | | doc bugfix: v3-compatiblity document had typo in config directiveRainer Gerhards2008-12-052-3/+5
| | | | | | | | | | | | thanks to Andrej for reporting this
* | | security fix: imudp emitted a message when a non-permitted sender...v3.20.2Rainer Gerhards2008-12-044-5/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | ...tried to send a message to it. This behaviour is operator-configurable. If enabled, a message was emitted each time. That way an attacker could effectively fill the disk via this facility. The message is now emitted only once in a minute (this currently is a hard-coded limit, if someone comes up with a good reason to make it configurable, we will probably do that).
* | | added proper release datev3.20.1Rainer Gerhards2008-12-041-1/+1
| | |
* | | bugfix: memory leaks in gtls netstream driverRainer Gerhards2008-12-033-1/+17
| | |
* | | bugfix: memory leak if sender was not permittedRainer Gerhards2008-12-031-6/+10
| | |
* | | minor: net.c did not compile if GSSAPI support was disabledRainer Gerhards2008-12-031-0/+2
| | |
* | | added interface function to nsd_gtls needed for ACL controlRainer Gerhards2008-12-013-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | The legacy ACL system needs access to the remote sockaddr_storage data structure. This has been implemented for the ptcp driver and now follows for gtls. See recent commits for reason. We also moved up the version numbers in preparation of the release.
* | | bugfix: $AllowedSender handled invalidly for plain TCP transportRainer Gerhards2008-11-296-4/+63
| | |
* | | minor cleanupRainer Gerhards2008-11-292-1/+2
| | |
* | | security bugfix: $AllowedSender was not honored,Rainer Gerhards2008-11-297-43/+64
| | | | | | | | | | | | ...all senders were permitted instead
* | | doc bugfix: propertiey-based filter modes had inconsistencyRainer Gerhards2008-11-211-3/+1
| | |
* | | Merge branch 'v2-stable' into v3-stableRainer Gerhards2008-11-182-2/+6
|\ \ \ | | |/ | |/| | | | | | | | | | Conflicts: ChangeLog
| * | bugfix: removed memory leak in ompgsqlKen2008-11-182-2/+6
| | | | | | | | | | | | Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
* | | enhance: regex nomatch option "ZERO" has been addedRainer Gerhards2008-11-115-5/+17
| | | | | | | | | | | | | | | | | | This allows to return the string 0 if a regular expression is not found. This is probably useful for storing numerical values into database columns.
* | | added small regex check tool to repositoryRainer Gerhards2008-11-111-0/+72
| | | | | | | | | | | | | | | | | | This is intended for debugging and considered worth preserving. However, it has not (yet) been added to the build diag tools as it is not considered important enough.
* | | improved debug output for regular expressions inside property replacerRainer Gerhards2008-11-112-1/+11
| | | | | | | | | | | | | | | | | | RE's seem to be a big trouble spot and I would like to have more information inside the debug log. So I decided to add some additional debug strings permanently.
* | | doc update: documented how to specify multiple property replacer optionsRainer Gerhards2008-11-103-0/+16
| | | | | | | | | | | | abd link to new online regex generator tool added
* | | minor nit: fixed branch identificationv3.20.0Rainer Gerhards2008-11-051-1/+1
| | |
* | | updated release date for 3.20.0Rainer Gerhards2008-11-051-1/+1
| | |
* | | bumped version number, preparing to be new stable branchRainer Gerhards2008-10-282-2/+2
| | |
* | | Merge branch 'v3-stable' into betaRainer Gerhards2008-10-281-1/+1
|\ \ \ | | |/ | |/| | | | | | | | | | Conflicts: ChangeLog
| * | preparing for 3.18.5 releasev3.18.5Rainer Gerhards2008-10-091-1/+1
| | |
* | | bugfix: double-free in pctp netstream drivervarmojfekoj2008-10-282-2/+16
| | | | | | | | | | | | Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
* | | bugfix: (potentially big) memory leak on HUPRainer Gerhards2008-10-213-20/+40
| | | | | | | | | | | | | | | | | | - if queues could not be drained before timeout - thanks to David Lang for pointing this out - added link to german-language forum to doc set
* | | clarified cost and effect of SIGHUPv3.19.12Rainer Gerhards2008-10-161-0/+8
| | |
* | | increased maximum size of a configuration statement to 4K (was 1K)Rainer Gerhards2008-10-162-4/+6
| | |
* | | Merge branch 'v3-stable' into betaRainer Gerhards2008-10-092-0/+4
|\| |
| * | Merge branch 'v2-stable' into v3-stableRainer Gerhards2008-10-092-0/+4
| |\| | | | | | | | | | | | | | | | Conflicts: syslogd.c
| | * bugfix: memory leaks in rsyslogd, primarily in singlethread modeFederico Nunez2008-10-091-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to Frederico Nunez for providing the fix. The actual patch was commited before this one - unfortunately I forgot to set the author correct when commiting it and then it was pushed to the online repository. Sorry for this ;) Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
| | * fixed memory leaksRainer Gerhards2008-10-092-1/+7
| | | | | | | | | | | | | | | | | | | | | - a big one in syslogd.c, which caused messages not to be freed when compiled for single-threading mode - a small one in the file output handler, outchannels, when a size-reached action was to be executed
* | | Merge branch 'v3-stable' into betaRainer Gerhards2008-10-092-1/+4
|\| |
| * | bugfix[minor]: light delay queue mark not properly handledRainer Gerhards2008-10-092-1/+4
| | | | | | | | | | | | | | | | | | if queue size reached light_delay mark, enqueuing could potentially be blocked for a longer period of time, which was not the behaviour desired.
* | | bugfix: subseconds where not correctly extracted from a timestampRainer Gerhards2008-10-022-1/+5
| | | | | | | | | | | | | | | if that timestamp did not contain any subsecond information (the resulting string was garbagge but should have been "0", what it now is).
* | | Merge branch 'v3-stable' into betaRainer Gerhards2008-10-029-13/+140
|\| | | | | | | | | | | | | | | | | Conflicts: runtime/net.c
| * | bugfix: rsyslogd could hang on HUPRainer Gerhards2008-10-022-3/+28
| | | | | | | | | | | | | | | | | | because getnameinfo() is not cancel-safe, but was not guarded against being cancelled. pthread_cancel() is routinely being called during HUP processing.