diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2015-12-24 14:29:33 +0530 |
---|---|---|
committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2016-03-07 00:31:38 -0800 |
commit | 6fbaa124e33e6604621826c54eac987dfe590d27 (patch) | |
tree | 8cf358c70bccad56bfa884490e7c01a601dab1ca | |
parent | 7d94b3a9333c3f85a67e5d6d09baa12f1cf54e85 (diff) |
performance/open-behind: Fix fdctx dump NULL dereference
Also printing flags correctly in statedump now
>Change-Id: Ibfdd74aab5643ecc47d0a88f109d5d1050685f5a
>BUG: 1294051
>Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
>Reviewed-on: http://review.gluster.org/13076
>Smoke: Gluster Build System <jenkins@build.gluster.com>
>NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
>CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
>Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
BUG: 1315140
Change-Id: Ib11306831b930cddaa2c6e293be5d00dff41e880
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/13615
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Tested-by: Raghavendra G <rgowdapp@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
-rw-r--r-- | xlators/performance/open-behind/src/open-behind.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/xlators/performance/open-behind/src/open-behind.c b/xlators/performance/open-behind/src/open-behind.c index 97c12fab411..efab88582ff 100644 --- a/xlators/performance/open-behind/src/open-behind.c +++ b/xlators/performance/open-behind/src/open-behind.c @@ -865,14 +865,15 @@ ob_fdctx_dump (xlator_t *this, fd_t *fd) gf_proc_dump_write ("open_frame", "%p", ob_fd->open_frame); - gf_proc_dump_write ("open_frame.root.unique", "%p", - ob_fd->open_frame->root->unique); + if (ob_fd->open_frame) + gf_proc_dump_write ("open_frame.root.unique", "%p", + ob_fd->open_frame->root->unique); gf_proc_dump_write ("loc.path", "%s", ob_fd->loc.path); gf_proc_dump_write ("loc.ino", "%s", uuid_utoa (ob_fd->loc.gfid)); - gf_proc_dump_write ("flags", "%p", ob_fd->open_frame); + gf_proc_dump_write ("flags", "%d", ob_fd->flags); UNLOCK (&fd->lock); |