diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2012-06-11 18:20:08 +0200 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2012-06-11 18:20:08 +0200 |
commit | 3ebbb9c4bbfe5d4a815861afde28df47d35b022e (patch) | |
tree | ca9132c1cc42215fe94457a23e7279f2db71f998 | |
parent | 7f109cca88816752a9c9cbe255df9e3a5b5a8b5a (diff) | |
download | rsyslog-3ebbb9c4bbfe5d4a815861afde28df47d35b022e.tar.gz rsyslog-3ebbb9c4bbfe5d4a815861afde28df47d35b022e.tar.bz2 rsyslog-3ebbb9c4bbfe5d4a815861afde28df47d35b022e.zip |
bugfix: "last message repeated n times" message was missing hostname
Thanks to Zdenek Salvet for finding this bug and to Bodik for reporting
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | action.c | 2 | ||||
-rw-r--r-- | runtime/msg.c | 12 |
3 files changed, 13 insertions, 5 deletions
@@ -1,4 +1,8 @@ --------------------------------------------------------------------------- +Version 5.8.13 [V5-stable] 2012-06-?? +- bugfix: "last message repeated n times" message was missing hostname + Thanks to Zdenek Salvet for finding this bug and to Bodik for reporting +--------------------------------------------------------------------------- Version 5.8.12 [V5-stable] 2012-06-06 - add small delay (50ms) after sending shutdown message There seem to be cases where the shutdown message is otherwise not @@ -961,7 +961,7 @@ tryDoAction(action_t *pAction, batch_t *pBatch, int *pnElem) i = pBatch->iDoneUpTo; /* all messages below that index are processed */ iElemProcessed = 0; iCommittedUpTo = i; -dbgprintf("XXXXX: tryDoAction %p, pnElem %d, nElem %d\n", pAction, *pnElem, pBatch->nElem); + DBGPRINTF("tryDoAction %p, pnElem %d, nElem %d\n", pAction, *pnElem, pBatch->nElem); while(iElemProcessed <= *pnElem && i < pBatch->nElem) { if(*(pBatch->pbShutdownImmediate)) ABORT_FINALIZE(RS_RET_FORCE_TERM); diff --git a/runtime/msg.c b/runtime/msg.c index 820a28a8..f9acb842 100644 --- a/runtime/msg.c +++ b/runtime/msg.c @@ -982,11 +982,15 @@ msg_t* MsgDup(msg_t* pOld) } else { tmpCOPYSZ(RawMsg); } - if(pOld->iLenHOSTNAME < CONF_HOSTNAME_BUFSIZE) { - memcpy(pNew->szHOSTNAME, pOld->szHOSTNAME, pOld->iLenHOSTNAME + 1); - pNew->pszHOSTNAME = pNew->szHOSTNAME; + if(pOld->pszHOSTNAME == NULL) { + pNew->pszHOSTNAME = NULL; } else { - tmpCOPYSZ(HOSTNAME); + if(pOld->iLenHOSTNAME < CONF_HOSTNAME_BUFSIZE) { + memcpy(pNew->szHOSTNAME, pOld->szHOSTNAME, pOld->iLenHOSTNAME + 1); + pNew->pszHOSTNAME = pNew->szHOSTNAME; + } else { + tmpCOPYSZ(HOSTNAME); + } } tmpCOPYCSTR(ProgName); |