diff options
| author | Krutika Dhananjay <kdhananj@redhat.com> | 2012-12-19 18:03:42 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2013-01-08 23:58:41 -0800 |
| commit | 61905d64188f2db8e33bfe7bc4c499844a6b4b3a (patch) | |
| tree | 8bd29b5409d978e0d09900e08ca9b245a2dfca23 /xlators/mgmt/glusterd/src/glusterd-handler.c | |
| parent | 386031c784537e3877a0797f3fc28f9a221246ae (diff) | |
| download | glusterfs-61905d64188f2db8e33bfe7bc4c499844a6b4b3a.tar.gz glusterfs-61905d64188f2db8e33bfe7bc4c499844a6b4b3a.tar.xz glusterfs-61905d64188f2db8e33bfe7bc4c499844a6b4b3a.zip | |
glusterd: Add GF_ASSERT check in glusterd volume op handlers
Change-Id: Iea6ac1e612812ba8ffc4b60899a9e574a3b09ea6
BUG: 873549
Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com>
Reviewed-on: http://review.gluster.org/4346
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Tested-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handler.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handler.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c index 21befaea16..de729b3acf 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handler.c +++ b/xlators/mgmt/glusterd/src/glusterd-handler.c @@ -458,6 +458,7 @@ glusterd_op_txn_begin (rpcsvc_request_t *req, glusterd_op_t op, void *ctx, GF_ASSERT (NULL != ctx); this = THIS; + GF_ASSERT (this); priv = this->private; GF_ASSERT (priv); @@ -1198,6 +1199,7 @@ glusterd_handle_reset_volume (rpcsvc_request_t *req) GF_ASSERT (req); this = THIS; + GF_ASSERT (this); ret = xdr_to_generic (req->msg[0], &cli_req, (xdrproc_t)xdr_gf_cli_req); if (ret < 0) { @@ -1268,6 +1270,7 @@ glusterd_handle_set_volume (rpcsvc_request_t *req) xlator_t *this = NULL; this = THIS; + GF_ASSERT (this); GF_ASSERT (req); @@ -1368,6 +1371,7 @@ glusterd_handle_sync_volume (rpcsvc_request_t *req) GF_ASSERT (req); this = THIS; + GF_ASSERT (this); ret = xdr_to_generic (req->msg[0], &cli_req, (xdrproc_t)xdr_gf_cli_req); if (ret < 0) { @@ -2061,6 +2065,7 @@ glusterd_handle_cli_profile_volume (rpcsvc_request_t *req) GF_ASSERT (req); this = THIS; + GF_ASSERT (this); ret = xdr_to_generic (req->msg[0], &cli_req, (xdrproc_t)xdr_gf_cli_req); if (ret < 0) { @@ -2898,6 +2903,7 @@ glusterd_handle_status_volume (rpcsvc_request_t *req) GF_ASSERT (req); this = THIS; + GF_ASSERT (this); ret = xdr_to_generic (req->msg[0], &cli_req, (xdrproc_t)xdr_gf_cli_req); if (ret < 0) { @@ -2973,6 +2979,7 @@ glusterd_handle_cli_clearlocks_volume (rpcsvc_request_t *req) GF_ASSERT (req); this = THIS; + GF_ASSERT (this); ret = -1; ret = xdr_to_generic (req->msg[0], &cli_req, (xdrproc_t)xdr_gf_cli_req); |