Compiler warning fixes

Author: ralight

apps/mosquitto_ctrl/ctrl_shell.c

@@ -21,7 +21,6 @@ SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
 #include <arpa/inet.h>
 #include <ctype.h>
 #include <errno.h>
-#include <editline/readline.h>
 #include <fcntl.h>
 #include <pthread.h>
 #include <signal.h>
@@ -131,7 +130,7 @@ static void term_set_canon(bool canon)
 void ctrl_shell_rtrim(char *buf)
 {
 	size_t slen = strlen(buf);
-	while(slen > 0 && isspace(buf[slen-1])){
+	while(slen > 0 && isspace((unsigned char)buf[slen-1])){
 		buf[slen-1] = '\0';
 		slen = strlen(buf);
 	}

apps/mosquitto_ctrl/ctrl_shell.h

@@ -24,6 +24,7 @@ extern "C" {
 
 #include <cjson/cJSON.h>
 #include <stdbool.h>
+#include <stdio.h>
 
 extern const char *ANSI_URL;
 extern const char *ANSI_MODULE;

apps/mosquitto_ctrl/ctrl_shell_dynsec.c

@@ -795,7 +795,7 @@ static void print_details(cJSON *j_data)
 	int64_t groupcount;
 	int64_t rolecount;
 	int64_t changeindex;
-	int align = strlen("Change index: ");
+	int align = (int)strlen("Change index: ");
 	json_get_int64(j_data, "clientCount", &clientcount, true, 0);
 	json_get_int64(j_data, "groupCount", &groupcount, true, 0);
 	json_get_int64(j_data, "roleCount", &rolecount, true, 0);

apps/mosquitto_ctrl/dynsec.c

@@ -184,7 +184,7 @@ static void print_json_array(cJSON *j_list, int slen, const char *label, const c
 static void print_client(cJSON *j_response)
 {
 	cJSON *j_data, *j_client, *jtmp;
-	const int label_width = strlen("Connections:");
+	const int label_width = (int)strlen("Connections:");
 
 	j_data = cJSON_GetObjectItem(j_response, "data");
 	if(j_data == NULL || !cJSON_IsObject(j_data)){
@@ -226,7 +226,7 @@ static void print_client(cJSON *j_response)
 static void print_group(cJSON *j_response)
 {
 	cJSON *j_data, *j_group;
-	int label_width = strlen("Groupname:");
+	int label_width = (int)strlen("Groupname:");
 	const char *groupname;
 
 	j_data = cJSON_GetObjectItem(j_response, "data");

apps/mosquitto_passwd/mosquitto_passwd.c

@@ -413,7 +413,7 @@ static bool is_username_valid(const char *username)
 			return false;
 		}
 		for(i=0; i<slen; i++){
-			if(iscntrl(username[i])){
+			if(iscntrl((unsigned char)username[i])){
 				fprintf(stderr, "Error: Username must not contain control characters.\n");
 				return false;
 			}

lib/packet_mosq.c

@@ -48,7 +48,7 @@ SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
 
 int packet__alloc(struct mosquitto__packet **packet, uint8_t command, uint32_t remaining_length)
 {
-	uint8_t remaining_bytes[5], byte;
+	uint8_t remaining_bytes[5] = {0}, byte;
 	int8_t remaining_count;
 	uint32_t packet_length;
 	uint32_t remaining_length_stored;

libcommon/file_common.c

@@ -262,11 +262,11 @@ char *mosquitto_trimblanks(char *str)
 		return NULL;
 	}
 
-	while(isspace(str[0])){
+	while(isspace((unsigned char)str[0])){
 		str++;
 	}
 	endptr = &str[strlen(str)-1];
-	while(endptr > str && isspace(endptr[0])){
+	while(endptr > str && isspace((unsigned char)endptr[0])){
 		endptr[0] = '\0';
 		endptr--;
 	}

plugins/acl-file/acl_parse.c

@@ -213,7 +213,7 @@ int acl_file__parse(struct acl_file_data *data)
 
 	while(mosquitto_fgets(&buf, &buflen, aclfptr)){
 		slen = strlen(buf);
-		while(slen > 0 && isspace(buf[slen-1])){
+		while(slen > 0 && isspace((unsigned char)buf[slen-1])){
 			buf[slen-1] = '\0';
 			slen = strlen(buf);
 		}

plugins/dynamic-security/config_init.c

@@ -82,7 +82,7 @@ static int get_password_from_init_file(struct dynsec__data *data, char **pw)
 	fclose(fptr);
 
 	pos = (int)strlen(buf)-1;
-	while(pos >= 0 && isspace(buf[pos])){
+	while(pos >= 0 && isspace((unsigned char)buf[pos])){
 		buf[pos] = '\0';
 		pos--;
 	}

src/conf.c

@@ -3003,7 +3003,7 @@ static int config__check(struct mosquitto__config *config)
 		id_prefix_len = config->security_options.auto_id_prefix_len;
 	}else{
 		id_prefix = "auto-";
-		id_prefix_len = strlen("auto-");
+		id_prefix_len = (int)strlen("auto-");
 	}
 
 	/* Default to auto_id_prefix = 'auto-' if none set. */