summaryrefslogtreecommitdiff
path: root/app-metrics/collectd/files/collectd-5.7.2-CVE-2017-16820.patch
diff options
context:
space:
mode:
Diffstat (limited to 'app-metrics/collectd/files/collectd-5.7.2-CVE-2017-16820.patch')
-rw-r--r--app-metrics/collectd/files/collectd-5.7.2-CVE-2017-16820.patch39
1 files changed, 0 insertions, 39 deletions
diff --git a/app-metrics/collectd/files/collectd-5.7.2-CVE-2017-16820.patch b/app-metrics/collectd/files/collectd-5.7.2-CVE-2017-16820.patch
deleted file mode 100644
index 0090f2940bf6..000000000000
--- a/app-metrics/collectd/files/collectd-5.7.2-CVE-2017-16820.patch
+++ /dev/null
@@ -1,39 +0,0 @@
-CVE-2017-16820: Fix double free of request PDU
-
-https://github.com/collectd/collectd/commit/d16c24542b2f96a194d43a73c2e5778822b9cb47
-
---- a/src/snmp.c
-+++ b/src/snmp.c
-@@ -1357,11 +1357,13 @@ static int csnmp_read_table(host_definition_t *host, data_definition_t *data) {
- if (oid_list_todo_num == 0) {
- /* The request is still empty - so we are finished */
- DEBUG("snmp plugin: all variables have left their subtree");
-+ snmp_free_pdu(req);
- status = 0;
- break;
- }
-
- res = NULL;
-+ /* snmp_sess_synch_response always frees our req PDU */
- status = snmp_sess_synch_response(host->sess_handle, req, &res);
- if ((status != STAT_SUCCESS) || (res == NULL)) {
- char *errstr = NULL;
-@@ -1376,8 +1378,6 @@ static int csnmp_read_table(host_definition_t *host, data_definition_t *data) {
- snmp_free_pdu(res);
- res = NULL;
-
-- /* snmp_synch_response already freed our PDU */
-- req = NULL;
- sfree(errstr);
- csnmp_host_close_session(host);
-
-@@ -1492,9 +1492,6 @@ static int csnmp_read_table(host_definition_t *host, data_definition_t *data) {
- snmp_free_pdu(res);
- res = NULL;
-
-- if (req != NULL)
-- snmp_free_pdu(req);
-- req = NULL;
-
- if (status == 0)
- csnmp_dispatch_table(host, data, instance_list_head, value_list_head);