summaryrefslogtreecommitdiffstats
path: root/xlators/protocol/server
diff options
context:
space:
mode:
authorArjun Sharma <arjsharm@redhat.com>2018-10-15 17:37:36 +0530
committerAmar Tumballi <amarts@redhat.com>2018-10-21 05:56:22 +0000
commite1a9c23b83b8d46bf44a7a9952da0c61c8a46c79 (patch)
tree3bc369bc7eeb18b955c338f5f4fc170020fb4e39 /xlators/protocol/server
parentc5f5ce2a9b1c085209cae21a7e1df5f60bc1a7da (diff)
protocol/server: added check for positve value in server-handshake.c
This patch fixes CID 1124651 Change-Id: I6f33954f08cfdd7cb4236f9a81ec7980f81d19e7 updates: bz#789278 Signed-off-by: Arjun <arjsharm@redhat.com>
Diffstat (limited to 'xlators/protocol/server')
-rw-r--r--xlators/protocol/server/src/server-handshake.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/xlators/protocol/server/src/server-handshake.c b/xlators/protocol/server/src/server-handshake.c
index 698c3f8579f..5f6b96792fa 100644
--- a/xlators/protocol/server/src/server-handshake.c
+++ b/xlators/protocol/server/src/server-handshake.c
@@ -929,8 +929,9 @@ fail:
GF_ASSERT(rsp);
rsp->op_ret = 0;
- rsp->dict.dict_len = dict_serialized_length(reply);
- if (rsp->dict.dict_len) {
+ ret = dict_serialized_length(reply);
+ if (ret > 0) {
+ rsp->dict.dict_len = ret;
rsp->dict.dict_val = GF_CALLOC(1, rsp->dict.dict_len,
gf_server_mt_rsp_buf_t);
if (rsp->dict.dict_val) {