ChildAt(0) can return NULL, so check for it!

This commit is contained in:
rjc%netscape.com 1999-03-02 04:07:10 +00:00
Родитель 67406839f2
Коммит f8b4395e6d
3 изменённых файлов: 3 добавлений и 0 удалений

Просмотреть файл

@ -527,6 +527,7 @@ RDFElementImpl::GetFirstChild(nsIDOMNode** aFirstChild)
nsresult rv;
nsIContent* child;
if (NS_SUCCEEDED(rv = ChildAt(0, child))) {
if (nsnull == child) return(NS_ERROR_FAILURE);
rv = child->QueryInterface(kIDOMNodeIID, (void**) aFirstChild);
NS_ASSERTION(NS_SUCCEEDED(rv), "not a DOM node");
NS_RELEASE(child); // balance the AddRef in ChildAt()

Просмотреть файл

@ -527,6 +527,7 @@ RDFElementImpl::GetFirstChild(nsIDOMNode** aFirstChild)
nsresult rv;
nsIContent* child;
if (NS_SUCCEEDED(rv = ChildAt(0, child))) {
if (nsnull == child) return(NS_ERROR_FAILURE);
rv = child->QueryInterface(kIDOMNodeIID, (void**) aFirstChild);
NS_ASSERTION(NS_SUCCEEDED(rv), "not a DOM node");
NS_RELEASE(child); // balance the AddRef in ChildAt()

Просмотреть файл

@ -527,6 +527,7 @@ RDFElementImpl::GetFirstChild(nsIDOMNode** aFirstChild)
nsresult rv;
nsIContent* child;
if (NS_SUCCEEDED(rv = ChildAt(0, child))) {
if (nsnull == child) return(NS_ERROR_FAILURE);
rv = child->QueryInterface(kIDOMNodeIID, (void**) aFirstChild);
NS_ASSERTION(NS_SUCCEEDED(rv), "not a DOM node");
NS_RELEASE(child); // balance the AddRef in ChildAt()