Piotr Caban : msxml3: Fixed ISAXXMLReader_getContentHandler and ISAXXMLReader_getErrorHandler when second argument is NULL .

[Alexandre Julliard]

Module: wine
Branch: master
Commit: b3da220229a8fdf887a3ab3fdd1ef44bbb2451a7
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=b3da220229a8fdf887a3ab3fdd...

Author: Piotr Caban <piotr.caban(a)gmail.com>
Date:   Fri Jul 11 00:36:34 2008 +0200

msxml3: Fixed ISAXXMLReader_getContentHandler and ISAXXMLReader_getErrorHandler when second argument is NULL.

---

 dlls/msxml3/saxreader.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/dlls/msxml3/saxreader.c b/dlls/msxml3/saxreader.c
index a5654e9..abb2168 100644
--- a/dlls/msxml3/saxreader.c
+++ b/dlls/msxml3/saxreader.c
@@ -514,6 +514,8 @@ static HRESULT WINAPI isaxxmlreader_getContentHandler(
     saxreader *This = impl_from_ISAXXMLReader( iface );
 
     TRACE("(%p)->(%p)\n", This, pContentHandler);
+    if(pContentHandler == NULL)
+        return E_POINTER;
     if(This->contentHandler)
         ISAXContentHandler_AddRef(This->contentHandler);
     *pContentHandler = This->contentHandler;
@@ -564,6 +566,8 @@ static HRESULT WINAPI isaxxmlreader_getErrorHandler(
     saxreader *This = impl_from_ISAXXMLReader( iface );
 
     TRACE("(%p)->(%p)\n", This, pErrorHandler);
+    if(pErrorHandler == NULL)
+        return E_POINTER;
     if(This->errorHandler)
         ISAXErrorHandler_AddRef(This->errorHandler);
     *pErrorHandler = This->errorHandler;