summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2020-05-26 19:45:45 +0000
committerFlorin Coras <fcoras@cisco.com>2020-05-26 20:35:19 +0000
commit0ab36f55753d3d1417c41f8a3aec5e79a882555c (patch)
tree327e838ca68bba6e289d086a66d2f893bbab629e
parent1bf6df4ff9c83bac1fc329a4b5c4d7061f13720a (diff)
vcl: fix ldp fcntl64
Type: fix Signed-off-by: Florin Coras <fcoras@cisco.com> Change-Id: I342d8f928fcc7de74f9bd288c1a5d63ea1f90020
-rw-r--r--src/vcl/ldp.c29
-rw-r--r--src/vcl/ldp_socket_wrapper.c33
-rw-r--r--src/vcl/ldp_socket_wrapper.h2
3 files changed, 24 insertions, 40 deletions
diff --git a/src/vcl/ldp.c b/src/vcl/ldp.c
index 19f3827bbcf..cda4425e574 100644
--- a/src/vcl/ldp.c
+++ b/src/vcl/ldp.c
@@ -478,17 +478,11 @@ writev (int fd, const struct iovec * iov, int iovcnt)
return size;
}
-int
-fcntl (int fd, int cmd, ...)
+static int
+fcntl_internal (int fd, int cmd, va_list ap)
{
vls_handle_t vlsh;
int rv = 0;
- va_list ap;
-
- if ((errno = -ldp_init ()))
- return -1;
-
- va_start (ap, cmd);
vlsh = ldp_fd_to_vlsh (fd);
LDBG (0, "fd %u vlsh %d, cmd %u", fd, vlsh, cmd);
@@ -534,6 +528,20 @@ fcntl (int fd, int cmd, ...)
#endif
}
+ return rv;
+}
+
+int
+fcntl (int fd, int cmd, ...)
+{
+ va_list ap;
+ int rv;
+
+ if ((errno = -ldp_init ()))
+ return -1;
+
+ va_start (ap, cmd);
+ rv = fcntl_internal (fd, cmd, ap);
va_end (ap);
return rv;
@@ -545,8 +553,11 @@ fcntl64 (int fd, int cmd, ...)
va_list ap;
int rv;
+ if ((errno = -ldp_init ()))
+ return -1;
+
va_start (ap, cmd);
- rv = fcntl (fd, cmd, ap);
+ rv = fcntl_internal (fd, cmd, ap);
va_end (ap);
return rv;
}
diff --git a/src/vcl/ldp_socket_wrapper.c b/src/vcl/ldp_socket_wrapper.c
index 38ee297174c..253847136ae 100644
--- a/src/vcl/ldp_socket_wrapper.c
+++ b/src/vcl/ldp_socket_wrapper.c
@@ -550,45 +550,16 @@ libc_eventfd (int count, int flags)
int
libc_vfcntl (int fd, int cmd, va_list ap)
{
- long int args[4];
- int rc;
- int i;
-
swrap_bind_symbol_libc (fcntl);
-
- for (i = 0; i < 4; i++)
- {
- args[i] = va_arg (ap, long int);
- }
-
- rc = swrap.libc.symbols._libc_fcntl.f (fd,
- cmd,
- args[0], args[1], args[2], args[3]);
-
- return rc;
+ return swrap.libc.symbols._libc_fcntl.f (fd, cmd, va_arg (ap, long int));
}
#ifdef HAVE_FCNTL64
int
libc_vfcntl64 (int fd, int cmd, va_list ap)
{
- long int args[4];
- int rc;
- int i;
-
swrap_bind_symbol_libc (fcntl64);
-
- for (i = 0; i < 4; i++)
- {
- args[i] = va_arg (ap, long int);
- }
-
- rc = swrap.libc.symbols._libc_fcntl64.f (fd,
- cmd,
- args[0], args[1], args[2],
- args[3]);
-
- return rc;
+ return swrap.libc.symbols._libc_fcntl64.f (fd, cmd, va_arg (ap, long int));
}
#endif
diff --git a/src/vcl/ldp_socket_wrapper.h b/src/vcl/ldp_socket_wrapper.h
index 94529e3b744..0d167cf1aeb 100644
--- a/src/vcl/ldp_socket_wrapper.h
+++ b/src/vcl/ldp_socket_wrapper.h
@@ -122,7 +122,9 @@ int libc_eventfd (int count, int flags);
int libc_vfcntl (int fd, int cmd, va_list ap);
+#ifdef HAVE_FCNTL64
int libc_vfcntl64 (int fd, int cmd, va_list ap);
+#endif
int libc_vioctl (int fd, int cmd, va_list ap);