diff mbox series

[31/47] libsemanage: adjust sizes to avoid implicit truncations

Message ID 20241111141706.38039-31-cgoettsche@seltendoof.de (mailing list archive)
State New
Delegated to: Petr Lautrbach
Headers show
Series [01/47] libsemanage: white space cleanup | expand

Commit Message

Christian Göttsche Nov. 11, 2024, 2:16 p.m. UTC 
From: Christian Göttsche <cgzones@googlemail.com>

Use size_t for sizes and align miscellaneous type mismatches.

Signed-off-by: Christian Göttsche <cgzones@googlemail.com>
---
 libsemanage/src/compressed_file.c |  2 +-
 libsemanage/src/handle.c          |  2 +-
 libsemanage/src/parse_utils.c     |  4 ++--
 libsemanage/src/semanage_store.c  | 12 ++++++------
 libsemanage/src/utilities.c       |  6 +++---
 libsemanage/src/utilities.h       |  2 +-
 6 files changed, 14 insertions(+), 14 deletions(-)
diff mbox series

Patch

diff --git a/libsemanage/src/compressed_file.c b/libsemanage/src/compressed_file.c
index 74185c92..e4a1efb0 100644
--- a/libsemanage/src/compressed_file.c
+++ b/libsemanage/src/compressed_file.c
@@ -104,7 +104,7 @@  static ssize_t bunzip(semanage_handle_t *sh, FILE *f, void **data)
 	size_t   total = 0;
 	uint8_t* uncompress = NULL;
 	uint8_t* tmpalloc = NULL;
-	int      ret = -1;
+	ssize_t  ret = -1;
 
 	buf = malloc(bufsize);
 	if (buf == NULL) {
diff --git a/libsemanage/src/handle.c b/libsemanage/src/handle.c
index 4577ac35..740bc83f 100644
--- a/libsemanage/src/handle.c
+++ b/libsemanage/src/handle.c
@@ -172,7 +172,7 @@  int semanage_get_hll_compiler_path(semanage_handle_t *sh,
 	}
 
 	num_printed = snprintf(compiler, len, "%s/%s", sh->conf->compiler_directory_path, lower_lang_ext);
-	if (num_printed < 0 || (int)num_printed >= (int)len) {
+	if (num_printed < 0 || (size_t)num_printed >= len) {
 		ERR(sh, "Error creating compiler path.");
 		status = -1;
 		goto cleanup;
diff --git a/libsemanage/src/parse_utils.c b/libsemanage/src/parse_utils.c
index d57e59c9..d1d6e930 100644
--- a/libsemanage/src/parse_utils.c
+++ b/libsemanage/src/parse_utils.c
@@ -85,7 +85,7 @@  int parse_skip_space(semanage_handle_t * handle, parse_info_t * info)
 
 	size_t buf_len = 0;
 	ssize_t len;
-	int lineno = info->lineno;
+	unsigned int lineno = info->lineno;
 	char *buffer = NULL;
 	char *ptr;
 
@@ -271,7 +271,7 @@  int parse_fetch_string(semanage_handle_t * handle,
 {
 
 	const char *start = info->ptr;
-	int len = 0;
+	size_t len = 0;
 	char *tmp_str = NULL;
 
 	if (parse_assert_noeof(handle, info) < 0)
diff --git a/libsemanage/src/semanage_store.c b/libsemanage/src/semanage_store.c
index 427952a1..937089b2 100644
--- a/libsemanage/src/semanage_store.c
+++ b/libsemanage/src/semanage_store.c
@@ -137,10 +137,10 @@  typedef struct semanage_file_context_node {
 	char *path;
 	char *file_type;
 	char *context;
-	int path_len;
-	int effective_len;
-	int type_len;
-	int context_len;
+	size_t path_len;
+	size_t effective_len;
+	size_t type_len;
+	size_t context_len;
 	int meta;		/* position of first meta char in path, -1 if none */
 	struct semanage_file_context_node *next;
 } semanage_file_context_node_t;
@@ -514,7 +514,7 @@  const char *semanage_final_path(enum semanage_final_defs store,
 char *semanage_conf_path(void)
 {
 	char *semanage_conf = NULL;
-	int len;
+	size_t len;
 	struct stat sb;
 
 	len = strlen(semanage_root()) + strlen(selinux_path()) + strlen(SEMANAGE_CONF_FILE);
@@ -2895,7 +2895,7 @@  int semanage_nc_sort(semanage_handle_t * sh, const char *buf, size_t buf_len,
 
 	/* parsing bits */
 	const char *priority_names[] = NC_SORT_NAMES;
-	const int priority_names_len[] = NC_SORT_NAMES_LEN;
+	const size_t priority_names_len[] = NC_SORT_NAMES_LEN;
 	size_t line_len, buf_remainder, i, offset;
 	const char *line_buf, *line_end;
 
diff --git a/libsemanage/src/utilities.c b/libsemanage/src/utilities.c
index a64015f8..38ac72e4 100644
--- a/libsemanage/src/utilities.c
+++ b/libsemanage/src/utilities.c
@@ -202,9 +202,9 @@  int semanage_cmp_plist_t(const void *x, const void *y)
 	return strcmp((*l1)->data, (*l2)->data);
 }
 
-int semanage_str_count(const char *data, char what)
+size_t semanage_str_count(const char *data, char what)
 {
-	int count = 0;
+	size_t count = 0;
 
 	if (!data)
 		return 0;
@@ -219,7 +219,7 @@  int semanage_str_count(const char *data, char what)
 
 void semanage_rtrim(char *str, char trim_to)
 {
-	int len = 0;
+	size_t len;
 
 	if (!str)
 		return;
diff --git a/libsemanage/src/utilities.h b/libsemanage/src/utilities.h
index 4cc41f84..c2d484a7 100644
--- a/libsemanage/src/utilities.h
+++ b/libsemanage/src/utilities.h
@@ -104,7 +104,7 @@  int semanage_cmp_plist_t(const void *x, const void *y);
  * @param      what  a character
  * @returns    the number of times the char appears in the string
  */
-int semanage_str_count(const char *data, char what);
+size_t semanage_str_count(const char *data, char what);
 /**
  * @param      - a string
  * @param            the character to trim to