When the commandID was being treated as the index, no check for index

greater than number of buttons was done. Now fixed.

When the commandID was being treated as the index, no check for index
d3a52c28 · Guy L. Albertelli · Alexandre Julliard · d2ace6ad · d3a52c28
Commit d3a52c28 authored Feb 12, 2002 by Guy L. Albertelli Committed by Alexandre Julliard Feb 12, 2002
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 0 deletions

toolbar.c dlls/comctl32/toolbar.c +1 -0

No files found.
--- a/dlls/comctl32/toolbar.c
+++ b/dlls/comctl32/toolbar.c
@@ -1332,6 +1332,7 @@ TOOLBAR_GetButtonIndex (TOOLBAR_INFO *infoPtr, INT idCommand, BOOL CommandIsInde

    if (CommandIsIndex) {
 	TRACE("command is really index command=%d\n", idCommand);
+	if (idCommand >= infoPtr->nNumButtons) return -1;
 	return idCommand;
    }
    btnPtr = infoPtr->buttons;