From: Marc Fournier <marc.fournier@camptocamp.com>
Date: Wed, 19 Nov 2014 16:59:30 +0000 (+0100)
Subject: Revert "Let snmp_synch_response deal with PDU freeing"
X-Git-Tag: collectd-5.4.2~7
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=232898b5a6ee58d432a63d932a0ff447a606241d;p=collectd.git

Revert "Let snmp_synch_response deal with PDU freeing"

This reverts commit 9d6f72e5054cfbb183585991fefcf119259d71c4.

The proper fix to this issue is in the collectd-4.10 branch. See also
79e90bba.
---

diff --git a/src/snmp.c b/src/snmp.c
index b90e0ffa..3e6cb911 100644
--- a/src/snmp.c
+++ b/src/snmp.c
@@ -1437,9 +1437,8 @@ static int csnmp_read_table (host_definition_t *host, data_definition_t *data)
     snmp_free_pdu (res);
   res = NULL;
 
-  /*
-   * memory is handled by snmp_synch_response
-   */
+  if (req != NULL)
+    snmp_free_pdu (req);
   req = NULL;
 
   if (status == 0)