On Thu, Feb 11, 2010 at 07:45:40PM +0100, Jiri Denemark wrote: > > > + for (i = 0; i < ncpus; i++) { > > > + doc = xmlParseMemory(xmlCPUs[i], strlen(xmlCPUs[i])); > > > + if (doc == NULL || (ctxt = xmlXPathNewContext(doc)) == NULL) > > > + goto no_memory; > > > > Hum, no, doc == NULL might come from parsing error, raising an OOMError > > in that case sounds weird, unless you're 100% sure this is well formed > > XML... > > Hmm, right. If you don't mind I will address this separately together with > similar older code. Yeah, sure ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list