From aa487e6c75a6a9bd70189707f977fd66ba9c12a7 Mon Sep 17 00:00:00 2001 From: Avuton Olrich Date: Sun, 20 Aug 2006 03:11:12 +0000 Subject: This fixes 5 potential bugs where the conditional would always be true. git-svn-id: https://svn.musicpd.org/mpd/trunk@4659 09075e82-0dd4-0310-85a5-a0d7c8717e4f --- src/charConv.c | 2 +- src/inputPlugins/_ogg_common.c | 2 +- src/inputPlugins/aac_plugin.c | 4 ++-- src/myfprintf.c | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/charConv.c b/src/charConv.c index 870e6c3e2..62fc5f68e 100644 --- a/src/charConv.c +++ b/src/charConv.c @@ -122,7 +122,7 @@ char *convStrDup(char *string) iconv(char_conv_iconv, &string, &inleft, &bufferPtr, &outleft); if (outleft == BUFFER_SIZE - || (err < 0 && errno != E2BIG)) { + || (err == -1L && errno != E2BIG)) { free(ret); return NULL; } diff --git a/src/inputPlugins/_ogg_common.c b/src/inputPlugins/_ogg_common.c index 8abdc33bc..e4a264ac3 100644 --- a/src/inputPlugins/_ogg_common.c +++ b/src/inputPlugins/_ogg_common.c @@ -40,7 +40,7 @@ ogg_stream_type ogg_stream_type_detect(InputStream * inStream) while (to_read) { r = readFromInputStream(inStream, buf, 1, to_read); - if (r < 0) + if (r < inStream->error) break; to_read -= r; if (!r && !inputStreamAtEOF(inStream)) diff --git a/src/inputPlugins/aac_plugin.c b/src/inputPlugins/aac_plugin.c index c24254b54..8f7f83a9f 100644 --- a/src/inputPlugins/aac_plugin.c +++ b/src/inputPlugins/aac_plugin.c @@ -239,7 +239,7 @@ static float getAacFloatTotalTime(char *file) unsigned int sampleRate; unsigned char channels; InputStream inStream; - size_t bread; + long bread; if (openInputStream(&inStream, file) < 0) return -1; @@ -291,7 +291,7 @@ static int aac_decode(OutputBuffer * cb, DecoderControl * dc, char *path) faacDecHandle decoder; faacDecFrameInfo frameInfo; faacDecConfigurationPtr config; - size_t bread; + long bread; unsigned int sampleRate; unsigned char channels; int eof = 0; diff --git a/src/myfprintf.c b/src/myfprintf.c index 6f0da2fdd..dac58ab34 100644 --- a/src/myfprintf.c +++ b/src/myfprintf.c @@ -36,7 +36,7 @@ static void blockingWrite(const int fd, const char *string, size_t len) { while (len) { - size_t ret = xwrite(fd, string, len); + ssize_t ret = xwrite(fd, string, len); if (ret == len) return; if (ret >= 0) { -- cgit v1.2.3