andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

afr: fix bug in passing child index in afr_inode_write_fill

Browse Source 
Change-Id: I7b70de317a5f15a3bf483ffe40b971143deddc11
BUG: 1401218
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Reviewed-on: http://review.gluster.org/16029
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
This commit is contained in:
Ravishankar N 2016-12-05 21:14:57 +05:30 committed by Pranith Kumar Karampuri
parent aa22f24f5d
commit ca13525a5d
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
xlators/cluster/afr/src/afr-transaction.c
View File

@ -1250,7 +1250,6 @@ afr_pre_op_writev_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
compound_args_cbk_t *args_cbk = data;
int call_count = -1;
int child_index = -1;
int i = 0;
local = frame->local;
child_index = (long) cookie;
@ -1268,11 +1267,11 @@ afr_pre_op_writev_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
* Handle that case by passing the op_ret, op_errno values explicitly.
*/
if ((op_ret == -1) && (args_cbk == NULL)) {
afr_inode_write_fill (frame, this, (long) i, op_ret, op_errno,
NULL, NULL, NULL);
afr_inode_write_fill (frame, this, child_index, op_ret,
op_errno, NULL, NULL, NULL);
} else {
write_args_cbk = &args_cbk->rsp_list[1];
afr_inode_write_fill (frame, this, (long) child_index,
afr_inode_write_fill (frame, this, child_index,
write_args_cbk->op_ret,
write_args_cbk->op_errno,
&write_args_cbk->prestat,