winhttp: Avoid buffer overrun in alloc_handle.

Signed-off-by: Thomas Faber <thomas.faber@reactos.org> Signed-off-by: Hans Leidekker <hans@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winhttp: Avoid buffer overrun in alloc_handle.
83413751 · Thomas Faber · Alexandre Julliard · af6f8ca4 · 83413751
Commit 83413751 authored Jun 26, 2016 by Thomas Faber Committed by Alexandre Julliard Jul 03, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

handle.c dlls/winhttp/handle.c +1 -1

No files found.
--- a/dlls/winhttp/handle.c
+++ b/dlls/winhttp/handle.c
@@ -112,7 +112,7 @@ HINTERNET alloc_handle( object_header_t *hdr )
    if (handles[handle]) ERR("handle isn't free but should be\n");

    handles[handle] = addref_object( hdr );
-    while (handles[next_handle] && (next_handle < max_handles)) next_handle++;
+    while ((next_handle < max_handles) && handles[next_handle]) next_handle++;

 end:
    LeaveCriticalSection( &handle_cs );