diff options
author | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2017-01-15 20:23:35 +0000 |
---|---|---|
committer | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2017-01-15 20:23:35 +0000 |
commit | 0e270dbe709e743ec840999c05e35f4d63be69b4 (patch) | |
tree | 810753d3258d55c35efc599f279d560accb6fe40 /src/libutil | |
parent | e0446369d13269d07cecb5711fbab7d7bc64398a (diff) | |
download | rspamd-0e270dbe709e743ec840999c05e35f4d63be69b4.tar.gz rspamd-0e270dbe709e743ec840999c05e35f4d63be69b4.zip |
[Fix] Fix couple of issues
Found by: Coverity
Diffstat (limited to 'src/libutil')
-rw-r--r-- | src/libutil/addr.c | 2 | ||||
-rw-r--r-- | src/libutil/map.c | 4 | ||||
-rw-r--r-- | src/libutil/sqlite_utils.c | 1 |
3 files changed, 1 insertions, 6 deletions
diff --git a/src/libutil/addr.c b/src/libutil/addr.c index 728189068..c72727c38 100644 --- a/src/libutil/addr.c +++ b/src/libutil/addr.c @@ -1341,7 +1341,7 @@ rspamd_inet_address_apply_mask (rspamd_inet_addr_t *addr, guint mask) p += 3; for (;;) { - if (mask > 32) { + if (mask >= 32) { mask -= 32; *p = 0; } diff --git a/src/libutil/map.c b/src/libutil/map.c index 361abcee4..3794cfcfa 100644 --- a/src/libutil/map.c +++ b/src/libutil/map.c @@ -1465,10 +1465,6 @@ err: g_slice_free1 (sizeof (*hdata), hdata); } - if (fdata) { - g_slice_free1 (sizeof (*fdata), fdata); - } - return NULL; } diff --git a/src/libutil/sqlite_utils.c b/src/libutil/sqlite_utils.c index 9686d221a..7da003fa5 100644 --- a/src/libutil/sqlite_utils.c +++ b/src/libutil/sqlite_utils.c @@ -220,7 +220,6 @@ rspamd_sqlite3_wait (rspamd_mempool_t *pool, const gchar *lock) return FALSE; } if (nanosleep (&sleep_ts, NULL) == -1 && errno != EINTR) { - close (fd); msg_err_pool_check ("cannot sleep open lock file %s: %s", lock, strerror (errno)); |