mfplat: Fix early return from GetString() (Coverity).

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

mfplat: Fix early return from GetString() (Coverity).
53f0a25e · Nikolay Sivov · Alexandre Julliard · 63352b5c · 53f0a25e
Commit 53f0a25e authored Apr 06, 2019 by Nikolay Sivov Committed by Alexandre Julliard Apr 08, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

main.c dlls/mfplat/main.c +3 -3

No files found.
--- a/dlls/mfplat/main.c
+++ b/dlls/mfplat/main.c
@@ -1260,9 +1260,9 @@ HRESULT attributes_GetString(struct attributes *attributes, REFGUID key, WCHAR *
                *length = len;

            if (size <= len)
-                return STRSAFE_E_INSUFFICIENT_BUFFER;
-
-            memcpy(value, attribute->value.u.pwszVal, (len + 1) * sizeof(WCHAR));
+                hr = STRSAFE_E_INSUFFICIENT_BUFFER;
+            else
+                memcpy(value, attribute->value.u.pwszVal, (len + 1) * sizeof(WCHAR));
        }
        else
            hr = MF_E_INVALIDTYPE;