Marcus Meissner : shell32: Check for NULL lpText (Coverity).
Alexandre Julliard
julliard at winehq.org
Wed Dec 2 10:22:13 CST 2009
Module: wine
Branch: master
Commit: 482354bdf74daaac18f0c3b572e6e3f569f3fbd3
URL: http://source.winehq.org/git/wine.git/?a=commit;h=482354bdf74daaac18f0c3b572e6e3f569f3fbd3
Author: Marcus Meissner <marcus at jet.franken.de>
Date: Tue Dec 1 23:14:10 2009 +0100
shell32: Check for NULL lpText (Coverity).
---
dlls/shell32/shlmenu.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/dlls/shell32/shlmenu.c b/dlls/shell32/shlmenu.c
index efdd6b5..957711f 100644
--- a/dlls/shell32/shlmenu.c
+++ b/dlls/shell32/shlmenu.c
@@ -392,12 +392,15 @@ BOOL WINAPI FileMenu_AppendItemAW(
{
BOOL ret;
- if ((SHELL_OsIsUnicode() && (lpText!=FM_SEPARATOR)) || (lpText == NULL))
+ if (!lpText) return FALSE;
+
+ if (SHELL_OsIsUnicode() || lpText == FM_SEPARATOR)
ret = FileMenu_AppendItemW(hMenu, lpText, uID, icon, hMenuPopup, nItemHeight);
else
{
DWORD len = MultiByteToWideChar( CP_ACP, 0, lpText, -1, NULL, 0 );
LPWSTR lpszText = HeapAlloc ( GetProcessHeap(), 0, len*sizeof(WCHAR) );
+ if (!lpszText) return FALSE;
MultiByteToWideChar( CP_ACP, 0, lpText, -1, lpszText, len );
ret = FileMenu_AppendItemW(hMenu, lpszText, uID, icon, hMenuPopup, nItemHeight);
HeapFree( GetProcessHeap(), 0, lpszText );
More information about the wine-cvs
mailing list