parseSelector: plug memory leaks, and don't set result until error free complete

Author: boxerab

src/parse/language.c

@@ -1737,16 +1737,17 @@ css_error parseSelector(css_language *c, const parserutils_vector *vector,
 	error = parseSimpleSelector(c, vector, ctx, &selector);
 	if (error != CSS_OK)
 		return error;
-	*result = selector;
 
 	while ((token = parserutils_vector_peek(vector, *ctx)) != NULL &&
 			tokenIsChar(token, ',') == false) {
 		css_combinator comb = CSS_COMBINATOR_NONE;
 		css_selector *other = NULL;
 
 		error = parseCombinator(c, vector, ctx, &comb);
-		if (error != CSS_OK)
+		if (error != CSS_OK) {
+			css__stylesheet_selector_destroy(c->sheet, selector);
 			return error;
+		}
 
 		/* In the case of "html , body { ... }", the whitespace after
 		 * "html" and "body" will be considered an ancestor combinator.
@@ -1761,21 +1762,24 @@ css_error parseSelector(css_language *c, const parserutils_vector *vector,
 			continue;
 
 		error = parseSimpleSelector(c, vector, ctx, &other);
-		if (error != CSS_OK)
+		if (error != CSS_OK) {
+			css__stylesheet_selector_destroy(c->sheet, selector);
 			return error;
-
-		*result = other;
+		}
 
 		error = css__stylesheet_selector_combine(c->sheet,
 				comb, selector, other);
 		if (error != CSS_OK) {
 			css__stylesheet_selector_destroy(c->sheet, selector);
+			css__stylesheet_selector_destroy(c->sheet, other);
 			return error;
 		}
 
 		selector = other;
 	}
 
+	*result = selector;
+
 	return CSS_OK;
 }