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

1473. [bug] create_map() and create_string() failed to handle out

Browse Source 
of memory cleanup.  [RT #6813]
This commit is contained in:
Mark Andrews
2003-07-03 00:43:28 +00:00
parent edc6991f4b
commit 3425cd6803
2 changed files with 8 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
CHANGES
View File

@@ -11,7 +11,7 @@
1474. [port] Provide strtoul() and memmove() for platforms
without them.
1473. [bug] free_map() and free_string() failed to handle out
1473. [bug] create_map() and create_string() failed to handle out
of memory cleanup. [RT #6813]
1472. [contrib] idnkit-1.0 from JPNIC, replaces mdnkit.

15
lib/isccfg/parser.c
View File

@@ -15,7 +15,7 @@
* WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*/
/* $Id: parser.c,v 1.108 2003/06/20 02:50:48 marka Exp $ */
/* $Id: parser.c,v 1.109 2003/07/03 00:43:28 marka Exp $ */
#include <config.h>
@@ -674,7 +674,7 @@ create_string(cfg_parser_t *pctx, const char *contents, const cfg_type_t *type,
obj->value.string.length = len;
obj->value.string.base = isc_mem_get(pctx->mctx, len + 1);
if (obj->value.string.base == 0) {
CLEANUP_OBJ(obj);
isc_mem_put(pctx->mctx, obj, sizeof(*obj));
return (ISC_R_NOMEMORY);
}
memcpy(obj->value.string.base, contents, len);
@@ -793,9 +793,8 @@ print_qstring(cfg_printer_t *pctx, cfg_obj_t *obj) {
static void
free_string(cfg_parser_t *pctx, cfg_obj_t *obj) {
if (obj->value.string.base != NULL)
isc_mem_put(pctx->mctx, obj->value.string.base,
obj->value.string.length + 1);
isc_mem_put(pctx->mctx, obj->value.string.base,
obj->value.string.length + 1);
}
isc_boolean_t
@@ -2226,14 +2225,14 @@ create_map(cfg_parser_t *pctx, const cfg_type_t *type, cfg_obj_t **ret) {
return (ISC_R_SUCCESS);
cleanup:
CLEANUP_OBJ(obj);
if (obj != NULL)
isc_mem_put(pctx->mctx, obj, sizeof(*obj));
return (result);
}
static void
free_map(cfg_parser_t *pctx, cfg_obj_t *obj) {
if (obj->value.map.id != NULL)
CLEANUP_OBJ(obj->value.map.id);
CLEANUP_OBJ(obj->value.map.id);
isc_symtab_destroy(&obj->value.map.symtab);
}