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