diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2010-12-16 12:57:55 +0100 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2010-12-16 12:57:55 +0100 |
commit | 371a8eec29fa25bbf58f4b1f0d7e3bf4c3ad6329 (patch) | |
tree | 661f442e4e5c61f56e72241830703267ae08e38b /runtime/msg.c | |
parent | 75bfef7fa1c9f71232a282aa66e52b2d1860100a (diff) | |
download | rsyslog-371a8eec29fa25bbf58f4b1f0d7e3bf4c3ad6329.tar.gz rsyslog-371a8eec29fa25bbf58f4b1f0d7e3bf4c3ad6329.tar.xz rsyslog-371a8eec29fa25bbf58f4b1f0d7e3bf4c3ad6329.zip |
some cleanup based on clang static analyzer results
Diffstat (limited to 'runtime/msg.c')
-rw-r--r-- | runtime/msg.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/runtime/msg.c b/runtime/msg.c index f5041b85..a9a09143 100644 --- a/runtime/msg.c +++ b/runtime/msg.c @@ -1476,7 +1476,7 @@ rsRetVal MsgSetPROCID(msg_t *pMsg, char* pszPROCID) CHKiRet(cstrConstruct(&pMsg->pCSPROCID)); } /* if we reach this point, we have the object */ - iRet = rsCStrSetSzStr(pMsg->pCSPROCID, (uchar*) pszPROCID); + CHKiRet(rsCStrSetSzStr(pMsg->pCSPROCID, (uchar*) pszPROCID)); CHKiRet(cstrFinalize(pMsg->pCSPROCID)); finalize_it: @@ -2872,7 +2872,6 @@ uchar *MsgGetProp(msg_t *pMsg, struct templateEntry *pTpe, if(pTpe->data.field.options.bCSV) { /* we need to obtain a private copy, as we need to at least add the double quotes */ int iBufLen; - int i; uchar *pBStart; uchar *pDst; uchar *pSrc; @@ -2887,7 +2886,6 @@ uchar *MsgGetProp(msg_t *pMsg, struct templateEntry *pTpe, RET_OUT_OF_MEMORY; } pSrc = pRes; - i = 0; *pDst++ = '"'; /* starting quote */ while(*pSrc) { if(*pSrc == '"') |