]> granicus.if.org Git - php/commitdiff
Added missing safety checks (Problem identified by Coverity scan)
authorIlia Alshanetsky <iliaa@php.net>
Sat, 7 Oct 2006 19:40:58 +0000 (19:40 +0000)
committerIlia Alshanetsky <iliaa@php.net>
Sat, 7 Oct 2006 19:40:58 +0000 (19:40 +0000)
ext/dom/document.c

index 8bbd4696575c6ede7bbd7aeaf5444678bd726d53..a8202dd63751df16d95557301c8b6da33395222b 100644 (file)
@@ -1355,7 +1355,9 @@ PHP_METHOD(domdocument, __construct)
                        }
                }
                intern->document = NULL;
-               php_libxml_increment_doc_ref((php_libxml_node_object *)intern, docp TSRMLS_CC);
+               if (php_libxml_increment_doc_ref((php_libxml_node_object *)intern, docp TSRMLS_CC) == -1) {
+                       RETURN_FALSE;
+               }
                php_libxml_increment_node_ptr((php_libxml_node_object *)intern, (xmlNodePtr)docp, (void *)intern TSRMLS_CC);
        }
 }
@@ -1568,7 +1570,9 @@ static void dom_parse_document(INTERNAL_FUNCTION_PARAMETERS, int mode) {
                                }
                        }
                        intern->document = NULL;
-                       php_libxml_increment_doc_ref((php_libxml_node_object *)intern, newdoc TSRMLS_CC);
+                       if (php_libxml_increment_doc_ref((php_libxml_node_object *)intern, newdoc TSRMLS_CC) == -1) {
+                               RETURN_FALSE;
+                       }
                        intern->document->doc_props = doc_prop;
                }
 
@@ -2069,7 +2073,9 @@ static void dom_load_html(INTERNAL_FUNCTION_PARAMETERS, int mode)
                                }
                        }
                        intern->document = NULL;
-                       php_libxml_increment_doc_ref((php_libxml_node_object *)intern, newdoc TSRMLS_CC);
+                       if (php_libxml_increment_doc_ref((php_libxml_node_object *)intern, newdoc TSRMLS_CC) == -1) {
+                               RETURN_FALSE;
+                       }
                        intern->document->doc_props = doc_prop;
                }