Do some bounds checking on iOrder.

c00ded58 · Ge van Geldorp · Alexandre Julliard · f9b3edb4 · c00ded58
Commit c00ded58 authored Sep 18, 2005 by Ge van Geldorp Committed by Alexandre Julliard Sep 18, 2005
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

header.c dlls/comctl32/header.c +4 -0

No files found.
--- a/dlls/comctl32/header.c
+++ b/dlls/comctl32/header.c
@@ -930,6 +930,10 @@ HEADER_InsertItemA (HWND hwnd, WPARAM wParam, LPARAM lParam)
        nItem = infoPtr->uNumItem;

    iOrder = (phdi->mask & HDI_ORDER) ? phdi->iOrder : nItem;
+    if (iOrder < 0)
+        iOrder = 0;
+    else if (infoPtr->uNumItem < iOrder)
+        iOrder = infoPtr->uNumItem;

    if (infoPtr->uNumItem == 0) {
        infoPtr->items = Alloc (sizeof (HEADER_ITEM));