wbemprox: Avoid a use after free (Coverity).

Signed-off-by: Sven Baars <sbaars@codeweavers.com> Signed-off-by: Hans Leidekker <hans@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

wbemprox: Avoid a use after free (Coverity).
ea60927f · Sven Baars · Alexandre Julliard · 462edf6e · ea60927f
Commit ea60927f authored Aug 02, 2020 by Sven Baars Committed by Alexandre Julliard Aug 03, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

builtin.c dlls/wbemprox/builtin.c +2 -2

No files found.
--- a/dlls/wbemprox/builtin.c
+++ b/dlls/wbemprox/builtin.c
@@ -2116,9 +2116,9 @@ static WCHAR *get_diskdrive_serialnumber( WCHAR letter )
            heap_free( desc );
            break;
        }
+        size = desc->Size;
        heap_free( desc );
-        if (GetLastError() == ERROR_MORE_DATA) size = desc->Size;
-        else break;
+        if (GetLastError() != ERROR_MORE_DATA) break;
    }

 done: