[PATCH 3/4] setupapi: Always fill output parameters in SetupCopyOEMInfA().

[Zebediah Figura]

Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
---
 dlls/setupapi/misc.c       | 12 +++---------
 dlls/setupapi/tests/misc.c |  1 -
 2 files changed, 3 insertions(+), 10 deletions(-)

diff --git a/dlls/setupapi/misc.c b/dlls/setupapi/misc.c
index 7298ac65b62..67e2eafa41e 100644
--- a/dlls/setupapi/misc.c
+++ b/dlls/setupapi/misc.c
@@ -851,12 +851,9 @@ BOOL WINAPI SetupCopyOEMInfA( PCSTR source, PCSTR location,
     if (source && !(sourceW = strdupAtoW( source ))) goto done;
     if (location && !(locationW = strdupAtoW( location ))) goto done;
 
-    if (!(ret = SetupCopyOEMInfW( sourceW, locationW, media_type, style, destW,
-                                  buffer_size, &size, NULL )))
-    {
-        if (required_size) *required_size = size;
-        goto done;
-    }
+    ret = SetupCopyOEMInfW( sourceW, locationW, media_type, style, destW, buffer_size, &size, NULL );
+
+    if (required_size) *required_size = size;
 
     if (dest)
     {
@@ -866,10 +863,7 @@ BOOL WINAPI SetupCopyOEMInfA( PCSTR source, PCSTR location,
             if (component) *component = strrchr( dest, '\\' ) + 1;
         }
         else
-        {
             SetLastError( ERROR_INSUFFICIENT_BUFFER );
-            goto done;
-        }
     }
 
 done:
diff --git a/dlls/setupapi/tests/misc.c b/dlls/setupapi/tests/misc.c
index e2dd12fa3f6..7e597708f07 100644
--- a/dlls/setupapi/tests/misc.c
+++ b/dlls/setupapi/tests/misc.c
@@ -246,7 +246,6 @@ static void test_SetupCopyOEMInf(void)
     ok(res == FALSE, "Expected FALSE, got %d\n", res);
     ok(GetLastError() == ERROR_FILE_EXISTS,
        "Expected ERROR_FILE_EXISTS, got %d\n", GetLastError());
-todo_wine
     ok(!strcmp(orig_dest, dest), "Expected '%s', got '%s'.\n", orig_dest, dest);
 
     SetLastError(0xdeadbeef);
-- 
2.21.0