Rob Shearman : urlmon: The pcbRead parameter for IInternetProtocol:: Read is optional, so fix the protocol implementation to check for this.

Alexandre Julliard julliard at wine.codeweavers.com
Wed Aug 8 08:59:53 CDT 2007


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

Author: Rob Shearman <rob at codeweavers.com>
Date:   Fri Aug  3 22:10:55 2007 +0100

urlmon: The pcbRead parameter for IInternetProtocol::Read is optional, so fix the protocol implementation to check for this.

---

 dlls/urlmon/binding.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/urlmon/binding.c b/dlls/urlmon/binding.c
index 5820d4f..894d7b5 100644
--- a/dlls/urlmon/binding.c
+++ b/dlls/urlmon/binding.c
@@ -487,12 +487,14 @@ static HRESULT WINAPI ProtocolStream_Read(IStream *iface, void *pv,
     }
 
     if(read == cb) {
-        *pcbRead = read;
+        if (pcbRead)
+            *pcbRead = read;
         return S_OK;
     }
 
     This->hres = IInternetProtocol_Read(This->protocol, (PBYTE)pv+read, cb-read, &pread);
-    *pcbRead = read + pread;
+    if (pcbRead)
+        *pcbRead = read + pread;
 
     if(This->hres == E_PENDING)
         return E_PENDING;




More information about the wine-cvs mailing list