diff options
| author | Andre Lorbach <alorbach@adiscon.com> | 2011-08-10 11:27:03 +0200 |
|---|---|---|
| committer | Andre Lorbach <alorbach@adiscon.com> | 2011-08-10 11:27:03 +0200 |
| commit | 116fb13d42695835487e7984e41090bd02d370c1 (patch) | |
| tree | 4dbf4fe3681a6de9a20904477266344281d38fb4 | |
| parent | 26220bb426802aa5c07a1904575ddd0d6a3de177 (diff) | |
| parent | 1d856763f9a3f078af7f066867ae88c59286285c (diff) | |
| download | rsyslog-116fb13d42695835487e7984e41090bd02d370c1.tar.gz rsyslog-116fb13d42695835487e7984e41090bd02d370c1.tar.bz2 rsyslog-116fb13d42695835487e7984e41090bd02d370c1.zip | |
Merge branch 'v5-stable'
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | runtime/msg.c | 13 |
2 files changed, 13 insertions, 3 deletions
@@ -350,6 +350,9 @@ Version 5.8.4 [V5-stable] (al), 2011-??-?? - bugfix: memcpy overflow can occur in allowed sender checkig if a name is resolved to IPv4-mapped-on-IPv6 address Found by Ismail Dönmez at suse +- bugfix: potential misadressing in property replacer +- bugfix: MSGID corruption in RFC5424 parser under some circumstances + closes: http://bugzilla.adiscon.com/show_bug.cgi?id=275 --------------------------------------------------------------------------- Version 5.8.3 [V5-stable] (rgerhards), 2011-07-11 - systemd support: set stdout/stderr to null - thx to Lennart for the patch diff --git a/runtime/msg.c b/runtime/msg.c index fdc2174f..40fbf839 100644 --- a/runtime/msg.c +++ b/runtime/msg.c @@ -1621,14 +1621,21 @@ finalize_it: } -/* rgerhards, 2005-11-24 +/* al, 2011-07-26: LockMsg to avoid race conditions */ static inline char *getMSGID(msg_t *pM) { - return (pM->pCSMSGID == NULL) ? "-" : (char*) rsCStrGetSzStrNoNULL(pM->pCSMSGID); + if (pM->pCSMSGID == NULL) { + return "-"; + } + else { + MsgLock(pM); + char* pszreturn = (char*) rsCStrGetSzStrNoNULL(pM->pCSMSGID); + MsgUnlock(pM); + return pszreturn; + } } - /* rgerhards 2009-06-12: set associated ruleset */ void MsgSetRuleset(msg_t *pMsg, ruleset_t *pRuleset) |