[Spice-devel] [PATCH 2/2] utils: handle errors writing to bitmap file
Christophe Fergeau
cfergeau at redhat.com
Fri Jan 15 04:20:25 PST 2016
Acked-by: Christophe Fergeau <cfergeau at redhat.com>
On Thu, Jan 14, 2016 at 05:32:45PM +0000, Frediano Ziglio wrote:
> ---
> server/spice-bitmap-utils.c | 36 ++++++++++++++++++++++++++++--------
> 1 file changed, 28 insertions(+), 8 deletions(-)
>
> diff --git a/server/spice-bitmap-utils.c b/server/spice-bitmap-utils.c
> index 64102f3..63ddb82 100644
> --- a/server/spice-bitmap-utils.c
> +++ b/server/spice-bitmap-utils.c
> @@ -153,24 +153,37 @@ static void put_32le(uint8_t **ptr, uint32_t val)
> put_16le(ptr, val & 0xffffu);
> }
>
> -static void dump_palette(FILE *f, SpicePalette* plt)
> +#define WRITE(buf, size, f) do { \
> + if (fwrite(buf, 1, (size), f) != (size)) \
> + goto write_err; \
> +} while(0)
> +
> +static bool dump_palette(FILE *f, SpicePalette* plt)
> {
> int i;
> for (i = 0; i < plt->num_ents; i++) {
> - fwrite(plt->ents + i, sizeof(uint32_t), 1, f);
> + WRITE(plt->ents + i, sizeof(uint32_t), f);
> }
> + return true;
> +
> +write_err:
> + return false;
> }
>
> -static void dump_line(FILE *f, uint8_t* line, uint16_t n_pixel_bits, int width, int row_size)
> +static bool dump_line(FILE *f, uint8_t* line, uint16_t n_pixel_bits, int width, int row_size)
> {
> static char zeroes[4] = { 0 };
> int copy_bytes_size = SPICE_ALIGN(n_pixel_bits * width, 8) / 8;
>
> - fwrite(line, 1, copy_bytes_size, f);
> + WRITE(line, copy_bytes_size, f);
> // each line should be 4 bytes aligned
> - fwrite(zeroes, 1, row_size - copy_bytes_size, f);
> g_return_val_if_fail(row_size - copy_bytes_size >= 0 && row_size - copy_bytes_size <= 4, false);
> spice_assert(row_size - copy_bytes_size >= 0 && row_size - copy_bytes_size <= 4);
> + WRITE(zeroes, row_size - copy_bytes_size, f);
> + return true;
> +
> +write_err:
> + return false;
> }
>
> void dump_bitmap(SpiceBitmap *bitmap)
> @@ -269,18 +282,25 @@ void dump_bitmap(SpiceBitmap *bitmap)
> put_32le(&ptr, !plt ? 0 : plt->num_ents); // plt entries
> put_32le(&ptr, 0);
>
> - fwrite(header, 1, ptr - header, f);
> + WRITE(header, ptr - header, f);
>
> if (plt) {
> - dump_palette(f, plt);
> + if (!dump_palette(f, plt))
> + goto write_err;
> }
> /* writing the data */
> for (i = 0; i < bitmap->data->num_chunks; i++) {
> SpiceChunk *chunk = &bitmap->data->chunk[i];
> int num_lines = chunk->len / bitmap->stride;
> for (j = 0; j < num_lines; j++) {
> - dump_line(f, chunk->data + (j * bitmap->stride), n_pixel_bits, bitmap->x, row_size);
> + if (!dump_line(f, chunk->data + (j * bitmap->stride), n_pixel_bits, bitmap->x, row_size))
> + goto write_err;
> }
> }
> fclose(f);
> + return;
> +
> +write_err:
> + fclose(f);
> + remove(file_str);
> }
> --
> 2.4.3
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20160115/ed2fed5a/attachment.sig>
More information about the Spice-devel
mailing list