Code Police raid: -W -Wall cleansing
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@17045 a1c6a512-1295-4272-9138-f99709370657
diff --git a/tools/creative.c b/tools/creative.c
index bcabc4f..6e0e468 100644
--- a/tools/creative.c
+++ b/tools/creative.c
@@ -17,40 +17,65 @@
*
****************************************************************************/
-#include "creative.h"
-#include "hmac-sha1.h"
#include <stdio.h>
#include <stdbool.h>
#include <stdlib.h>
#include <string.h>
+#include "creative.h"
+#include "hmac-sha1.h"
+
+static const char null_key_v1[] = "CTL:N0MAD|PDE0.SIGN.";
+static const char null_key_v2[] = "CTL:N0MAD|PDE0.DPMP.";
+static const char null_key_v3[] = "CTL:Z3N07|PDE0.DPMP.";
+static const char null_key_v4[] = "CTL:N0MAD|PDE0.DPFP.";
+
+static const struct device_info devices[] =
+{
+ {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0e\0n\0 \0V\0i\0s\0i\0o\0n\0:\0M",
+ 42, null_key_v2},
+ {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0e\0n\0 \0V\0i\0s\0i\0o\0n\0:\0M\0 \0G\0o\0!",
+ 50, null_key_v2},
+ {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0e\0n\0 \0V\0i\0s\0i\0o\0n\0 \0©\0T\0L",
+ 48, null_key_v2},
+ {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0E\0N\0 \0V", 42, null_key_v4}
+};
+
+
/*
Create a Zen Vision:M FRESCUE structure file
*/
+extern void int2le(unsigned int val, unsigned char* addr);
+extern unsigned int le2int(unsigned char* buf);
-static int make_ciff_file(char *inbuf, int length, char *outbuf, int device)
+static int make_ciff_file(unsigned char *inbuf, int length,
+ unsigned char *outbuf, int device)
{
+ char key[20];
memcpy(outbuf, "FFIC", 4);
int2le(length+90, &outbuf[4]);
memcpy(&outbuf[8], "FNIC", 4);
int2le(96, &outbuf[0xC]);
memcpy(&outbuf[0x10], devices[device].cinf, devices[device].cinf_size);
- memset(&outbuf[0x10+devices[device].cinf_size], 0, 96 - devices[device].cinf_size);
+ memset(&outbuf[0x10+devices[device].cinf_size], 0,
+ 96 - devices[device].cinf_size);
memcpy(&outbuf[0x70], "ATAD", 4);
int2le(length+32, &outbuf[0x74]);
- memcpy(&outbuf[0x78], "H\0j\0u\0k\0e\0b\0o\0x\0\x32\0.\0j\0r\0m", 32); /*Unicode encoded*/
+ memcpy(&outbuf[0x78], "H\0j\0u\0k\0e\0b\0o\0x\0\x32\0.\0j\0r\0m",
+ 32); /*Unicode encoded*/
memcpy(&outbuf[0x98], inbuf, length);
memcpy(&outbuf[0x98+length], "LLUN", 4);
int2le(20, &outbuf[0x98+length+4]);
/* Do checksum */
- char key[20];
- hmac_sha((char*)devices[device].null, strlen(devices[device].null), outbuf, 0x98+length, key, 20);
+ hmac_sha((char *)devices[device].null, strlen(devices[device].null),
+ (char *)outbuf, 0x98+length, key, 20);
memcpy(&outbuf[0x98+length+8], key, 20);
return length+0x90+0x1C+8;
}
-static int make_jrm_file(char *inbuf, int length, char *outbuf)
+static int make_jrm_file(unsigned char *inbuf, int length,
+ unsigned char *outbuf)
{
int i;
unsigned int sum = 0;
@@ -101,7 +126,6 @@
FILE *file;
unsigned char *outbuf;
unsigned char *buf;
- int i;
file = fopen(iname, "rb");
if (!file) {
@@ -120,7 +144,7 @@
}
len = fread(buf, 1, length, file);
- if(len < length) {
+ if(len < (size_t)length) {
perror(iname);
return -2;
}
@@ -147,7 +171,7 @@
}
len = fwrite(buf, 1, length, file);
- if(len < length) {
+ if(len < (size_t)length) {
free(buf);
perror(oname);
return -4;
diff --git a/tools/creative.h b/tools/creative.h
index c28001a..ff8b4a0 100644
--- a/tools/creative.h
+++ b/tools/creative.h
@@ -28,26 +28,13 @@
ZENV = 3
};
-static struct device_info
+struct device_info
{
const char* cinf; /*Must be Unicode encoded*/
const int cinf_size;
const char* null;
} device_info;
-static const char null_key_v1[] = "CTL:N0MAD|PDE0.SIGN.";
-static const char null_key_v2[] = "CTL:N0MAD|PDE0.DPMP.";
-static const char null_key_v3[] = "CTL:Z3N07|PDE0.DPMP.";
-static const char null_key_v4[] = "CTL:N0MAD|PDE0.DPFP.";
-
-static const struct device_info devices[] =
-{
- {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0e\0n\0 \0V\0i\0s\0i\0o\0n\0:\0M", 42, null_key_v2},
- {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0e\0n\0 \0V\0i\0s\0i\0o\0n\0:\0M\0 \0G\0o\0!", 50, null_key_v2},
- {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0e\0n\0 \0V\0i\0s\0i\0o\0n\0 \0©\0T\0L", 48, null_key_v2},
- {"C\0r\0e\0a\0t\0i\0v\0e\0 \0Z\0E\0N\0 \0V", 42, null_key_v4}
-};
-
int zvm_encode(char *iname, char *oname, int device);
#endif /*CREATIVE_H_*/
diff --git a/tools/iriver.c b/tools/iriver.c
index 4d26b98..240f9e3 100644
--- a/tools/iriver.c
+++ b/tools/iriver.c
@@ -273,8 +273,6 @@
unsigned char * pChecksums, * ppChecksums;
unsigned char ck;
- enum striptype stripmode = STRIP_NONE;
-
infile = openinfile(infile_name);
outfile = openoutfile(outfile_name);
diff --git a/tools/scramble.c b/tools/scramble.c
index 91f46be..7efade3 100644
--- a/tools/scramble.c
+++ b/tools/scramble.c
@@ -41,7 +41,7 @@
ARCHOS_ONDIO_FM
};
-int size_limit[] =
+static unsigned int size_limit[] =
{
0x32000, /* ARCHOS_PLAYER */
0x64000, /* ARCHOS_V2RECORDER */
@@ -132,10 +132,10 @@
unsigned char header[24];
char *iname = argv[1];
char *oname = argv[2];
- char *xorstring;
+ char *xorstring=NULL;
int headerlen = 6;
FILE* file;
- int version;
+ int version=0;
unsigned long modelnum;
char modelname[5];
int model_id;
@@ -351,7 +351,6 @@
}
else if(!strncmp(argv[1], "-mi4", 4)) {
int mi4magic;
- int version;
char model[4] = "";
char type[4] = "";
@@ -402,7 +401,7 @@
if ((method == scramble) &&
((length + headerlen) >= size_limit[model_id])) {
- printf("error: firmware image is %d bytes while max size is %d!\n",
+ printf("error: firmware image is %ld bytes while max size is %u!\n",
length + headerlen,
size_limit[model_id]);
fclose(file);
@@ -462,6 +461,10 @@
outbuf[slen++] = inbuf[i];
}
break;
+ case none:
+ default:
+ /* dummy case just to silence picky compilers */
+ break;
}
if((method == none) || (method == scramble) || (method == xor)) {
@@ -598,7 +601,7 @@
}
len = fread(outbuf+0x1030, 1, length, file);
- if(len < length) {
+ if(len < (size_t) length) {
perror(iname);
return -2;
}
@@ -626,12 +629,13 @@
}
len = fwrite(outbuf, 1, length+0x1030, file);
- if(len < length) {
+ if(len < (size_t)length) {
perror(oname);
return -4;
}
fclose(file);
+ return 0;
}
@@ -697,7 +701,7 @@
}
len = fread(outbuf+0x4600, 1, length, file);
- if(len < length) {
+ if(len < (size_t)length) {
perror(iname);
return -2;
}
@@ -754,7 +758,7 @@
}
len = fwrite(outbuf, 1, length+0x4600, file);
- if(len < length) {
+ if(len < (size_t)length) {
perror(oname);
return -4;
}