wineps: Don't leak memory in error case (Coverity).

69b335b5 · Fabian Maurer · Alexandre Julliard · 1d412eac · 69b335b5
Commit 69b335b5 authored Dec 10, 2023 by Fabian Maurer Committed by Alexandre Julliard Dec 19, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 5 deletions

ps.c dlls/wineps.drv/ps.c +7 -5

No files found.
--- a/dlls/wineps.drv/ps.c
+++ b/dlls/wineps.drv/ps.c
@@ -985,6 +985,12 @@ BOOL PSDRV_WriteDIBPatternDict(print_ctx *ctx, const BITMAPINFO *bmi, BYTE *bits
 	return FALSE;
    }

+    if (usage > 2)
+    {
+        FIXME("wrong usage: %d\n", usage);
+        return FALSE;
+    }
+
    w = bmi->bmiHeader.biWidth;
    h = abs_height;

@@ -1025,11 +1031,7 @@ BOOL PSDRV_WriteDIBPatternDict(print_ctx *ctx, const BITMAPINFO *bmi, BYTE *bits
        map[0] = GetTextColor( ctx->hdc );
        map[1] = GetBkColor( ctx->hdc );
    }
-    else
-    {
-        FIXME("wrong usage: %d\n", usage);
-        return FALSE;
-    }
+
    PSDRV_WriteRGB(ctx, map, 2);
    PSDRV_WriteIndexColorSpaceEnd(ctx);