andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

features/locks: NULL pointer deferencing clang fix

Browse Source 
Added checks to avoid NULL pointer dereferencing

Updates: bz#1622665

Change-Id: I8e441c2931f406a6012e418127550bdf454a599a
Signed-off-by: Iraj Jamali <ijamali@redhat.com>
This commit is contained in:
Iraj Jamali 2018-09-24 15:12:25 +05:30 committed by Amar Tumballi
parent 65aed1070c
commit 5ad31b5b17
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
xlators/features/locks/src/posix.c
View File

@ -850,7 +850,7 @@ truncate_stat_cbk(call_frame_t *frame, void *cookie, xlator_t *this,
}
unwind:
if (op_ret == -1) {
gf_log(this->name, GF_LOG_ERROR,
gf_log(this ? this->name : "locks", GF_LOG_ERROR,
"truncate failed with "
"ret: %d, error: %s",
op_ret, strerror(op_errno));
@ -902,7 +902,7 @@ pl_truncate(call_frame_t *frame, xlator_t *this, loc_t *loc, off_t offset,
ret = 0;
unwind:
if (ret == -1) {
gf_log(this->name, GF_LOG_ERROR,
gf_log(this ? this->name : "locks", GF_LOG_ERROR,
"truncate on %s failed with"
" ret: %d, error: %s",
loc->path, -1, strerror(ENOMEM));
@ -935,7 +935,7 @@ pl_ftruncate(call_frame_t *frame, xlator_t *this, fd_t *fd, off_t offset,
ret = 0;
unwind:
if (ret == -1) {
gf_log(this->name, GF_LOG_ERROR,
gf_log(this ? this->name : "locks", GF_LOG_ERROR,
"ftruncate failed with"
" ret: %d, error: %s",
-1, strerror(ENOMEM));