mscms: Free memory and handle on error path (Coverity).

3f3404d2 · Nikolay Sivov · Alexandre Julliard · 6ee869da · 3f3404d2
Commit 3f3404d2 authored Mar 31, 2013 by Nikolay Sivov Committed by Alexandre Julliard Apr 01, 2013
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

profile.c dlls/mscms/profile.c +5 -1

No files found.
--- a/dlls/mscms/profile.c
+++ b/dlls/mscms/profile.c
@@ -1528,14 +1528,18 @@ HPROFILE WINAPI OpenColorProfileW( PPROFILE profile, DWORD access, DWORD sharing
    if (cmsprofile)
    {
        struct profile profile;
+        HPROFILE hprof;

        profile.file = handle;
        profile.access = access;
        profile.iccprofile = iccprofile;
        profile.cmsprofile = cmsprofile;

-        return create_profile( &profile );
+        if ((hprof = create_profile( &profile ))) return hprof;
+        HeapFree( GetProcessHeap(), 0, iccprofile );
+        cmsCloseProfile( cmsprofile );
    }
+    CloseHandle( handle );

 #endif /* HAVE_LCMS */
    return NULL;