diff --git a/apps/codec_thread.c b/apps/codec_thread.c
index e45e9aec6b..7189332673 100644
--- a/apps/codec_thread.c
+++ b/apps/codec_thread.c
@@ -127,13 +127,13 @@ static int codec_check_queue__have_msg(void)
     switch (ev.id)
     {
     case Q_CODEC_SEEK:
-        LOGFQUEUE("codec - Q_CODEC_SEEK", ev.id);
+        LOGFQUEUE("codec - Q_CODEC_SEEK %ld", ev.id);
         return -1;
     case Q_CODEC_PAUSE:
-        LOGFQUEUE("codec - Q_CODEC_PAUSE", ev.id);
+        LOGFQUEUE("codec - Q_CODEC_PAUSE %ld", ev.id);
         return -1;
     case Q_CODEC_STOP:
-        LOGFQUEUE("codec - Q_CODEC_STOP", ev.id);
+        LOGFQUEUE("codec - Q_CODEC_STOP %ld", ev.id);
         return -1;
     }
 
diff --git a/firmware/storage.c b/firmware/storage.c
index 790cddcd1a..1c7dfd7afb 100644
--- a/firmware/storage.c
+++ b/firmware/storage.c
@@ -694,6 +694,7 @@ int storage_spinup_time(void)
 #if (CONFIG_STORAGE & STORAGE_SD)
     //t=sd_spinup_time();
     //if (t>max) max=t;
+    (void)t;
 #endif
 
 #if (CONFIG_STORAGE & STORAGE_NAND)
diff --git a/tools/rbspeex/rbspeexdec.c b/tools/rbspeex/rbspeexdec.c
index 14ee971697..3cf65f0331 100644
--- a/tools/rbspeex/rbspeexdec.c
+++ b/tools/rbspeex/rbspeexdec.c
@@ -70,7 +70,7 @@ int main(int argc, char **argv)
     insize = ftell(fin);
     fseek(fin, 0, SEEK_SET);
     indata = malloc(insize);
-    fread(indata, 1, insize, fin);
+    insize = fread(indata, 1, insize, fin);
     fclose(fin);
 
     /* fill in wav header */
diff --git a/tools/voicefont.c b/tools/voicefont.c
index 4d1363b1bc..f4318b3b26 100644
--- a/tools/voicefont.c
+++ b/tools/voicefont.c
@@ -103,7 +103,8 @@ int voicefont(FILE* voicefontids,int targetnum,char* filedir, FILE* output, unsi
     memset(voiceonly, 0, sizeof(voiceonly));
     while (!feof(voicefontids))
     {
-        fgets(line, sizeof(line), voicefontids);
+        if (!fgets(line, sizeof(line), voicefontids))
+            break;
         if (line[0] == '#') /* comment */
             continue;