ComputerSurge/bind9
3
0
Fork 0
Code Pull Requests 1 Activity

2431. [bug] Acl processing could leak memory. [RT #18323]

Browse Source
This commit is contained in:
Mark Andrews
2008-09-01 05:40:22 +00:00
parent a11d334790
commit 2be97eb46a
2 changed files with 3 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
CHANGES
View File

@@ -1,3 +1,5 @@
2431. [bug] Acl processing could leak memory. [RT #18323]
2430. [bug] win32: isc_interval_set() could round down to
zero if the input was less than NS_INTERVAL
nanoseconds. Round up instead. [RT #18549]

15
lib/isccfg/aclconf.c
View File

@@ -15,7 +15,7 @@
* PERFORMANCE OF THIS SOFTWARE.
*/
/* $Id: aclconf.c,v 1.17.2.3 2008/07/19 00:09:17 each Exp $ */
/* $Id: aclconf.c,v 1.17.2.4 2008/09/01 05:40:22 marka Exp $ */
#include <config.h>
@@ -175,7 +175,6 @@ cfg_acl_fromconfig(const cfg_obj_t *caml,
const cfg_listelt_t *elt;
dns_iptable_t *iptab;
int new_nest_level = 0;
int nelem;
if (nest_level != 0)
new_nest_level = nest_level - 1;
@@ -207,8 +206,6 @@ cfg_acl_fromconfig(const cfg_obj_t *caml,
return (result);
}
nelem = cfg_list_length(caml, ISC_FALSE);
de = dacl->elements;
for (elt = cfg_list_first(caml);
elt != NULL;
@@ -354,16 +351,6 @@ nested_acl:
if (result != ISC_R_SUCCESS)
goto cleanup;
/*
* There was only one element and it was
* a nested named ACL; attach it to the
* target and let's go home.
*/
if (nelem == 1) {
dns_acl_attach(inneracl, target);
goto cleanup;
}
goto nested_acl;
}
} else {