Bug 1433414: Add missing NS_ENSURE_SUCCESS r?kmag
This adds a missing NS_ENSURE_SUCCESS, to catch XML parsing failures (e.g. unclosed tags).
MozReview-Commit-ID: BvxIWTF7IWf
--- a/dom/base/nsContentUtils.cpp
+++ b/dom/base/nsContentUtils.cpp
@@ -5215,16 +5215,17 @@ nsContentUtils::ParseFragmentXML(const n
NS_IF_RELEASE(sXMLFragmentParser);
NS_IF_RELEASE(sXMLFragmentSink);
return rv;
}
rv = sXMLFragmentSink->FinishFragmentParsing(aReturn);
sXMLFragmentParser->Reset();
+ NS_ENSURE_SUCCESS(rv, rv);
// If this is a chrome-privileged document, sanitize the fragment before
// returning.
if (aSanitize != NeverSanitize && !aDocument->AllowUnsafeHTML()) {
// Don't fire mutation events for nodes removed by the sanitizer.
nsAutoScriptBlockerSuppressNodeRemoved scriptBlocker;
RefPtr<DocumentFragment> fragment = static_cast<DocumentFragment*>(*aReturn);