From ab5c1f5d8c2855e56198676cef2b5fd418d96d42 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Fri, 1 Jul 2016 08:46:20 +0200 Subject: Fix warning sign-compare MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit python3.5 CFLAGS contains warning Wsign-compare and file confdb_setup.c was added to SSSD_TOOLS_OBJ which is required for python3-sss (pysss.so) src/confdb/confdb_setup.c: In function 'confdb_purge': src/confdb/confdb_setup.c:95:15: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for(i=0; icount; i++) { ^ src/confdb/confdb_setup.c: In function 'confdb_init_db': src/confdb/confdb_setup.c:219:25: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (ret <= 0 || ret >= sizeof(timestr)) { ^ cc1: all warnings being treated as errors Reviewed-by: Michal Židek --- src/confdb/confdb_setup.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src/confdb') diff --git a/src/confdb/confdb_setup.c b/src/confdb/confdb_setup.c index 09b6fee0d..d6feab900 100644 --- a/src/confdb/confdb_setup.c +++ b/src/confdb/confdb_setup.c @@ -74,7 +74,8 @@ static int confdb_test(struct confdb_ctx *cdb) static int confdb_purge(struct confdb_ctx *cdb) { - int ret, i; + int ret; + unsigned int i; TALLOC_CTX *tmp_ctx; struct ldb_result *res; struct ldb_dn *dn; @@ -216,7 +217,7 @@ static int confdb_init_db(const char *config_file, const char *config_dir, errno = 0; ret = sss_ini_get_mtime(init_data, sizeof(timestr), timestr); - if (ret <= 0 || ret >= sizeof(timestr)) { + if (ret <= 0 || ret >= (int)sizeof(timestr)) { DEBUG(SSSDBG_FATAL_FAILURE, "Failed to convert time_t to string ??\n"); ret = errno ? errno : EFAULT; -- cgit