From patchwork Thu Mar 8 14:20:41 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: iptables: missing free() in function delete_entry() Date: Thu, 08 Mar 2012 04:20:41 -0000 From: Christian Engelmayer X-Patchwork-Id: 145537 Message-Id: <20120308152041.7a789c4f@frequentis.com> To: Cc: , From: Franz Flasch Fixed a memory leak in the dry run path of function delete_entry(). Signed-off-by: Franz Flasch Signed-off-by: Christian Engelmayer --- -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- iptables-1.4.12.2/libiptc/libiptc.c 2012-01-02 18:19:09.000000000 +0100 +++ iptables-1.4.12.2-modified/libiptc/libiptc.c 2012-03-08 14:27:24.000000000 +0100 @@ -2001,8 +2001,10 @@ static int delete_entry(const IPT_CHAINL continue; /* if we are just doing a dry run, we simply skip the rest */ - if (dry_run) + if (dry_run){ + free(r); return 1; + } /* If we are about to delete the rule that is the * current iterator, move rule iterator back. next