[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 9/9] tcx: tcx_screen_dump(): add error handling
From: |
Luiz Capitulino |
Subject: |
[Qemu-devel] [PATCH 9/9] tcx: tcx_screen_dump(): add error handling |
Date: |
Wed, 29 Aug 2012 16:53:06 -0300 |
Signed-off-by: Luiz Capitulino <address@hidden>
---
hw/tcx.c | 35 +++++++++++++++++++++++++++++------
1 file changed, 29 insertions(+), 6 deletions(-)
diff --git a/hw/tcx.c b/hw/tcx.c
index 428649e..93994d6 100644
--- a/hw/tcx.c
+++ b/hw/tcx.c
@@ -582,26 +582,49 @@ static void tcx_screen_dump(void *opaque, const char
*filename, bool cswitch,
TCXState *s = opaque;
FILE *f;
uint8_t *d, *d1, v;
- int y, x;
+ int ret, y, x;
f = fopen(filename, "wb");
- if (!f)
+ if (!f) {
+ error_setg(errp, "failed to open file '%s': %s", filename,
+ strerror(errno));
return;
- fprintf(f, "P6\n%d %d\n%d\n", s->width, s->height, 255);
+ }
+ ret = fprintf(f, "P6\n%d %d\n%d\n", s->width, s->height, 255);
+ if (ret < 0) {
+ goto write_err;
+ }
d1 = s->vram;
for(y = 0; y < s->height; y++) {
d = d1;
for(x = 0; x < s->width; x++) {
v = *d;
- fputc(s->r[v], f);
- fputc(s->g[v], f);
- fputc(s->b[v], f);
+ ret = fputc(s->r[v], f);
+ if (ret == EOF) {
+ goto write_err;
+ }
+ ret = fputc(s->g[v], f);
+ if (ret == EOF) {
+ goto write_err;
+ }
+ ret = fputc(s->b[v], f);
+ if (ret == EOF) {
+ goto write_err;
+ }
d++;
}
d1 += MAXX;
}
+
+out:
fclose(f);
return;
+
+write_err:
+ error_setg(errp, "failed to write to file '%s': %s", filename,
+ strerror(errno));
+ unlink(filename);
+ goto out;
}
static void tcx24_screen_dump(void *opaque, const char *filename, bool cswitch,
--
1.7.11.2.249.g31c7954.dirty
- [Qemu-devel] [PATCH 2/9] console: vga_hw_screen_dump_ptr: take Error argument, (continued)
- [Qemu-devel] [PATCH 2/9] console: vga_hw_screen_dump_ptr: take Error argument, Luiz Capitulino, 2012/08/29
- [Qemu-devel] [PATCH 3/9] qapi: convert screendump, Luiz Capitulino, 2012/08/29
- [Qemu-devel] [PATCH 4/9] vga: ppm_save(): add error handling, Luiz Capitulino, 2012/08/29
- [Qemu-devel] [PATCH 5/9] omap_lcdc: rename ppm_save() to omap_ppm_save(), Luiz Capitulino, 2012/08/29
- [Qemu-devel] [PATCH 6/9] omap_lcdc: omap_ppm_save(): add error handling, Luiz Capitulino, 2012/08/29
- [Qemu-devel] [PATCH 7/9] g364fb: g364fb_screen_dump(): add error handling, Luiz Capitulino, 2012/08/29
- [Qemu-devel] [PATCH 8/9] tcx: tcx24_screen_dump(): add error handling, Luiz Capitulino, 2012/08/29
- [Qemu-devel] [PATCH 9/9] tcx: tcx_screen_dump(): add error handling,
Luiz Capitulino <=