summaryrefslogtreecommitdiffstats
path: root/src/confdb
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2016-07-01 08:46:20 +0200
committerLukas Slebodnik <lslebodn@redhat.com>2016-07-01 14:34:12 +0200
commitab5c1f5d8c2855e56198676cef2b5fd418d96d42 (patch)
tree6358ac90d43bdc919c6a751be3a7333b9bc5b176 /src/confdb
parente27c0ecef59272d4723f904a17061b4e59f0a6db (diff)
downloadsssd-ab5c1f5d8c2855e56198676cef2b5fd418d96d42.tar.gz
sssd-ab5c1f5d8c2855e56198676cef2b5fd418d96d42.tar.xz
sssd-ab5c1f5d8c2855e56198676cef2b5fd418d96d42.zip
Fix warning sign-compare
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; i<res->count; 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 <mzidek@redhat.com>
Diffstat (limited to 'src/confdb')
-rw-r--r--src/confdb/confdb_setup.c5
1 files changed, 3 insertions, 2 deletions
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;