diff options
| author | Raghavendra Bhat <raghavendra@redhat.com> | 2012-11-14 12:50:01 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2012-11-19 03:41:09 -0500 |
| commit | ab4fed1445d35230b0ae6467a7d1fc7fdf1dbebc (patch) | |
| tree | 8d1f8bde2146db9cace2dd6eebc856908e661bbc /xlators/protocol/server/src/server.c | |
| parent | 49720aef1da73ec460c11be871d7e0cdd8f535d4 (diff) | |
protocol/server: check bound_xl for NULL before dumping ltable
* If bound_xl in the setvolume is NULL, then put the connection
back otherwise it might result in segfault when bound_xl from
that connection object is accessed.
Change-Id: I2805e8b45e5767121f323b0f6fed4084dfc020c0
BUG: 874913
Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-on: https://code.engineering.redhat.com/gerrit/1544
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Tested-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/protocol/server/src/server.c')
| -rw-r--r-- | xlators/protocol/server/src/server.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/protocol/server/src/server.c b/xlators/protocol/server/src/server.c index 4f6527fa46e..5e20bf34ea3 100644 --- a/xlators/protocol/server/src/server.c +++ b/xlators/protocol/server/src/server.c @@ -307,7 +307,7 @@ ltable_dump (server_connection_t *trav) gf_proc_dump_build_key(key, "conn","bound_xl.ltable.inodelk.%s", - trav->bound_xl->name); + trav->bound_xl?trav->bound_xl->name:""); gf_proc_dump_add_section(key); list_for_each_entry (locker, &trav->ltable->inodelk_lockers, lockers) { @@ -341,7 +341,7 @@ ltable_dump (server_connection_t *trav) locker = NULL; gf_proc_dump_build_key(key, "conn","bound_xl.ltable.entrylk.%s", - trav->bound_xl->name); + trav->bound_xl?trav->bound_xl->name:""); gf_proc_dump_add_section(key); list_for_each_entry (locker, &trav->ltable->entrylk_lockers, |