diff options
author | emaste <emaste@FreeBSD.org> | 2010-10-05 15:27:44 +0000 |
---|---|---|
committer | emaste <emaste@FreeBSD.org> | 2010-10-05 15:27:44 +0000 |
commit | 58eabff0a0ef2cf6d939dca7ab351d6e3e2d8e49 (patch) | |
tree | 5ea9a9900facb6ecaafbb8c60d34bb21d0c88850 /lib/libgeom | |
parent | e1cc06374b84d53c44a5d2c5832466696d8aa196 (diff) | |
download | FreeBSD-src-58eabff0a0ef2cf6d939dca7ab351d6e3e2d8e49.zip FreeBSD-src-58eabff0a0ef2cf6d939dca7ab351d6e3e2d8e49.tar.gz |
Handle null return from XML_ParserCreate and fix a few memory leaks on
error conditions.
Submitted by: Mark Johnston <mjohnston at sandvine dot com>
MFC after: 2 weeks
Diffstat (limited to 'lib/libgeom')
-rw-r--r-- | lib/libgeom/geom_xml2tree.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/lib/libgeom/geom_xml2tree.c b/lib/libgeom/geom_xml2tree.c index 821449b..24315cd 100644 --- a/lib/libgeom/geom_xml2tree.c +++ b/lib/libgeom/geom_xml2tree.c @@ -339,21 +339,27 @@ geom_xml2tree(struct gmesh *gmp, char *p) memset(gmp, 0, sizeof *gmp); LIST_INIT(&gmp->lg_class); parser = XML_ParserCreate(NULL); + if (parser == NULL) + return (ENOMEM); mt = calloc(1, sizeof *mt); - if (mt == NULL) + if (mt == NULL) { + XML_ParserFree(parser); return (ENOMEM); + } mt->mesh = gmp; XML_SetUserData(parser, mt); XML_SetElementHandler(parser, StartElement, EndElement); XML_SetCharacterDataHandler(parser, CharData); i = XML_Parse(parser, p, strlen(p), 1); - if (i != 1) - return (-1); XML_ParserFree(parser); + if (i != 1) { + free(mt); + return (-1); + } gmp->lg_ident = calloc(sizeof *gmp->lg_ident, mt->nident + 1); + free(mt); if (gmp->lg_ident == NULL) return (ENOMEM); - free(mt); i = 0; /* Collect all identifiers */ LIST_FOREACH(cl, &gmp->lg_class, lg_class) { |