Alexandre Julliard : wldap32: Fix some potential infinite loops because of an unsigned loop counter.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 4397e9958534b24b72fb66b836426a5cf07c379b
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=4397e9958534b24b72fb66b836...

Author: Alexandre Julliard 
Date:   Thu Jul 12 17:32:24 2007 +0200

wldap32: Fix some potential infinite loops because of an unsigned loop counter.

---

dlls/wldap32/init.c  |    3 +--
 dlls/wldap32/value.c |    4 +---
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/dlls/wldap32/init.c b/dlls/wldap32/init.c
index 71dab84..a1cf79b 100644
--- a/dlls/wldap32/init.c
+++ b/dlls/wldap32/init.c
@@ -109,8 +109,7 @@ static char **split_hostnames( const char *hostnames )
     return res;

oom:
-    for (--i; i >= 0; i--)
-        strfreeU( res[i] );
+    while (i > 0) strfreeU( res[--i] );

HeapFree( GetProcessHeap(), 0, res );
     HeapFree( GetProcessHeap(), 0, str );
diff --git a/dlls/wldap32/value.c b/dlls/wldap32/value.c
index 4a9aa37..1d1155b 100644
--- a/dlls/wldap32/value.c
+++ b/dlls/wldap32/value.c
@@ -190,9 +190,7 @@ static char **bv2str_array( struct berval **bv )
         str[i] = bv2str( *p );
         if (!str[i])
         {
-            for (--i; i >= 0; i--)
-                HeapFree( GetProcessHeap(), 0, str[i] );
-
+            while (i > 0) HeapFree( GetProcessHeap(), 0, str[--i] );
             HeapFree( GetProcessHeap(), 0, str );
             return NULL;
         }