[PATCH] shldocvw: Fixed typo = vs == (Coverity)

Marcus Meissner marcus at jet.franken.de
Wed Mar 2 12:42:06 CST 2011


On Fri, Feb 25, 2011 at 09:30:21PM +0100, Eric Pouech wrote:
> Le 25/02/2011 14:16, Marcus Meissner a écrit :
> >CID1801
> >---
> >  dlls/shdocvw/shlinstobj.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> >diff --git a/dlls/shdocvw/shlinstobj.c b/dlls/shdocvw/shlinstobj.c
> >index d272c91..067e678 100644
> >--- a/dlls/shdocvw/shlinstobj.c
> >+++ b/dlls/shdocvw/shlinstobj.c
> >@@ -376,7 +376,7 @@ HRESULT SHDOCVW_GetShellInstanceObjectClassObject(REFCLSID rclsid, REFIID riid,
> >
> >      /* Figure if there is an 'Instance' subkey for the given CLSID and acquire a handle. */
> >      if (!StringFromGUID2(rclsid, wszInstanceKey + 6, CHARS_IN_GUID) ||
> >-        !(wszInstanceKey[5+CHARS_IN_GUID]='\\') || /* Repair the null-termination. */
> >+        !(wszInstanceKey[5+CHARS_IN_GUID] == '\\') || /* Repair the null-termination. */
> >          ERROR_SUCCESS != RegOpenKeyExW(HKEY_CLASSES_ROOT, wszInstanceKey, 0, KEY_READ,&hInstanceKey))
> >      {
> >          /* If there is no 'Instance' subkey, then it's not a Shell Instance Object. */
> are you sure about the fix ?
> it looks like it's intentional that the test always succeed (it just
> replaces the null termination char written in StringFromGUID2 with a
> '\\')
> (and this would explain the comment as well ;-)
> I'm not familiar with the code... just being suspicious

This was right, I fixed it. :)

Ciao, Marcus



More information about the wine-devel mailing list