diff options
| author | Kaleb S. KEITHLEY <kkeithle@redhat.com> | 2015-10-01 16:31:19 -0400 |
|---|---|---|
| committer | Jeff Darcy <jdarcy@redhat.com> | 2015-10-28 13:38:42 -0700 |
| commit | 3066a21caafab6305527991de11c8eb43ec0044c (patch) | |
| tree | 5efc91272ac76ff1613cee1e8a41aeb32aa92d73 /xlators/storage/bd/src/bd.c | |
| parent | 063d4ead61ee47433793de81a1c77e0ba69e6e07 (diff) | |
core: use syscall wrappers instead of direct syscalls - miscellaneous
various xlators and other components are invoking system calls
directly instead of using the libglusterfs/syscall.[ch] wrappers.
If not using the system call wrappers there should be a comment
in the source explaining why the wrapper isn't used.
Change-Id: I1f47820534c890a00b452fa61f7438eb2b3f667c
BUG: 1267967
Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Reviewed-on: http://review.gluster.org/12276
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Diffstat (limited to 'xlators/storage/bd/src/bd.c')
| -rw-r--r-- | xlators/storage/bd/src/bd.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xlators/storage/bd/src/bd.c b/xlators/storage/bd/src/bd.c index f732b85f030..9599f4acbd5 100644 --- a/xlators/storage/bd/src/bd.c +++ b/xlators/storage/bd/src/bd.c @@ -614,7 +614,7 @@ bd_open_cbk (call_frame_t *frame, void *cookie, xlator_t *this, "bd_fd is NULL from fd=%p", fd); goto out; } - close (bd_fd->fd); + sys_close (bd_fd->fd); GF_FREE (bd_fd); out: @@ -690,7 +690,7 @@ out: GF_FREE (devpath); if (ret) { if (_fd >= 0) - close (_fd); + sys_close (_fd); GF_FREE (bd_fd); } @@ -892,7 +892,7 @@ bd_release (xlator_t *this, fd_t *fd) } bd_fd = (bd_fd_t *)(long)tmp_bfd; - close (bd_fd->fd); + sys_close (bd_fd->fd); GF_FREE (bd_fd); out: return 0; |