diff options
| author | Pranith Kumar K <pkarampu@redhat.com> | 2015-04-26 17:59:49 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2015-04-27 05:04:15 -0700 |
| commit | 472d5c67013913ca8646f32ece214a767a955ef9 (patch) | |
| tree | 6dca08356ed36c6a876b5155f310ac54e1e67d87 /xlators | |
| parent | 5dd40bd4ad768c4d2eb07520efeb49467c709df6 (diff) | |
storage/posix: prevent NULL dereference
filler->fd is never set but used.
Change-Id: Icf21c439b37c9faa3751658a9e63a74570ed153c
BUG: 1215265
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/10383
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Krutika Dhananjay <kdhananj@redhat.com>
Tested-by: NetBSD Build System
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators')
| -rw-r--r-- | xlators/storage/posix/src/posix.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c index 8cbe2c5a02e..2a29f167250 100644 --- a/xlators/storage/posix/src/posix.c +++ b/xlators/storage/posix/src/posix.c @@ -4683,7 +4683,7 @@ _posix_handle_xattr_keyvalue_pair (dict_t *d, char *k, data_t *v, "fgetxattr failed on gfid=%s " "while doing xattrop: " "Key:%s (%s)", - uuid_utoa (filler->fd->inode->gfid), + uuid_utoa (filler->inode->gfid), k, strerror (op_errno)); } @@ -4738,7 +4738,7 @@ unlock: gf_log (this->name, GF_LOG_ERROR, "fsetxattr failed on gfid=%s while doing xattrop: " "key=%s (%s)", - uuid_utoa (filler->fd->inode->gfid), + uuid_utoa (filler->inode->gfid), k, strerror (op_errno)); op_ret = -1; @@ -4756,7 +4756,7 @@ unlock: gf_log (this->name, GF_LOG_DEBUG, "dict_set_bin failed (gfid=%s): " "key=%s (%s)", - uuid_utoa (filler->fd->inode->gfid), + uuid_utoa (filler->inode->gfid), k, strerror (-size)); op_ret = -1; |