Marcus Meissner : shell32: Arrays are never NULL (Coverity).

Alexandre Julliard julliard at winehq.org
Tue Mar 15 11:33:57 CDT 2011


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

Author: Marcus Meissner <meissner at suse.de>
Date:   Tue Mar 15 02:13:54 2011 +0100

shell32: Arrays are never NULL (Coverity).

---

 dlls/shell32/control.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/shell32/control.c b/dlls/shell32/control.c
index 3be0494..ffe96ff 100644
--- a/dlls/shell32/control.c
+++ b/dlls/shell32/control.c
@@ -129,13 +129,13 @@ CPlApplet*	Control_LoadApplet(HWND hWnd, LPCWSTR cmd, CPanel* panel)
           but proper data in CPL_NEWINQUIRE. if we get an empty string or a null
           icon, see what we can get from CPL_NEWINQUIRE */
 
-       if ((applet->info[i].szName == 0) || (lstrlenW(applet->info[i].szName) == 0))
+       if (lstrlenW(applet->info[i].szName) == 0)
            info.idName = CPL_DYNAMIC_RES;
 
        /* zero-length szInfo may not be a buggy applet, but it doesn't hurt for us
           to check anyway */
 
-       if ((applet->info[i].szInfo == 0) || (lstrlenW(applet->info[i].szInfo) == 0))
+       if (lstrlenW(applet->info[i].szInfo) == 0)
            info.idInfo = CPL_DYNAMIC_RES;
 
        if (applet->info[i].hIcon == NULL)




More information about the wine-cvs mailing list