From e5c84f0b7a2e2cef8d8630bd8c26a2f859e959ff Mon Sep 17 00:00:00 2001 From: Pierre Joye Date: Tue, 7 Sep 2021 22:03:21 +0700 Subject: [PATCH] Partial fix for #750 --- src/gd_bmp.c | 14 +++++++++++--- src/gd_webp.c | 7 ++++++- 2 files changed, 17 insertions(+), 4 deletions(-) diff --git a/src/gd_bmp.c b/src/gd_bmp.c index e186ac9..29a05ce 100644 --- a/src/gd_bmp.c +++ b/src/gd_bmp.c @@ -266,7 +266,11 @@ static int _gdImageBmpCtx(gdImagePtr im, gdIOCtxPtr out, int compression) bitmap_size += compressed_size; - gdPutBuf(uncompressed_row, compressed_size, out); + if (gdPutBuf(uncompressed_row, compressed_size, out) != compressed_size){ + gd_error("gd-bmp write error\n"); + error = 1; + break; + } gdPutC(BMP_RLE_COMMAND, out); gdPutC(BMP_RLE_ENDOFLINE, out); bitmap_size += 2; @@ -322,10 +326,14 @@ static int _gdImageBmpCtx(gdImagePtr im, gdIOCtxPtr out, int compression) } while ((buffer_size = gdGetBuf(copy_buffer, 1024, out)) != EOF) { + int res; if (buffer_size == 0) { break; } - gdPutBuf(copy_buffer , buffer_size, out_original); + if (gdPutBuf(copy_buffer , buffer_size, out_original) != buffer_size) { + gd_error("gd-bmp write error\n"); + error = 1; + } } gdFree(copy_buffer); @@ -335,7 +343,7 @@ static int _gdImageBmpCtx(gdImagePtr im, gdIOCtxPtr out, int compression) out_original = NULL; } - ret = 0; + ret = error; cleanup: if (tmpfile_for_compression) { #ifdef _WIN32 diff --git a/src/gd_webp.c b/src/gd_webp.c index bf89087..e9ad802 100644 --- a/src/gd_webp.c +++ b/src/gd_webp.c @@ -227,8 +227,13 @@ static int _gdImageWebpCtx (gdImagePtr im, gdIOCtx * outfile, int quality) ret = 1; goto freeargb; } - gdPutBuf(out, out_size, outfile); + + int res = gdPutBuf(out, out_size, outfile); free(out); + if (res != out_size) { + gd_error("gd-webp write error\n"); + ret = 1; + } freeargb: gdFree(argb);