From 78749b497596137827d66b15de92e213f7bdac3b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?IOhannes=20m=20zm=C3=B6lnig?= Date: Wed, 18 Aug 2010 09:02:16 +0000 Subject: read the return value of fread (but then ignore it) svn path=/trunk/externals/iemlib/; revision=13825 --- iem_mp3/src/mp3play~.c | 28 ++++++++++++++++++---------- 1 file changed, 18 insertions(+), 10 deletions(-) (limited to 'iem_mp3') diff --git a/iem_mp3/src/mp3play~.c b/iem_mp3/src/mp3play~.c index 73f37ad..9a14e79 100644 --- a/iem_mp3/src/mp3play~.c +++ b/iem_mp3/src/mp3play~.c @@ -3190,18 +3190,20 @@ static int mp3play_tilde_read_frame_length_first(t_mp3play_tilde *x, int *frsz) unsigned long head; unsigned char chead; + size_t len; + *frsz = 0; if((x->file_size) >= 4) { - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head = (unsigned int)chead; - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head <<= 8; head |= (unsigned int)chead; - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head <<= 8; head |= (unsigned int)chead; - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head <<= 8; head |= (unsigned int)chead; syncword = (head >> 20) & 0x0fff; @@ -3276,18 +3278,20 @@ static int mp3play_tilde_read_frame_length_next(t_mp3play_tilde *x, int *frsz, i unsigned long head; unsigned char chead; + size_t len; + *frsz = 0; if(begframeseek[frame_counter] < ((x->file_size) - 4)) { - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head = (unsigned int)chead; - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head <<= 8; head |= (unsigned int)chead; - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head <<= 8; head |= (unsigned int)chead; - fread(&chead, 1, sizeof(char), x->fh); + len = fread(&chead, 1, sizeof(char), x->fh); head <<= 8; head |= (unsigned int)chead; if(head & (1<<20)) @@ -3371,6 +3375,8 @@ static void mp3play_tilde_do_open(t_mp3play_tilde *x, char *str, int calc_it) static char *layers[4] = { "Unknown" , "I", "II", "III" }; char completefilename[400]; + size_t len; + if(x->file_is_open) { post("mp3play-ERROR: file is already open, please stop it first!"); @@ -3442,7 +3448,7 @@ static void mp3play_tilde_do_open(t_mp3play_tilde *x, char *str, int calc_it) } if(mp3_read_length > 0) { - fread(x->mp3inbuf, mp3_read_length, sizeof(char), x->fh); + len = fread(x->mp3inbuf, mp3_read_length, sizeof(char), x->fh); mp3_encode_return = decodeMP3(&(x->mp), x->mp3inbuf, mp3_read_length, x->mp3outbuf, MY_MP3_MALLOC_IN_SIZE2, &size); if(mp3_encode_return == MP3_EX) @@ -3539,6 +3545,8 @@ static t_int *mp3play_tilde_perform(t_int *w) float scale = x->scale, outa, outb; int mp3_out_index = x->mp3_out_index; + size_t len; + if (!x->file_is_open) goto mp3play_tilde_labelzero; if (x->play_state != 1) @@ -3577,7 +3585,7 @@ static t_int *mp3play_tilde_perform(t_int *w) goto mp3play_tilde_labelzero; } - fread(x->mp3inbuf, mp3_read_length, sizeof(char), x->fh); + len = fread(x->mp3inbuf, mp3_read_length, sizeof(char), x->fh); mp3_encode_return = decodeMP3(&(x->mp), x->mp3inbuf, mp3_read_length, x->mp3outbuf, MY_MP3_MALLOC_IN_SIZE2, &size); x->mp3_encode_size = size * sizeof(char) / sizeof(short); -- cgit v1.2.1