diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2013-07-17 13:33:13 +0200 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2013-07-17 13:33:13 +0200 |
commit | cd2b917feff1f292085cef15574bb5974a11d4a7 (patch) | |
tree | ce01c0df6b2e7dcf890e29beffccd12b87f65094 | |
parent | 44699a0d15f57e3d9e1408dc802a78d286f7813d (diff) | |
download | rsyslog-cd2b917feff1f292085cef15574bb5974a11d4a7.tar.gz rsyslog-cd2b917feff1f292085cef15574bb5974a11d4a7.tar.bz2 rsyslog-cd2b917feff1f292085cef15574bb5974a11d4a7.zip |
cosmetic bugfix: file name buffer was not freed on disk queue destruction
This was an extremely small one-time per run memleak, so nothing of
concern. However, it bugs under valgrind and similar memory debuggers.
-rw-r--r-- | ChangeLog | 3 | ||||
-rw-r--r-- | runtime/queue.c | 3 |
2 files changed, 5 insertions, 1 deletions
@@ -9,6 +9,9 @@ Version 7.4.3 [v7.4-stable] 2013-07-?? - bugfix: omlibdbi did not properly close connection on some errors This happened to errors occuring in Begin/End Transaction entry points. +- cosmetic bugfix: file name buffer was not freed on disk queue destruction + This was an extremely small one-time per run memleak, so nothing of + concern. However, it bugs under valgrind and similar memory debuggers. - fix build on FreeBSD Thanks to Christiano Rolim for the patch --------------------------------------------------------------------------- diff --git a/runtime/queue.c b/runtime/queue.c index 5ccdd814..08ef475f 100644 --- a/runtime/queue.c +++ b/runtime/queue.c @@ -877,7 +877,8 @@ static rsRetVal qDestructDisk(qqueue_t *pThis) DEFiRet; ASSERT(pThis != NULL); - + + free(pThis->pszQIFNam); if(pThis->tVars.disk.pWrite != NULL) strm.Destruct(&pThis->tVars.disk.pWrite); if(pThis->tVars.disk.pReadDeq != NULL) |