[PATCH 4/4] comdlg32/fontdlg: Remove dead assignment (Coverity)

Nikolay Sivov nsivov at codeweavers.com
Mon Dec 21 16:19:33 CST 2015


Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---

For https://bugs.winehq.org/show_bug.cgi?id=39486

 dlls/comdlg32/fontdlg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/comdlg32/fontdlg.c b/dlls/comdlg32/fontdlg.c
index 37c82db..344a356 100644
--- a/dlls/comdlg32/fontdlg.c
+++ b/dlls/comdlg32/fontdlg.c
@@ -1011,7 +1011,7 @@ static LRESULT CFn_WMCommand(HWND hDlg, WPARAM wParam, LPARAM lParam, LPCHOOSEFO
             /* face name */
             i=SendDlgItemMessageW(hDlg,cmb1,CB_GETCURSEL,0,0);
             if (i==CB_ERR)
-                i=GetDlgItemTextW( hDlg, cmb1, str, 256 );
+                GetDlgItemTextW( hDlg, cmb1, str, sizeof(str)/sizeof(str[0]) );
             else
             {
                 SendDlgItemMessageW(hDlg,cmb1,CB_GETLBTEXT,i,
-- 
2.6.4




More information about the wine-patches mailing list