Jacek Caban : mshtml: Fixed argument checking in IHTMLDOMChildrenCollection ::item.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 91dcf287c7818eec598e69803da6f864e6b8a8c4
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=91dcf287c7818eec598e69803d...

Author: Jacek Caban <jacek(a)codeweavers.com>
Date:   Mon Jun 23 20:03:06 2008 -0500

mshtml: Fixed argument checking in IHTMLDOMChildrenCollection::item.

---

 dlls/mshtml/htmlnode.c  |    2 +-
 dlls/mshtml/tests/dom.c |    5 +++++
 2 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/dlls/mshtml/htmlnode.c b/dlls/mshtml/htmlnode.c
index bfa999f..056e38a 100644
--- a/dlls/mshtml/htmlnode.c
+++ b/dlls/mshtml/htmlnode.c
@@ -166,7 +166,7 @@ static HRESULT WINAPI HTMLDOMChildrenCollection_item(IHTMLDOMChildrenCollection
     TRACE("(%p)->(%ld %p)\n", This, index, ppItem);
 
     nsIDOMNodeList_GetLength(This->nslist, &length);
-    if(index < 0 || index > length)
+    if(index < 0 || index >= length)
         return E_INVALIDARG;
 
     nsres = nsIDOMNodeList_Item(This->nslist, index, &nsnode);
diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c
index 7d57b24..4da9110 100644
--- a/dlls/mshtml/tests/dom.c
+++ b/dlls/mshtml/tests/dom.c
@@ -2042,6 +2042,11 @@ static void test_elems(IHTMLDocument2 *doc)
         ok(hres == E_INVALIDARG, "item failed: %08x, expected E_INVALIDARG\n", hres);
         ok(disp == (void*)0xdeadbeef, "disp=%p\n", disp);
 
+        disp = (void*)0xdeadbeef;
+        hres = IHTMLDOMChildrenCollection_item(child_col, length, &disp);
+        ok(hres == E_INVALIDARG, "item failed: %08x, expected E_INVALIDARG\n", hres);
+        ok(disp == (void*)0xdeadbeef, "disp=%p\n", disp);
+
         test_child_col_disp(child_col);
 
         IHTMLDOMChildrenCollection_Release(child_col);