From 90f1e5caf225a2e73806ccb4fb0b3003dab8cfa9 Mon Sep 17 00:00:00 2001 From: Dave Chapman Date: Fri, 8 May 2009 17:52:38 +0000 Subject: Check some previously unchecked return values in the standalone FLAC test program - fixes some warnings spotted by Mohamed Tarek El Haddad (mt). git-svn-id: svn://svn.rockbox.org/rockbox/trunk@20875 a1c6a512-1295-4272-9138-f99709370657 --- apps/codecs/libffmpegFLAC/main.c | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/apps/codecs/libffmpegFLAC/main.c b/apps/codecs/libffmpegFLAC/main.c index 4a989a4dd0..058c581aea 100644 --- a/apps/codecs/libffmpegFLAC/main.c +++ b/apps/codecs/libffmpegFLAC/main.c @@ -33,6 +33,7 @@ #include #include +#include #include #include #include @@ -63,7 +64,10 @@ int open_wav(char* filename) { fd=open(filename,O_CREAT|O_WRONLY|O_TRUNC,S_IRUSR|S_IWUSR); if (fd >= 0) { - write(fd,wav_header,sizeof(wav_header)); + if (write(fd,wav_header,sizeof(wav_header)) < sizeof(wav_header)) { + fprintf(stderr,"[ERR} Failed to write wav header\n"); + exit(1); + } } return(fd); } @@ -114,7 +118,10 @@ void close_wav(int fd, FLACContext* fc) { wav_header[43]=(x&0xff000000)>>24; lseek(fd,0,SEEK_SET); - write(fd,wav_header,sizeof(wav_header)); + if (write(fd,wav_header,sizeof(wav_header)) < sizeof(wav_header)) { + fprintf(stderr,"[ERR} Failed to write wav header\n"); + exit(1); + } close(fd); } @@ -295,7 +302,11 @@ int main(int argc, char* argv[]) { if (fc.bps==24) *(p++)=(decoded1[i]&0xff0000)>>16; } } - write(fdout,wavbuf,fc.blocksize*fc.channels*(fc.bps/8)); + n = fc.blocksize*fc.channels*(fc.bps/8); + if (write(fdout,wavbuf,n) < n) { + fprintf(stderr,"[ERR] Write failed\n"); + exit(1); + } memmove(buf,&buf[consumed],bytesleft-consumed); bytesleft-=consumed; -- cgit v1.2.3