Mikołaj Zalewski : comctl32: toolbar: Test and fix invalid indexes passed in TB_ISBUTTON*.
Alexandre Julliard
julliard at wine.codeweavers.com
Fri Jul 20 06:02:17 CDT 2007
Module: wine
Branch: master
Commit: 31be5005c633dee1476a77d93aa27e9c3f628c79
URL: http://source.winehq.org/git/wine.git/?a=commit;h=31be5005c633dee1476a77d93aa27e9c3f628c79
Author: Mikołaj Zalewski <mikolaj at zalewski.pl>
Date: Thu Jul 19 12:48:46 2007 +0200
comctl32: toolbar: Test and fix invalid indexes passed in TB_ISBUTTON*.
---
dlls/comctl32/tests/toolbar.c | 9 +++++++++
dlls/comctl32/toolbar.c | 12 ++++++------
2 files changed, 15 insertions(+), 6 deletions(-)
diff --git a/dlls/comctl32/tests/toolbar.c b/dlls/comctl32/tests/toolbar.c
index f5476d3..b46ef0d 100644
--- a/dlls/comctl32/tests/toolbar.c
+++ b/dlls/comctl32/tests/toolbar.c
@@ -147,6 +147,15 @@ static void basic_test(void)
ok(SendMessage(hToolbar, TB_ISBUTTONCHECKED, 1000, 0), "A1 pressed\n");
ok(!SendMessage(hToolbar, TB_ISBUTTONCHECKED, 1007, 0), "B2 not pressed\n");
ok(SendMessage(hToolbar, TB_ISBUTTONCHECKED, 1008, 0), "B3 pressed\n");
+
+ /* tests with invalid index */
+ compare(SendMessage(hToolbar, TB_ISBUTTONCHECKED, 0xdeadbeef, 0), -1L, "%ld");
+ compare(SendMessage(hToolbar, TB_ISBUTTONPRESSED, 0xdeadbeef, 0), -1L, "%ld");
+ compare(SendMessage(hToolbar, TB_ISBUTTONENABLED, 0xdeadbeef, 0), -1L, "%ld");
+ compare(SendMessage(hToolbar, TB_ISBUTTONINDETERMINATE, 0xdeadbeef, 0), -1L, "%ld");
+ compare(SendMessage(hToolbar, TB_ISBUTTONHIGHLIGHTED, 0xdeadbeef, 0), -1L, "%ld");
+ compare(SendMessage(hToolbar, TB_ISBUTTONHIDDEN, 0xdeadbeef, 0), -1L, "%ld");
+
DestroyWindow(hToolbar);
}
diff --git a/dlls/comctl32/toolbar.c b/dlls/comctl32/toolbar.c
index da6ceab..275d3d5 100644
--- a/dlls/comctl32/toolbar.c
+++ b/dlls/comctl32/toolbar.c
@@ -3840,7 +3840,7 @@ TOOLBAR_IsButtonChecked (HWND hwnd, WPARAM wParam, LPARAM lParam)
nIndex = TOOLBAR_GetButtonIndex (infoPtr, (INT)wParam, FALSE);
if (nIndex == -1)
- return FALSE;
+ return -1;
return (infoPtr->buttons[nIndex].fsState & TBSTATE_CHECKED);
}
@@ -3854,7 +3854,7 @@ TOOLBAR_IsButtonEnabled (HWND hwnd, WPARAM wParam, LPARAM lParam)
nIndex = TOOLBAR_GetButtonIndex (infoPtr, (INT)wParam, FALSE);
if (nIndex == -1)
- return FALSE;
+ return -1;
return (infoPtr->buttons[nIndex].fsState & TBSTATE_ENABLED);
}
@@ -3868,7 +3868,7 @@ TOOLBAR_IsButtonHidden (HWND hwnd, WPARAM wParam, LPARAM lParam)
nIndex = TOOLBAR_GetButtonIndex (infoPtr, (INT)wParam, FALSE);
if (nIndex == -1)
- return TRUE;
+ return -1;
return (infoPtr->buttons[nIndex].fsState & TBSTATE_HIDDEN);
}
@@ -3882,7 +3882,7 @@ TOOLBAR_IsButtonHighlighted (HWND hwnd, WPARAM wParam, LPARAM lParam)
nIndex = TOOLBAR_GetButtonIndex (infoPtr, (INT)wParam, FALSE);
if (nIndex == -1)
- return FALSE;
+ return -1;
return (infoPtr->buttons[nIndex].fsState & TBSTATE_MARKED);
}
@@ -3896,7 +3896,7 @@ TOOLBAR_IsButtonIndeterminate (HWND hwnd, WPARAM wParam, LPARAM lParam)
nIndex = TOOLBAR_GetButtonIndex (infoPtr, (INT)wParam, FALSE);
if (nIndex == -1)
- return FALSE;
+ return -1;
return (infoPtr->buttons[nIndex].fsState & TBSTATE_INDETERMINATE);
}
@@ -3910,7 +3910,7 @@ TOOLBAR_IsButtonPressed (HWND hwnd, WPARAM wParam, LPARAM lParam)
nIndex = TOOLBAR_GetButtonIndex (infoPtr, (INT)wParam, FALSE);
if (nIndex == -1)
- return FALSE;
+ return -1;
return (infoPtr->buttons[nIndex].fsState & TBSTATE_PRESSED);
}
More information about the wine-cvs
mailing list