core: use syscall wrappers instead of direct syscalls - tail

tail, as in dog chasing its tail. These are the unwrapped
syscalls that have crept in (or were missed) in the previous
patches.

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: If183487de92fc7cbc47d4c5aa3f3e80eae50b84f
BUG: 1267967
Signed-off-by: Kaleb S KEITHLEY <kkeithle@redhat.com>
Reviewed-on: http://review.gluster.org/12589
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Jeff Darcy <jdarcy@redhat.com>

4 files changed, 9 insertions, 7 deletions

diff --git a/xlators/cluster/dht/src/tier.c b/xlators/cluster/dht/src/tier.c
index e857dace038..6a9bddfc179 100644
--- a/xlators/cluster/dht/src/tier.c
+++ b/xlators/cluster/dht/src/tier.c
@@ -743,7 +743,7 @@ out:
         }
 
         if (query_cbk_args && query_cbk_args->query_fd >= 0) {
-                close (query_cbk_args->query_fd);
+                sys_close (query_cbk_args->query_fd);
                 query_cbk_args->query_fd = -1;
         }
         gfdb_methods.fini_db (conn_node);
@@ -1067,7 +1067,7 @@ tier_migrate_files_using_qfile (demotion_args_t *comp,
                 goto out;
         }
         ret = tier_migrate_using_query_file ((void *)query_cbk_args);
-        close (query_cbk_args->query_fd);
+        sys_close (query_cbk_args->query_fd);
         query_cbk_args->query_fd = -1;
         if (ret) {
                 snprintf (renamed_file, sizeof renamed_file, "%s.err", qfile);
diff --git a/xlators/debug/io-stats/src/io-stats.c b/xlators/debug/io-stats/src/io-stats.c
index 4aab469a585..8ce2ebbc6e9 100644
--- a/xlators/debug/io-stats/src/io-stats.c
+++ b/xlators/debug/io-stats/src/io-stats.c
@@ -8,6 +8,7 @@
    cases as published by the Free Software Foundation.
 */
 #include "xlator.h"
+#include "syscall.h"
 
 /**
  * xlators/debug/io_stats :
@@ -2905,7 +2906,7 @@ _ios_dump_thread (xlator_t *this) {
                 xlator_name = "nfsd";
                 instance_name = this->prev->instance_name;
         }
-        if (mkdir (_IOS_DUMP_DIR, S_IRWXU | S_IRWXO | S_IRWXG) == (-1)) {
+        if (sys_mkdir (_IOS_DUMP_DIR, S_IRWXU | S_IRWXO | S_IRWXG) == (-1)) {
                 if (errno != EEXIST) {
                         gf_log (this->name, GF_LOG_ERROR,
                                 "could not create stats-dump directory %s",
@@ -2913,7 +2914,7 @@ _ios_dump_thread (xlator_t *this) {
                         goto out;
                 }
         }
-        if (mkdir (_IOS_SAMP_DIR, S_IRWXU | S_IRWXO | S_IRWXG) == (-1)) {
+        if (sys_mkdir (_IOS_SAMP_DIR, S_IRWXU | S_IRWXO | S_IRWXG) == (-1)) {
                 if (errno != EEXIST) {
                         gf_log (this->name, GF_LOG_ERROR,
                                 "could not create stats-sample directory %s",
diff --git a/xlators/features/changetimerecorder/src/changetimerecorder.c b/xlators/features/changetimerecorder/src/changetimerecorder.c
index 1831316f8a3..242fb2b51fc 100644
--- a/xlators/features/changetimerecorder/src/changetimerecorder.c
+++ b/xlators/features/changetimerecorder/src/changetimerecorder.c
@@ -13,6 +13,7 @@
 #include "gfdb_sqlite3.h"
 #include "ctr-helper.h"
 #include "ctr-messages.h"
+#include "syscall.h"
 
 /*******************************inode forget***********************************/
 
@@ -1623,7 +1624,7 @@ out:
                 ret = query_cbk_args.count;
 
         if (query_cbk_args.query_fd >= 0) {
-                close (query_cbk_args.query_fd);
+                sys_close (query_cbk_args.query_fd);
                 query_cbk_args.query_fd = -1;
         }
 
diff --git a/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c b/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
index ccb9e7c17bc..14caf361544 100644
--- a/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
+++ b/xlators/mgmt/glusterd/src/glusterd-snapshot-utils.c
@@ -3537,7 +3537,7 @@ glusterd_copy_nfs_ganesha_file (glusterd_volinfo_t *src_vol,
                         goto out;
         }
 
-        ret = lstat (src_path, &stbuf);
+        ret = sys_lstat (src_path, &stbuf);
         if (ret) {
                 /* *
                 * If export file is not present, volume is not exported
@@ -3736,7 +3736,7 @@ glusterd_restore_nfs_ganesha_file (glusterd_volinfo_t *src_vol,
         if (ret < 0)
                 goto out;
 
-        ret = lstat (src_path, &stbuf);
+        ret = sys_lstat (src_path, &stbuf);
         if (ret) {
                 if (errno == ENOENT) {
                         ret = 0;