mshtml: Check for null pointers in IHTMLLocation_get*.
authorAndrew Eikum <aeikum@codeweavers.com>
Mon, 28 Sep 2009 18:31:08 +0000 (13:31 -0500)
committerAlexandre Julliard <julliard@winehq.org>
Tue, 29 Sep 2009 10:06:06 +0000 (12:06 +0200)
dlls/mshtml/htmllocation.c

index 470d679c26ac299c3ee961128abcdaf16fb78830..67ebf99143c30fc558cca56703826f5e47625c48 100644 (file)
@@ -168,6 +168,10 @@ static HRESULT WINAPI HTMLLocation_get_protocol(IHTMLLocation *iface, BSTR *p)
 {
     HTMLLocation *This = HTMLLOCATION_THIS(iface);
     FIXME("(%p)->(%p)\n", This, p);
+
+    if(!p)
+        return E_POINTER;
+
     return E_NOTIMPL;
 }
 
@@ -182,6 +186,10 @@ static HRESULT WINAPI HTMLLocation_get_host(IHTMLLocation *iface, BSTR *p)
 {
     HTMLLocation *This = HTMLLOCATION_THIS(iface);
     FIXME("(%p)->(%p)\n", This, p);
+
+    if(!p)
+        return E_POINTER;
+
     return E_NOTIMPL;
 }
 
@@ -196,6 +204,10 @@ static HRESULT WINAPI HTMLLocation_get_hostname(IHTMLLocation *iface, BSTR *p)
 {
     HTMLLocation *This = HTMLLOCATION_THIS(iface);
     FIXME("(%p)->(%p)\n", This, p);
+
+    if(!p)
+        return E_POINTER;
+
     return E_NOTIMPL;
 }
 
@@ -210,6 +222,10 @@ static HRESULT WINAPI HTMLLocation_get_port(IHTMLLocation *iface, BSTR *p)
 {
     HTMLLocation *This = HTMLLOCATION_THIS(iface);
     FIXME("(%p)->(%p)\n", This, p);
+
+    if(!p)
+        return E_POINTER;
+
     return E_NOTIMPL;
 }
 
@@ -231,6 +247,9 @@ static HRESULT WINAPI HTMLLocation_get_pathname(IHTMLLocation *iface, BSTR *p)
 
     TRACE("(%p)->(%p)\n", This, p);
 
+    if(!p)
+        return E_POINTER;
+
     hres = get_url(This, &doc_url);
     if(FAILED(hres))
         return hres;
@@ -271,6 +290,10 @@ static HRESULT WINAPI HTMLLocation_get_search(IHTMLLocation *iface, BSTR *p)
 {
     HTMLLocation *This = HTMLLOCATION_THIS(iface);
     FIXME("(%p)->(%p)\n", This, p);
+
+    if(!p)
+        return E_POINTER;
+
     return E_NOTIMPL;
 }
 
@@ -285,6 +308,10 @@ static HRESULT WINAPI HTMLLocation_get_hash(IHTMLLocation *iface, BSTR *p)
 {
     HTMLLocation *This = HTMLLOCATION_THIS(iface);
     FIXME("(%p)->(%p)\n", This, p);
+
+    if(!p)
+        return E_POINTER;
+
     return E_NOTIMPL;
 }