urlmon: Assign to structs instead of using memcpy

Andrew Talbot andrew.talbot at talbotville.com
Tue Mar 18 16:42:20 CDT 2008


Changelog:
    urlmon: Assign to structs instead of using memcpy.

diff --git a/dlls/urlmon/binding.c b/dlls/urlmon/binding.c
index c647230..47b521c 100644
--- a/dlls/urlmon/binding.c
+++ b/dlls/urlmon/binding.c
@@ -1083,7 +1083,7 @@ static HRESULT WINAPI InternetProtocolSink_Switch(IInternetProtocolSink *iface,
     TRACE("(%p)->(%p)\n", This, pProtocolData);
 
     task = heap_alloc(sizeof(switch_task_t));
-    memcpy(&task->data, pProtocolData, sizeof(PROTOCOLDATA));
+    task->data = *pProtocolData;
 
     push_task(This, &task->header, switch_proc);
 
@@ -1382,7 +1382,7 @@ static HRESULT WINAPI InternetBindInfo_GetBindInfo(IInternetBindInfo *iface,
 
     *grfBINDF = This->bindf;
 
-    memcpy(pbindinfo, &This->bindinfo, sizeof(BINDINFO));
+    *pbindinfo = This->bindinfo;
 
     if(pbindinfo->szExtraInfo || pbindinfo->szCustomVerb)
         FIXME("copy strings\n");
diff --git a/dlls/urlmon/urlmon_main.c b/dlls/urlmon/urlmon_main.c
index 7d56062..bc3de93 100644
--- a/dlls/urlmon/urlmon_main.c
+++ b/dlls/urlmon/urlmon_main.c
@@ -398,7 +398,7 @@ HRESULT WINAPI CopyStgMedium(const STGMEDIUM *src, STGMEDIUM *dst)
     if(!src || !dst)
         return E_POINTER;
 
-    memcpy(dst, src, sizeof(STGMEDIUM));
+    *dst = *src;
 
     switch(dst->tymed) {
     case TYMED_NULL:



More information about the wine-patches mailing list