diff options
| author | Amar Tumballi <amarts@redhat.com> | 2019-05-20 11:11:39 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2019-06-04 10:30:29 +0000 |
| commit | e7aeab3063ac5645136303278b477d7de35266c0 (patch) | |
| tree | 73ab2e1ea4e26b1019318178d114c3d89dc34301 /libglusterfs | |
| parent | abc4d4cfa01d922c46ebb1abfa5d801ba8c6b5c8 (diff) | |
across: clang-scan: fix NULL dereferencing warnings
All these checks are done after analyzing clang-scan report produced
by the CI job @ https://build.gluster.org/job/clang-scan
updates: bz#1622665
Change-Id: I590305af4ceb779be952974b2a36066ffc4865ca
Signed-off-by: Amar Tumballi <amarts@redhat.com>
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/logging.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libglusterfs/src/logging.c b/libglusterfs/src/logging.c index f0fb9a097da..32d98e932e6 100644 --- a/libglusterfs/src/logging.c +++ b/libglusterfs/src/logging.c @@ -1873,7 +1873,7 @@ _gf_msg_internal(const char *domain, const char *file, const char *function, if (size == 0) { flush_logged_msg = _gf_true; goto unlock; - } else if ((ctx->log.lru_cur_size + 1) > size) { + } else if (((ctx->log.lru_cur_size + 1) > size) && (first)) { /* If the list is full, flush the lru msg to disk and also * release it after unlock, and ... * */ |