xplist: fix memory leak when adding a node to its parent fails

node_from_xml() ignored the return values of plist_dict_set_item() and
plist_array_append_item(). When the insertion failed (e.g. when the
nesting depth limit is hit in node_attach()), the newly created node was
never attached, yet subnode was reset to NULL, so the node, its data and
its string value were leaked.

Check the return values and bail out with PLIST_ERR_PARSE so that the
unattached node is freed in the error path.

Credit to OSSFuzz

0 files changed, 0 insertions, 0 deletions