diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2008-04-03 13:57:18 +0000 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2008-04-03 13:57:18 +0000 |
commit | 61931c57c39de8d2dbeb0a6a6d27e2762e1821d4 (patch) | |
tree | 689e6fd93c1eeb801d70b32baecf71ac8feb307a /obj.c | |
parent | 6b982c8e5dbfd54eff7e22291d7f63400ff3c555 (diff) | |
download | rsyslog-61931c57c39de8d2dbeb0a6a6d27e2762e1821d4.tar.gz rsyslog-61931c57c39de8d2dbeb0a6a6d27e2762e1821d4.tar.xz rsyslog-61931c57c39de8d2dbeb0a6a6d27e2762e1821d4.zip |
bugfix: some memory leak when queue is runing in disk mode
Diffstat (limited to 'obj.c')
-rw-r--r-- | obj.c | 12 |
1 files changed, 10 insertions, 2 deletions
@@ -731,7 +731,7 @@ finalize_it: static rsRetVal objDeserializeProperties(obj_t *pObj, objInfo_t *pObjInfo, strm_t *pStrm) { DEFiRet; - var_t *pVar; + var_t *pVar = NULL; ISOBJ_assert(pObj); ISOBJ_TYPE_assert(pStrm, strm); @@ -743,15 +743,23 @@ static rsRetVal objDeserializeProperties(obj_t *pObj, objInfo_t *pObjInfo, strm_ iRet = objDeserializeProperty(pVar, pStrm); while(iRet == RS_RET_OK) { CHKiRet(pObjInfo->objMethods[objMethod_SETPROPERTY](pObj, pVar)); + /* re-init var object - TODO: method of var! */ + rsCStrDestruct(&pVar->pcsName); /* no longer needed */ + if(pVar->varType == VARTYPE_STR) { + if(pVar->val.pStr != NULL) + rsCStrDestruct(&pVar->val.pStr); + } iRet = objDeserializeProperty(pVar, pStrm); } - var.Destruct(&pVar); if(iRet != RS_RET_NO_PROPLINE) FINALIZE; CHKiRet(objDeserializeTrailer(pStrm)); /* do trailer checks */ finalize_it: + if(pVar != NULL) + var.Destruct(&pVar); + RETiRet; } |