50dc83
From e44b75fdb86dcf759204816c873b4f9f4efbefa8 Mon Sep 17 00:00:00 2001
50dc83
From: Susant Palai <spalai@redhat.com>
50dc83
Date: Tue, 21 May 2019 16:17:09 +0530
50dc83
Subject: [PATCH 142/169] lock: check null value of dict to avoid log flooding
50dc83
50dc83
> updates: bz#1712322
50dc83
> Change-Id: I120a1d23506f9ebcf88c7ea2f2eff4978a61cf4a
50dc83
> Signed-off-by: Susant Palai <spalai@redhat.com>
50dc83
(backport of fix https://review.gluster.org/#/c/glusterfs/+/22756/)
50dc83
50dc83
BUG: bz#1704181
50dc83
Change-Id: I2a192236328ebb39666ffef1146df312c08a377d
50dc83
Signed-off-by: Susant Palai <spalai@redhat.com>
50dc83
Reviewed-on: https://code.engineering.redhat.com/gerrit/171325
50dc83
Tested-by: RHGS Build Bot <nigelb@redhat.com>
50dc83
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
50dc83
---
50dc83
 xlators/features/locks/src/posix.c | 2 +-
50dc83
 1 file changed, 1 insertion(+), 1 deletion(-)
50dc83
50dc83
diff --git a/xlators/features/locks/src/posix.c b/xlators/features/locks/src/posix.c
50dc83
index 3f1c7a7..adb0df5 100644
50dc83
--- a/xlators/features/locks/src/posix.c
50dc83
+++ b/xlators/features/locks/src/posix.c
50dc83
@@ -121,7 +121,7 @@ fetch_pathinfo(xlator_t *, inode_t *, int32_t *, char **);
50dc83
 
50dc83
 #define PL_CHECK_LOCK_ENFORCE_KEY(frame, dict, name, this, loc, fd, priv)      \
50dc83
     do {                                                                       \
50dc83
-        if (dict_get(dict, GF_ENFORCE_MANDATORY_LOCK) ||                       \
50dc83
+        if ((dict && (dict_get(dict, GF_ENFORCE_MANDATORY_LOCK))) ||           \
50dc83
             (name && (strcmp(name, GF_ENFORCE_MANDATORY_LOCK) == 0))) {        \
50dc83
             inode_t *__inode = (loc ? loc->inode : fd->inode);                 \
50dc83
             pl_inode_t *__pl_inode = pl_inode_get(this, __inode, NULL);        \
50dc83
-- 
50dc83
1.8.3.1
50dc83