can-utils: fix sign-compare warnings (#513)

hartkopp · web-flow · commit 8d7d7650d41f · 2024-04-24T20:02:02.000+02:00
Fixing several build issues reported by Gary Bisson when he was building can-utils with clang (AOSP14). URL: #512 Reported-by: Gary Bisson (https://github.com/gibsson) Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
diff --git a/Makefile b/Makefile
@@ -43,7 +43,7 @@ PREFIX ?= /usr/local
 
 MAKEFLAGS := -k
 
-CFLAGS := -O2 -Wall -Wno-parentheses
+CFLAGS := -O2 -Wall -Wno-parentheses -Wsign-compare
 
 HAVE_FORK := $(shell ./check_cc.sh "$(CC)" fork_test.c)
 
diff --git a/candump.c b/candump.c
@@ -755,13 +755,13 @@ int main(int argc, char **argv)
 			}
 
 			/* mark dual-use struct canfd_frame */
-			if (nbytes < CANXL_HDR_SIZE + CANXL_MIN_DLEN) {
+			if (nbytes < (int)CANXL_HDR_SIZE + CANXL_MIN_DLEN) {
 				fprintf(stderr, "read: no CAN frame\n");
 				return 1;
 			}
 
 			if (cu.xl.flags & CANXL_XLF) {
-				if (nbytes != CANXL_HDR_SIZE + cu.xl.len) {
+				if (nbytes != (int)CANXL_HDR_SIZE + cu.xl.len) {
 					printf("nbytes = %d\n", nbytes);
 					fprintf(stderr, "read: no CAN XL frame\n");
 					return 1;
diff --git a/canfdtest.c b/canfdtest.c
@@ -215,11 +215,12 @@ static int recv_frame(struct canfd_frame *frame, int *flags)
 	ssize_t ret;
 
 	ret = recvmsg(sockfd, &msg, 0);
-	if (ret != iov.iov_len) {
-		if (ret < 0)
-			perror("recvmsg() failed");
-		else
-			fprintf(stderr, "recvmsg() returned %zd", ret);
+	if (ret < 0) {
+		perror("recvmsg() failed");
+		return -1;
+	}
+	if ((size_t)ret != iov.iov_len) {
+		fprintf(stderr, "recvmsg() returned %zd", ret);
 		return -1;
 	}
 
diff --git a/cangen.c b/cangen.c
@@ -774,17 +774,17 @@ int main(int argc, char **argv)
 				cu.fd.len = CANFD_MAX_DLEN;
 		}
 
-		if (canxl && (ifr.ifr_mtu < CANXL_MIN_MTU)) {
+		if (canxl && (ifr.ifr_mtu < (int)CANXL_MIN_MTU)) {
 			printf("CAN interface not CAN XL capable - sorry.\n");
 			return 1;
 		}
 
-		if (canfd && (ifr.ifr_mtu < CANFD_MTU)) {
+		if (canfd && (ifr.ifr_mtu < (int)CANFD_MTU)) {
 			printf("CAN interface not CAN FD capable - sorry.\n");
 			return 1;
 		}
 
-		if (ifr.ifr_mtu == CANFD_MTU) {
+		if (ifr.ifr_mtu == (int)CANFD_MTU) {
 			/* interface is ok - try to switch the socket into CAN FD mode */
 			if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_FD_FRAMES,
 				       &enable_canfx, sizeof(enable_canfx))){
@@ -793,7 +793,7 @@ int main(int argc, char **argv)
 			}
 		}
 
-		if (ifr.ifr_mtu >= CANXL_MIN_MTU) {
+		if (ifr.ifr_mtu >= (int)CANXL_MIN_MTU) {
 			/* interface is ok - try to switch the socket into CAN XL mode */
 			if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_XL_FRAMES,
 				       &enable_canfx, sizeof(enable_canfx))){
@@ -1070,7 +1070,7 @@ int main(int argc, char **argv)
 				esi = i & 8;
 			}
 			/* generate CAN XL traffic if the interface is capable */
-			if (ifr.ifr_mtu >= CANXL_MIN_MTU)
+			if (ifr.ifr_mtu >= (int)CANXL_MIN_MTU)
 				canxl = ((i & 96) == 96);
 
 			rtr_frame = ((i & 24) == 24); /* reduce RTR frames to 1/4 */
diff --git a/canlogserver.c b/canlogserver.c
@@ -407,13 +407,13 @@ int main(int argc, char **argv)
 					return 1;
 				}
 
-				if (nbytes < CANXL_HDR_SIZE + CANXL_MIN_DLEN) {
+				if (nbytes < (int)CANXL_HDR_SIZE + CANXL_MIN_DLEN) {
 					fprintf(stderr, "read: no CAN frame\n");
 					return 1;
 				}
 
 				if (cu.xl.flags & CANXL_XLF) {
-					if (nbytes != CANXL_HDR_SIZE + cu.xl.len) {
+					if (nbytes != (int)CANXL_HDR_SIZE + cu.xl.len) {
 						printf("nbytes = %d\n", nbytes);
 						fprintf(stderr, "read: no CAN XL frame\n");
 						return 1;
diff --git a/cansend.c b/cansend.c
@@ -156,7 +156,7 @@ int main(int argc, char **argv)
 		}
 		mtu = ifr.ifr_mtu;
 
-		if (mtu == CANFD_MTU) {
+		if (mtu == (int)CANFD_MTU) {
 			/* interface is ok - try to switch the socket into CAN FD mode */
 			if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_FD_FRAMES,
 				       &enable_canfx, sizeof(enable_canfx))){
@@ -165,7 +165,7 @@ int main(int argc, char **argv)
 			}
 		}
 
-		if (mtu >= CANXL_MIN_MTU) {
+		if (mtu >= (int)CANXL_MIN_MTU) {
 			/* interface is ok - try to switch the socket into CAN XL mode */
 			if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_XL_FRAMES,
 				       &enable_canfx, sizeof(enable_canfx))){
diff --git a/isobusfs/isobusfs_cli.c b/isobusfs/isobusfs_cli.c
@@ -268,10 +268,10 @@ static int isobusfs_cli_rx_one(struct isobusfs_priv *priv, int sock)
 	return 0;
 }
 
-static int isobusfs_cli_handle_events(struct isobusfs_priv *priv, int nfds)
+static int isobusfs_cli_handle_events(struct isobusfs_priv *priv, unsigned int nfds)
 {
 	int ret;
-	int n;
+	unsigned int n;
 
 	for (n = 0; n < nfds && n < priv->cmn.epoll_events_size; ++n) {
 		struct epoll_event *ev = &priv->cmn.epoll_events[n];
@@ -341,7 +341,7 @@ int isobusfs_cli_process_events_and_tasks(struct isobusfs_priv *priv)
 		return ret;
 
 	if (nfds > 0) {
-		ret = isobusfs_cli_handle_events(priv, nfds);
+		ret = isobusfs_cli_handle_events(priv, (unsigned int)nfds);
 		if (ret)
 			return ret;
 	}
diff --git a/isobusfs/isobusfs_cmn.c b/isobusfs/isobusfs_cmn.c
@@ -823,7 +823,7 @@ void isobusfs_cmn_dump_last_x_bytes(const uint8_t *buffer, size_t buffer_size,
 	size_t start_offset = 0;
 	char *output_ptr;
 	unsigned char c;
-	size_t remaining;
+	int remaining;
 	char output[80];
 	int n, j;
 
@@ -832,7 +832,7 @@ void isobusfs_cmn_dump_last_x_bytes(const uint8_t *buffer, size_t buffer_size,
 
 	for (size_t i = start_offset; i < buffer_size; i += 8) {
 		output_ptr = output;
-		remaining = sizeof(output);
+		remaining = (int)sizeof(output);
 
 		n = snprintf(output_ptr, remaining, "%08lx: ",
 			     (unsigned long)(start_offset + i));
diff --git a/isobusfs/isobusfs_srv.c b/isobusfs/isobusfs_srv.c
@@ -201,10 +201,10 @@ static int isobusfs_srv_recv_one(struct isobusfs_srv_priv *priv, int sock)
 	return EXIT_SUCCESS;
 }
 
-static int isobusfs_srv_handle_events(struct isobusfs_srv_priv *priv, int nfds)
+static int isobusfs_srv_handle_events(struct isobusfs_srv_priv *priv, unsigned int nfds)
 {
 	int ret;
-	int n;
+	unsigned int n;
 
 	for (n = 0; n < nfds && n < priv->cmn.epoll_events_size; ++n) {
 		struct epoll_event *ev = &priv->cmn.epoll_events[n];
diff --git a/isobusfs/isobusfs_srv_cm.c b/isobusfs/isobusfs_srv_cm.c
@@ -36,7 +36,7 @@ int isobusfs_srv_request_volume(struct isobusfs_srv_priv *priv,
 				struct isobusfs_srv_client *client,
 				struct isobusfs_srv_volume *volume)
 {
-	int j;
+	unsigned int j;
 
 	/* Check if the client already requested this volume */
 	for (j = 0; j < ARRAY_SIZE(volume->clients); j++) {
diff --git a/isobusfs/isobusfs_srv_dh.c b/isobusfs/isobusfs_srv_dh.c
@@ -110,7 +110,7 @@ int isobusfs_path_to_linux_path(struct isobusfs_srv_priv *priv,
 
 		ptr++;
 		vol_end++;
-		if (ptr - linux_path >= linux_path_size) {
+		if (ptr - linux_path >= (long int)linux_path_size) {
 			/* Ensure null termination */
 			linux_path[linux_path_size - 1] = '\0';
 			break;
diff --git a/isobusfs/isobusfs_srv_fa.c b/isobusfs/isobusfs_srv_fa.c
@@ -16,7 +16,7 @@
 static struct isobusfs_srv_handles *
 isobusfs_srv_walk_handles(struct isobusfs_srv_priv *priv, const char *path)
 {
-	int i;
+	unsigned int i;
 
 	for (i = 0; i < ARRAY_SIZE(priv->handles); i++) {
 		if (priv->handles[i].path == NULL)
@@ -32,9 +32,9 @@ isobusfs_srv_walk_handles(struct isobusfs_srv_priv *priv, const char *path)
 static int isobusfs_srv_add_file(struct isobusfs_srv_priv *priv,
 				 const char *path, int fd, DIR *dir)
 {
-	int j;
+	unsigned int j;
 
-	if (priv->handles_count >= ARRAY_SIZE(priv->handles)) {
+	if (priv->handles_count >= (int)ARRAY_SIZE(priv->handles)) {
 		pr_err("too many handles");
 		return -ENOSPC;
 	}
@@ -55,7 +55,7 @@ static int isobusfs_srv_add_file(struct isobusfs_srv_priv *priv,
 static int isobusfs_srv_add_client_to_file(struct isobusfs_srv_handles *file,
 					   struct isobusfs_srv_client *client)
 {
-	int j;
+	unsigned int j;
 
 	for (j = 0; j < ARRAY_SIZE(file->clients); j++) {
 		if (file->clients[j] == client)
@@ -102,7 +102,7 @@ static int isobusfs_srv_request_file(struct isobusfs_srv_priv *priv,
 static struct isobusfs_srv_handles *
 isobusfs_srv_get_handle(struct isobusfs_srv_priv *priv, int handle)
 {
-	if (handle < 0 || handle >= ARRAY_SIZE(priv->handles))
+	if (handle < 0 || handle >= (int)ARRAY_SIZE(priv->handles))
 		return NULL;
 
 	return &priv->handles[handle];
@@ -113,7 +113,7 @@ static int isobusfs_srv_release_handle(struct isobusfs_srv_priv *priv,
 				       int handle)
 {
 	struct isobusfs_srv_handles *hdl = isobusfs_srv_get_handle(priv, handle);
-	int client_index;
+	unsigned int client_index;
 
 	if (!hdl) {
 		pr_warn("%s: invalid handle %d", __func__, handle);
@@ -152,8 +152,8 @@ static int isobusfs_srv_release_handle(struct isobusfs_srv_priv *priv,
 void isobusfs_srv_remove_client_from_handles(struct isobusfs_srv_priv *priv,
 					   struct isobusfs_srv_client *client)
 {
-	int handle;
-	int client_index;
+	unsigned int handle;
+	unsigned int client_index;
 
 	for (handle = 0; handle < ARRAY_SIZE(priv->handles); handle++) {
 		struct isobusfs_srv_handles *hdl = &priv->handles[handle];
@@ -488,7 +488,7 @@ static int check_access_with_base(const char *base_dir,
 	char full_path[ISOBUSFS_SRV_MAX_PATH_LEN];
 
 	if (snprintf(full_path, sizeof(full_path), "%s/%s", base_dir,
-		 relative_path) >= sizeof(full_path)) {
+		     relative_path) >= (int)sizeof(full_path)) {
 		return -ENAMETOOLONG;
 	}
 
@@ -519,7 +519,7 @@ static int isobusfs_srv_read_directory(struct isobusfs_srv_handles *handle,
 	 *   either returning an error or restarting from the beginning of the directory, depending
 	 *   on the application's requirements.
 	 */
-	for (size_t i = 0; i < handle->dir_pos &&
+	for (int i = 0; i < handle->dir_pos &&
 	     (entry = readdir(dir)) != NULL; i++) {
 		/* Iterating to the desired position */
 	}
diff --git a/lib.c b/lib.c
@@ -437,7 +437,8 @@ int snprintf_long_canframe(char *buf, size_t size, cu_t *cu, int view)
 	/* documentation see lib.h */
 
 	unsigned char is_canfd = cu->fd.flags;
-	int i, j, dlen, offset, maxsize;
+	int i, j, dlen, offset;
+	size_t maxsize;
 	int len;
 
 	/* ensure space for string termination */
diff --git a/slcanpty.c b/slcanpty.c
@@ -53,13 +53,13 @@
 static int pty2can(int pty, int socket, struct can_filter *fi,
 		   int *is_open, int *tstamp)
 {
-	unsigned int nbytes;
+	unsigned int nbytes, tmp;
 	char cmd;
 	static char buf[200];
 	char replybuf[10]; /* for answers to received commands */
-	int ptr;
+	unsigned int ptr;
 	struct can_frame frame;
-	int ret, tmp, i;
+	int ret, i;
 	static unsigned int rxoffset = 0; /* points to the end of an received incomplete SLCAN message */
 
 	ret = read(pty, &buf[rxoffset], sizeof(buf) - rxoffset - 1);
@@ -282,8 +282,8 @@ static int pty2can(int pty, int socket, struct can_filter *fi,
 			ptr--;
 	}
 
-	tmp = write(socket, &frame, sizeof(frame));
-	if (tmp != sizeof(frame)) {
+	ret = write(socket, &frame, sizeof(frame));
+	if (ret != sizeof(frame)) {
 		perror("write socket");
 		return 1;
 	}
@@ -296,8 +296,8 @@ static int pty2can(int pty, int socket, struct can_filter *fi,
 	replybuf[0] = '\a';
 	tmp = 1;
 rx_out:
-	tmp = write(pty, replybuf, tmp);
-	if (tmp < 0) {
+	ret = write(pty, replybuf, tmp);
+	if (ret < 0) {
 		perror("write pty replybuf");
 		return 1;
 	}

Original file line number	Diff line number	Diff line change
`@@ -755,13 +755,13 @@ int main(int argc, char **argv)`
`755`	`755`	`}`
`756`	`756`
`757`	`757`	`/* mark dual-use struct canfd_frame */`
`758`		`- if (nbytes < CANXL_HDR_SIZE + CANXL_MIN_DLEN) {`
	`758`	`+ if (nbytes < (int)CANXL_HDR_SIZE + CANXL_MIN_DLEN) {`
`759`	`759`	`fprintf(stderr, "read: no CAN frame\n");`
`760`	`760`	`return 1;`
`761`	`761`	`}`
`762`	`762`
`763`	`763`	`if (cu.xl.flags & CANXL_XLF) {`
`764`		`- if (nbytes != CANXL_HDR_SIZE + cu.xl.len) {`
	`764`	`+ if (nbytes != (int)CANXL_HDR_SIZE + cu.xl.len) {`
`765`	`765`	`printf("nbytes = %d\n", nbytes);`
`766`	`766`	`fprintf(stderr, "read: no CAN XL frame\n");`
`767`	`767`	`return 1;`
Original file line number	Diff line number	Diff line change
`@@ -774,17 +774,17 @@ int main(int argc, char **argv)`
`774`	`774`	`cu.fd.len = CANFD_MAX_DLEN;`
`775`	`775`	`}`
`776`	`776`
`777`		`- if (canxl && (ifr.ifr_mtu < CANXL_MIN_MTU)) {`
	`777`	`+ if (canxl && (ifr.ifr_mtu < (int)CANXL_MIN_MTU)) {`
`778`	`778`	`printf("CAN interface not CAN XL capable - sorry.\n");`
`779`	`779`	`return 1;`
`780`	`780`	`}`
`781`	`781`
`782`		`- if (canfd && (ifr.ifr_mtu < CANFD_MTU)) {`
	`782`	`+ if (canfd && (ifr.ifr_mtu < (int)CANFD_MTU)) {`
`783`	`783`	`printf("CAN interface not CAN FD capable - sorry.\n");`
`784`	`784`	`return 1;`
`785`	`785`	`}`
`786`	`786`
`787`		`- if (ifr.ifr_mtu == CANFD_MTU) {`
	`787`	`+ if (ifr.ifr_mtu == (int)CANFD_MTU) {`
`788`	`788`	`/* interface is ok - try to switch the socket into CAN FD mode */`
`789`	`789`	`if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_FD_FRAMES,`
`790`	`790`	`&enable_canfx, sizeof(enable_canfx))){`
`@@ -793,7 +793,7 @@ int main(int argc, char **argv)`
`793`	`793`	`}`
`794`	`794`	`}`
`795`	`795`
`796`		`- if (ifr.ifr_mtu >= CANXL_MIN_MTU) {`
	`796`	`+ if (ifr.ifr_mtu >= (int)CANXL_MIN_MTU) {`
`797`	`797`	`/* interface is ok - try to switch the socket into CAN XL mode */`
`798`	`798`	`if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_XL_FRAMES,`
`799`	`799`	`&enable_canfx, sizeof(enable_canfx))){`
`@@ -1070,7 +1070,7 @@ int main(int argc, char **argv)`
`1070`	`1070`	`esi = i & 8;`
`1071`	`1071`	`}`
`1072`	`1072`	`/* generate CAN XL traffic if the interface is capable */`
`1073`		`- if (ifr.ifr_mtu >= CANXL_MIN_MTU)`
	`1073`	`+ if (ifr.ifr_mtu >= (int)CANXL_MIN_MTU)`
`1074`	`1074`	`canxl = ((i & 96) == 96);`
`1075`	`1075`
`1076`	`1076`	`rtr_frame = ((i & 24) == 24); /* reduce RTR frames to 1/4 */`
Original file line number	Diff line number	Diff line change
`@@ -407,13 +407,13 @@ int main(int argc, char **argv)`
`407`	`407`	`return 1;`
`408`	`408`	`}`
`409`	`409`
`410`		`- if (nbytes < CANXL_HDR_SIZE + CANXL_MIN_DLEN) {`
	`410`	`+ if (nbytes < (int)CANXL_HDR_SIZE + CANXL_MIN_DLEN) {`
`411`	`411`	`fprintf(stderr, "read: no CAN frame\n");`
`412`	`412`	`return 1;`
`413`	`413`	`}`
`414`	`414`
`415`	`415`	`if (cu.xl.flags & CANXL_XLF) {`
`416`		`- if (nbytes != CANXL_HDR_SIZE + cu.xl.len) {`
	`416`	`+ if (nbytes != (int)CANXL_HDR_SIZE + cu.xl.len) {`
`417`	`417`	`printf("nbytes = %d\n", nbytes);`
`418`	`418`	`fprintf(stderr, "read: no CAN XL frame\n");`
`419`	`419`	`return 1;`
Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ int main(int argc, char **argv)`
`156`	`156`	`}`
`157`	`157`	`mtu = ifr.ifr_mtu;`
`158`	`158`
`159`		`- if (mtu == CANFD_MTU) {`
	`159`	`+ if (mtu == (int)CANFD_MTU) {`
`160`	`160`	`/* interface is ok - try to switch the socket into CAN FD mode */`
`161`	`161`	`if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_FD_FRAMES,`
`162`	`162`	`&enable_canfx, sizeof(enable_canfx))){`
`@@ -165,7 +165,7 @@ int main(int argc, char **argv)`
`165`	`165`	`}`
`166`	`166`	`}`
`167`	`167`
`168`		`- if (mtu >= CANXL_MIN_MTU) {`
	`168`	`+ if (mtu >= (int)CANXL_MIN_MTU) {`
`169`	`169`	`/* interface is ok - try to switch the socket into CAN XL mode */`
`170`	`170`	`if (setsockopt(s, SOL_CAN_RAW, CAN_RAW_XL_FRAMES,`
`171`	`171`	`&enable_canfx, sizeof(enable_canfx))){`
Original file line number	Diff line number	Diff line change
`@@ -268,10 +268,10 @@ static int isobusfs_cli_rx_one(struct isobusfs_priv *priv, int sock)`
`268`	`268`	`return 0;`
`269`	`269`	`}`
`270`	`270`
`271`		`-static int isobusfs_cli_handle_events(struct isobusfs_priv *priv, int nfds)`
	`271`	`+static int isobusfs_cli_handle_events(struct isobusfs_priv *priv, unsigned int nfds)`
`272`	`272`	`{`
`273`	`273`	`int ret;`
`274`		`- int n;`
	`274`	`+ unsigned int n;`
`275`	`275`
`276`	`276`	`for (n = 0; n < nfds && n < priv->cmn.epoll_events_size; ++n) {`
`277`	`277`	`struct epoll_event *ev = &priv->cmn.epoll_events[n];`
`@@ -341,7 +341,7 @@ int isobusfs_cli_process_events_and_tasks(struct isobusfs_priv *priv)`
`341`	`341`	`return ret;`
`342`	`342`
`343`	`343`	`if (nfds > 0) {`
`344`		`- ret = isobusfs_cli_handle_events(priv, nfds);`
	`344`	`+ ret = isobusfs_cli_handle_events(priv, (unsigned int)nfds);`
`345`	`345`	`if (ret)`
`346`	`346`	`return ret;`
`347`	`347`	`}`
Original file line number	Diff line number	Diff line change
`@@ -201,10 +201,10 @@ static int isobusfs_srv_recv_one(struct isobusfs_srv_priv *priv, int sock)`
`201`	`201`	`return EXIT_SUCCESS;`
`202`	`202`	`}`
`203`	`203`
`204`		`-static int isobusfs_srv_handle_events(struct isobusfs_srv_priv *priv, int nfds)`
	`204`	`+static int isobusfs_srv_handle_events(struct isobusfs_srv_priv *priv, unsigned int nfds)`
`205`	`205`	`{`
`206`	`206`	`int ret;`
`207`		`- int n;`
	`207`	`+ unsigned int n;`
`208`	`208`
`209`	`209`	`for (n = 0; n < nfds && n < priv->cmn.epoll_events_size; ++n) {`
`210`	`210`	`struct epoll_event *ev = &priv->cmn.epoll_events[n];`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ int isobusfs_srv_request_volume(struct isobusfs_srv_priv *priv,`
`36`	`36`	`struct isobusfs_srv_client *client,`
`37`	`37`	`struct isobusfs_srv_volume *volume)`
`38`	`38`	`{`
`39`		`- int j;`
	`39`	`+ unsigned int j;`
`40`	`40`
`41`	`41`	`/* Check if the client already requested this volume */`
`42`	`42`	`for (j = 0; j < ARRAY_SIZE(volume->clients); j++) {`