# HG changeset patch # User peter1138 # Date 1150218826 0 # Node ID d6cfec2cda04e48d08d2b30e23bef28e17450b0e # Parent 1d6387c126ff882c2b9220103f486ef981e75425 (svn r5249) - Add code to copy the palette to a temporary byte aligned array when making a PCX screenshot, if the palette array is not byte aligned. diff --git a/screenshot.c b/screenshot.c --- a/screenshot.c +++ b/screenshot.c @@ -285,6 +285,7 @@ uint maxlines; uint y; PcxHeader pcx; + bool success; if (pixelformat != 8 || w == 0) return false; @@ -388,14 +389,24 @@ return false; } - {assert_compile(sizeof(*palette) == 3);} - if (fwrite(palette, 256 * sizeof(*palette), 1, f) != 1) { - fclose(f); - return false; + if (sizeof(*palette) == 3) { + success = fwrite(palette, 256 * sizeof(*palette), 1, f) == 1; + } else { + /* If the palette is word-aligned, copy it to a temporary byte array */ + byte *tmp = malloc(256 * 3); + uint i; + for (i = 0; i < 256; i++) { + tmp[i * 3 + 0] = palette[i].r; + tmp[i * 3 + 1] = palette[i].g; + tmp[i * 3 + 2] = palette[i].b; + } + success = fwrite(tmp, 256 * 3, 1, f) == 1; + free(tmp); } + fclose(f); - return true; + return success; } //************************************************