Marcus Meissner : shell32: Avoid memcmp result truncation (Coverity).
Alexandre Julliard
julliard at winehq.org
Mon Jul 9 14:56:37 CDT 2012
Module: wine
Branch: master
Commit: 0ef705fb69903d68c53a77700e167f68f9b8c55c
URL: http://source.winehq.org/git/wine.git/?a=commit;h=0ef705fb69903d68c53a77700e167f68f9b8c55c
Author: Marcus Meissner <marcus at jet.franken.de>
Date: Sat Jul 7 11:52:23 2012 +0200
shell32: Avoid memcmp result truncation (Coverity).
---
dlls/shell32/recyclebin.c | 9 ++++++++-
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/dlls/shell32/recyclebin.c b/dlls/shell32/recyclebin.c
index 6b7d4aa..a10bffa 100644
--- a/dlls/shell32/recyclebin.c
+++ b/dlls/shell32/recyclebin.c
@@ -457,12 +457,19 @@ static HRESULT WINAPI RecycleBin_BindToStorage(IShellFolder2 *This, LPCITEMIDLIS
static HRESULT WINAPI RecycleBin_CompareIDs(IShellFolder2 *iface, LPARAM lParam, LPCITEMIDLIST pidl1, LPCITEMIDLIST pidl2)
{
RecycleBin *This = impl_from_IShellFolder2(iface);
+ int ret;
/* TODO */
TRACE("(%p, %p, %p, %p)\n", This, (void *)lParam, pidl1, pidl2);
if (pidl1->mkid.cb != pidl2->mkid.cb)
return MAKE_HRESULT(SEVERITY_SUCCESS, 0, pidl1->mkid.cb - pidl2->mkid.cb);
- return MAKE_HRESULT(SEVERITY_SUCCESS, 0, (unsigned short)memcmp(pidl1->mkid.abID, pidl2->mkid.abID, pidl1->mkid.cb));
+ /* Looks too complicated, but in optimized memcpy we might get
+ * a 32bit wide difference and would truncate it to 16 bit, so
+ * erroneously returning equality. */
+ ret = memcmp(pidl1->mkid.abID, pidl2->mkid.abID, pidl1->mkid.cb);
+ if (ret < 0) ret = -1;
+ if (ret > 0) ret = 1;
+ return MAKE_HRESULT(SEVERITY_SUCCESS, 0, (unsigned short)ret);
}
static HRESULT WINAPI RecycleBin_CreateViewObject(IShellFolder2 *iface, HWND hwndOwner, REFIID riid, void **ppv)
More information about the wine-cvs
mailing list