bugfix: double-free in omelasticsearch
authorRainer Gerhards <rgerhards@adiscon.com>
Tue, 2 Jul 2013 09:48:52 +0000 (11:48 +0200)
committerRainer Gerhards <rgerhards@adiscon.com>
Tue, 2 Jul 2013 09:48:52 +0000 (11:48 +0200)
closes: http://bugzilla.adiscon.com/show_bug.cgi?id=461
Thanks to Marius Ionescu for providing a detailled bug report

ChangeLog
plugins/omelasticsearch/omelasticsearch.c

index cb93bda..729c360 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -4,6 +4,9 @@ Version 7.4.2  [v7.4-stable] 2013-06-??
 - bugfix: RainerScript object required parameters were not properly
   checked - this clould result to segfaults on startup if parameters
   were missing.
+- bugfix: double-free in omelasticsearch
+  closes: http://bugzilla.adiscon.com/show_bug.cgi?id=461
+  Thanks to Marius Ionescu for providing a detailled bug report
 - bugfix: omrelp potential segfault at startup on invalid config parameters
 - bugfix: small memory leak when $uptime property was used
 - bugfix: potential segfault on rsyslog termination in imudp
index 33e58c1..57abefd 100644 (file)
@@ -483,7 +483,6 @@ writeDataError(instanceData *pData, cJSON **pReplyRoot, uchar *reqmsg)
                DBGPRINTF("omelasticsearch: error %d writing error file, write returns %lld\n",
                          errno, (long long) wrRet);
        }
-       free(rendered);
        cJSON_Delete(errRoot);
        *pReplyRoot = NULL; /* tell caller not to delete once again! */