[PATCH 12/16] [Msxml]: get_length

Eric Pouech eric.pouech at wanadoo.fr
Sun Oct 22 14:48:42 CDT 2006


- fixed list walking in xmlnodelist_get_length
  (spotted by valgrind)

A+
---

 dlls/msxml3/nodelist.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/dlls/msxml3/nodelist.c b/dlls/msxml3/nodelist.c
index 6dd2d21..a1b4306 100644
--- a/dlls/msxml3/nodelist.c
+++ b/dlls/msxml3/nodelist.c
@@ -334,17 +334,20 @@ static HRESULT WINAPI xmlnodelist_get_le
 	return S_OK;
     }
         
+    curr = This->node;
+
     if(This->enum_children)
     {
         tmp = curr;
         top_level_node = &tmp;
     }
 
-    for(curr = This->node; curr; curr = get_next_node(&This->xinfo, curr, top_level_node))
+    while (curr)
     {
         r = xslt_next_match( &This->xinfo, &curr, top_level_node );
         if(FAILED(r) || !curr) break;
         nodeCount++;
+        curr = get_next_node(&This->xinfo, curr, top_level_node);
     }
 
     *listLength = nodeCount;



More information about the wine-patches mailing list