winhttp: Do not free name too early (Coverity).

ab636923 · Marcus Meissner · Alexandre Julliard · e0fa39b3 · ab636923
Commit ab636923 authored Dec 27, 2012 by Marcus Meissner Committed by Alexandre Julliard Dec 28, 2012
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

session.c dlls/winhttp/session.c +2 -1

No files found.
--- a/dlls/winhttp/session.c
+++ b/dlls/winhttp/session.c
@@ -1247,7 +1247,6 @@ BOOL WINAPI WinHttpDetectAutoProxyConfigUrl( DWORD flags, LPWSTR *url )
            strcpy( name, "wpad" );
            strcat( name, p );
            res = getaddrinfo( name, NULL, NULL, &ai );
-            heap_free( name );
            if (!res)
            {
                *url = build_wpad_url( name, ai );
@@ -1255,10 +1254,12 @@ BOOL WINAPI WinHttpDetectAutoProxyConfigUrl( DWORD flags, LPWSTR *url )
                if (*url)
                {
                    TRACE("returning %s\n", debugstr_w(*url));
+                    heap_free( name );
                    ret = TRUE;
                    break;
                }
            }
+            heap_free( name );
            p++;
        }
        heap_free( domain );