Bug 158977, GetResponseXML() can return null document even with NS_OK return value, so callers need to check the document exists before using it. r=bzbarsky, sr=jst.

This commit is contained in:
heikki%netscape.com 2002-08-16 23:20:49 +00:00
parent c923b74618
commit c3bcc86756
4 changed files with 10 additions and 4 deletions

View File

@ -358,7 +358,9 @@ LoadListener::HandleEvent(nsIDOMEvent *event)
if (NS_SUCCEEDED(rv)) {
nsCOMPtr<nsIDOMElement> element;
document->GetDocumentElement(getter_AddRefs(element));
if (document)
document->GetDocumentElement(getter_AddRefs(element));
if (element) {
rv = mLoader->ProcessSchemaElement(element, getter_AddRefs(schema));
}

View File

@ -364,7 +364,8 @@ nsWSDLLoadRequest::LoadDefinition(const nsAString& aURI)
}
nsCOMPtr<nsIDOMElement> element;
document->GetDocumentElement(getter_AddRefs(element));
if (document)
document->GetDocumentElement(getter_AddRefs(element));
if (element) {
if (IsElementOfNamespace(element,
NS_LITERAL_STRING(NS_WSDL_NAMESPACE))) {

View File

@ -358,7 +358,9 @@ LoadListener::HandleEvent(nsIDOMEvent *event)
if (NS_SUCCEEDED(rv)) {
nsCOMPtr<nsIDOMElement> element;
document->GetDocumentElement(getter_AddRefs(element));
if (document)
document->GetDocumentElement(getter_AddRefs(element));
if (element) {
rv = mLoader->ProcessSchemaElement(element, getter_AddRefs(schema));
}

View File

@ -364,7 +364,8 @@ nsWSDLLoadRequest::LoadDefinition(const nsAString& aURI)
}
nsCOMPtr<nsIDOMElement> element;
document->GetDocumentElement(getter_AddRefs(element));
if (document)
document->GetDocumentElement(getter_AddRefs(element));
if (element) {
if (IsElementOfNamespace(element,
NS_LITERAL_STRING(NS_WSDL_NAMESPACE))) {