diff options
author | Ralf Baechle <ralf@linux-mips.org> | 2015-06-07 23:47:33 +0200 |
---|---|---|
committer | Ralf Baechle <ralf@linux-mips.org> | 2015-06-08 01:31:26 +0200 |
commit | b17dea93bf9a89b652bf155445889f9736ed2798 (patch) | |
tree | f8d50dda57e685073c4cdda92ea2d5311a5a0b9a | |
parent | 868d75cd2bdb0545518c4b651aa3f8b5953954c5 (diff) |
listen: Fix signedness compiler warnings.
make[2]: Entering directory '/home/ralf/src/ax25/ax25-apps/listen'
gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT listen.o -MD -MP -MF .deps/listen.Tpo -c -o listen.o listen.c
listen.c: In function ‘ascii_dump’:
listen.c:231:4: warning: pointer targets in passing argument 2 of ‘__builtin_strncat’ differ in signedness [-Wpointer-sign]
strncat(buf, &c, 1);
^
listen.c:231:4: note: expected ‘const char *’ but argument is of type ‘unsigned char *’
listen.c: In function ‘hex_dump’:
listen.c:282:8: warning: pointer targets in assignment differ in signedness [-Wpointer-sign]
data2 = data;
^
[...]
gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT rosedump.o -MD -MP -MF .deps/rosedump.Tpo -c -o rosedump.o rosedump.c
rosedump.c: In function ‘facility’:
rosedump.c:331:9: warning: pointer targets in assignment differ in signedness [-Wpointer-sign]
factot = data;
^
rosedump.c:398:11: warning: pointer targets in assignment differ in signedness [-Wpointer-sign]
for (d = data, l = 0; l < lgaddcall;
^
rosedump.c:406:28: warning: pointer targets in passing argument 1 of ‘dump_ax25_call’ differ in signedness [-Wpointer-sign]
dump_ax25_call(d, 7));
^
rosedump.c:296:14: note: expected ‘unsigned char *’ but argument is of type ‘char *’
static char *dump_ax25_call(unsigned char *data, int l_data)
^
rosedump.c:410:28: warning: pointer targets in passing argument 1 of ‘dump_ax25_call’ differ in signedness [-Wpointer-sign]
dump_ax25_call(d, 7));
^
rosedump.c:296:14: note: expected ‘unsigned char *’ but argument is of type ‘char *’
static char *dump_ax25_call(unsigned char *data, int l_data)
^
rosedump.c:438:4: warning: pointer targets in passing argument 2 of ‘__builtin_strncpy’ differ in signedness [-Wpointer-sign]
strncpy(indorig, data, lgadind);
^
rosedump.c:438:4: note: expected ‘const char *’ but argument is of type ‘unsigned char *’
rosedump.c:441:4: warning: pointer targets in passing argument 2 of ‘__builtin_strncpy’ differ in signedness [-Wpointer-sign]
strncpy(inddest, data, lgadind);
^
rosedump.c:441:4: note: expected ‘const char *’ but argument is of type ‘unsigned char *’
rosedump.c:451:14: warning: pointer targets in passing argument 1 of ‘data_dump’ differ in signedness [-Wpointer-sign]
data_dump(factot, lgtot, 1);
^
In file included from rosedump.c:11:0:
listen.h:31:6: note: expected ‘unsigned char *’ but argument is of type ‘char *’
void data_dump(unsigned char *, int, int);
^
[...]
gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT utils.o -MD -MP -MF .deps/utils.Tpo -c -o utils.o utils.c
utils.c: In function ‘lprintf’:
utils.c:49:10: warning: pointer targets in assignment differ in signedness [-Wpointer-sign]
for (p = str; *p != '\0'; p++) {
^
utils.c:71:10: warning: pointer targets in assignment differ in signedness [-Wpointer-sign]
for (p = str; *p != '\0'; p++)
^
[...]
make[2]: Leaving directory '/home/ralf/src/ax25/ax25-apps/listen'
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
-rw-r--r-- | call/yapp.c | 10 | ||||
-rw-r--r-- | listen/listen.c | 6 | ||||
-rw-r--r-- | listen/listen.h | 2 | ||||
-rw-r--r-- | listen/rosedump.c | 6 | ||||
-rw-r--r-- | listen/utils.c | 2 |
5 files changed, 13 insertions, 13 deletions
diff --git a/call/yapp.c b/call/yapp.c index 3d7bf45..8260ab0 100644 --- a/call/yapp.c +++ b/call/yapp.c @@ -288,13 +288,13 @@ static int yapp_download_data(int *filefd, unsigned char *buffer) case STATE_RH: if (buffer[0] == SOH) { /* Parse header: 3 fields == YAPP C */ - char *hptr, *hfield[3]; + unsigned char *hptr, *hfield[3]; if ((length = buffer[1]) == 0) length = 256; hptr = buffer + 2; while (length > 0) { int hlen; - hlen = strlen(hptr) + 1; + hlen = strlen((char *)hptr) + 1; hfield[(int) yappc++] = hptr; hptr += hlen; length -= hlen; @@ -303,13 +303,13 @@ static int yapp_download_data(int *filefd, unsigned char *buffer) if (yappc < 3) { yappc = 0; } else { - file_time = yapp2unix(hfield[2]); + file_time = yapp2unix((char *)hfield[2]); yappc = 1; } if (*filefd == -1) { if ((*filefd = - open(hfield[0], + open((char *)hfield[0], O_RDWR | O_APPEND | O_CREAT, 0666)) == -1) { printf("\n[Unable to open %s]\n", @@ -549,7 +549,7 @@ static int yapp_upload_data(int filefd, char *filename, int filelength, len = buffer[1]; if (buffer[len] == 'C') yappc = 1; - rpos = atol(buffer + 4); + rpos = atol((char *)buffer + 4); lseek(filefd, rpos, SEEK_SET); buffer[0] = ACK; buffer[1] = yappc ? ACK : 0x02; diff --git a/listen/listen.c b/listen/listen.c index 11f365e..bf60281 100644 --- a/listen/listen.c +++ b/listen/listen.c @@ -216,7 +216,7 @@ int main(int argc, char **argv) static void ascii_dump(unsigned char *data, int length) { - unsigned char c; + char c; int i, j; char buf[100]; @@ -272,9 +272,9 @@ static void readable_dump(unsigned char *data, int length) static void hex_dump(unsigned char *data, int length) { + unsigned char *data2; int i, j, length2; unsigned char c; - char *data2; char buf[4], hexd[49], ascd[17]; @@ -309,7 +309,7 @@ static void hex_dump(unsigned char *data, int length) } } -void data_dump(unsigned char *data, int length, int dumpstyle) +void data_dump(void *data, int length, int dumpstyle) { switch (dumpstyle) { diff --git a/listen/listen.h b/listen/listen.h index 73a5d06..841a494 100644 --- a/listen/listen.h +++ b/listen/listen.h @@ -28,7 +28,7 @@ int initcolor(void); char *servname(int port, char *proto); /* In listen.c */ -void data_dump(unsigned char *, int, int); +void data_dump(void *, int, int); int get16(unsigned char *); int get32(unsigned char *); diff --git a/listen/rosedump.c b/listen/rosedump.c index c9beaee..e8f8fdc 100644 --- a/listen/rosedump.c +++ b/listen/rosedump.c @@ -324,7 +324,7 @@ static void facility(unsigned char *data, int lgtot) char digis[80], digid[80]; char indorig[10], inddest[10]; char addstorig[20], addstdest[20]; - char *d, *factot; + unsigned char *d, *factot; char buf[512]; char *result = buf; @@ -435,10 +435,10 @@ static void facility(unsigned char *data, int lgtot) lgadind = lgaddcall - (lgad + 1) / 2 - 5; if (fct == 0xCB) { - strncpy(indorig, data, lgadind); + strncpy(indorig, (char *)data, lgadind); indorig[lgadind] = '\0'; } else { - strncpy(inddest, data, lgadind); + strncpy(inddest, (char *)data, lgadind); inddest[lgadind] = '\0'; } diff --git a/listen/utils.c b/listen/utils.c index 8f522e5..667bab7 100644 --- a/listen/utils.c +++ b/listen/utils.c @@ -38,8 +38,8 @@ void lprintf(int dtype, char *fmt, ...) { va_list args; char str[1024]; - unsigned char *p; chtype ch; + char *p; va_start(args, fmt); vsnprintf(str, 1024, fmt, args); |