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

Revert "Let snmp_synch_response deal with PDU freeing"

This reverts commit e767dc103a142e1c57800f06a9c384f21a48aef5.

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)