advapi32: Don't read a value that might be uninitialized (Coverity).

Signed-off-by: Alistair Leslie-Hughes <leslie_alistair@hotmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

advapi32: Don't read a value that might be uninitialized (Coverity).
1b714ca0 · Alistair Leslie-Hughes · Alexandre Julliard · bf5b8ae8 · 1b714ca0
Commit 1b714ca0 authored Jun 17, 2019 by Alistair Leslie-Hughes Committed by Alexandre Julliard Jun 17, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

registry.c dlls/advapi32/registry.c +2 -2

No files found.
--- a/dlls/advapi32/registry.c
+++ b/dlls/advapi32/registry.c
@@ -2118,7 +2118,7 @@ LSTATUS WINAPI RegGetValueW( HKEY hKey, LPCWSTR pszSubKey, LPCWSTR pszValue,
                          DWORD dwFlags, LPDWORD pdwType, PVOID pvData,
                          LPDWORD pcbData )
 {
-    DWORD dwType, cbData = pcbData ? *pcbData : 0;
+    DWORD dwType, cbData = (pvData && pcbData) ? *pcbData : 0;
    PVOID pvBuf = NULL;
    LONG ret;

@@ -2214,7 +2214,7 @@ LSTATUS WINAPI RegGetValueA( HKEY hKey, LPCSTR pszSubKey, LPCSTR pszValue,
                          DWORD dwFlags, LPDWORD pdwType, PVOID pvData, 
                          LPDWORD pcbData )
 {
-    DWORD dwType, cbData = pcbData ? *pcbData : 0;
+    DWORD dwType, cbData = (pvData && pcbData) ? *pcbData : 0;
    PVOID pvBuf = NULL;
    LONG ret;