From 9ca9b657e4f1d9481cc795422d74274abf9d41f1 Mon Sep 17 00:00:00 2001 From: Frank Gevaerts Date: Tue, 3 Apr 2012 20:59:11 +0200 Subject: Revert 1207c7b and fix the actual warnings. The warnings 1207c7b was supposed to get rid of are sim-only. This means the binsize argument used for the _FORTIFY_SOURCE fix did not apply at all. This one actually checks the return values. Change-Id: Ic4cd8c25de4305310baa868c077a66981cdfcb4b --- firmware/target/hosted/sdl/lcd-charcells.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) (limited to 'firmware/target/hosted') diff --git a/firmware/target/hosted/sdl/lcd-charcells.c b/firmware/target/hosted/sdl/lcd-charcells.c index 9ceb5de391..3a623827db 100644 --- a/firmware/target/hosted/sdl/lcd-charcells.c +++ b/firmware/target/hosted/sdl/lcd-charcells.c @@ -167,7 +167,11 @@ void screen_dump(void) if (fd < 0) return; - write(fd, bmpheader, sizeof(bmpheader)); + if(write(fd, bmpheader, sizeof(bmpheader)) != sizeof(bmpheader)) + { + close(fd); + return; + } SDL_LockSurface(lcd_surface); /* BMP image goes bottom up */ @@ -191,7 +195,11 @@ void screen_dump(void) dst_mask = 0x80; } } - write(fd, line, sizeof(line)); + if(write(fd, line, sizeof(line)) != sizeof(line)) + { + close(fd); + return; + } } SDL_UnlockSurface(lcd_surface); close(fd); -- cgit