diff options
| author | Raghavendra Bhat <raghavendra@redhat.com> | 2013-01-30 13:47:31 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2013-01-30 23:40:37 -0800 |
| commit | e979c0de9dde14fe18d0ad7298c6da9cc878bbab (patch) | |
| tree | c37dc5a0f29d16622d4eb902cddce57abe7891db /libglusterfs/src | |
| parent | 559219492477624290b4962a9f82c49bcda011d9 (diff) | |
libglusterfs/syncop: do not hold ref on the fd in cbk
* Do not do fd_ref in cbks of the fops which return a fd (such as
open, opendir, create).
Change-Id: Ic2f5b234c5c09c258494f4fb5d600a64813823ad
BUG: 885008
Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-on: http://review.gluster.org/4282
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Amar Tumballi <amarts@redhat.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'libglusterfs/src')
| -rw-r--r-- | libglusterfs/src/syncop.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/libglusterfs/src/syncop.c b/libglusterfs/src/syncop.c index 0c807ef5d3a..c8a9b5848c1 100644 --- a/libglusterfs/src/syncop.c +++ b/libglusterfs/src/syncop.c @@ -995,9 +995,6 @@ syncop_open_cbk (call_frame_t *frame, void *cookie, xlator_t *this, args->op_ret = op_ret; args->op_errno = op_errno; - if (op_ret != -1) - fd_ref (fd); - __wake (args); return 0; @@ -1148,9 +1145,6 @@ syncop_create_cbk (call_frame_t *frame, void *cookie, xlator_t *this, args->op_ret = op_ret; args->op_errno = op_errno; - if (op_ret != -1) - fd_ref (fd); - __wake (args); return 0; |