diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-01 08:46:20 +0200 |
---|---|---|
committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-01 14:34:12 +0200 |
commit | ab5c1f5d8c2855e56198676cef2b5fd418d96d42 (patch) | |
tree | 6358ac90d43bdc919c6a751be3a7333b9bc5b176 /src/confdb | |
parent | e27c0ecef59272d4723f904a17061b4e59f0a6db (diff) | |
download | sssd-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.c | 5 |
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; |