Compare commits
No commits in common. 'i10cs' and 'c9' have entirely different histories.
@ -1 +1 @@
|
|||||||
SOURCES/mptcpd-0.12.tar.gz
|
SOURCES/mptcpd-0.8.tar.gz
|
||||||
|
|||||||
@ -1 +1 @@
|
|||||||
c1403b83fac20bc17fce199f2f7f741d6a9376bc SOURCES/mptcpd-0.12.tar.gz
|
a75235d98b7d2ab1ed71e0bd9d1683e6da462bfb SOURCES/mptcpd-0.8.tar.gz
|
||||||
|
|||||||
@ -0,0 +1,31 @@
|
|||||||
|
From 5633f08e35552295b2f9414ff32ca4e8e081b2f2 Mon Sep 17 00:00:00 2001
|
||||||
|
Message-Id: <5633f08e35552295b2f9414ff32ca4e8e081b2f2.1638196305.git.dcaratti@redhat.com>
|
||||||
|
From: Paolo Abeni <paolo.abeni@gmail.com>
|
||||||
|
Date: Tue, 12 Oct 2021 19:24:33 +0200
|
||||||
|
Subject: [PATCH] mptcpize: force MPTCP usage for IPPROTO_IP, too (#159)
|
||||||
|
|
||||||
|
The current ignores calls alike:
|
||||||
|
|
||||||
|
socket(AF_INET, SOCK_STREAM, IPPROTO_IP)
|
||||||
|
|
||||||
|
We should hijack them, too.
|
||||||
|
---
|
||||||
|
src/mptcpwrap.c | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/src/mptcpwrap.c b/src/mptcpwrap.c
|
||||||
|
index 37b0545..1aaf00f 100644
|
||||||
|
--- a/src/mptcpwrap.c
|
||||||
|
+++ b/src/mptcpwrap.c
|
||||||
|
@@ -27,7 +27,7 @@ int __attribute__((visibility("default"))) socket(int family, int type, int prot
|
||||||
|
goto do_socket;
|
||||||
|
|
||||||
|
// socket(AF_INET, SOCK_STREM, 0) maps to TCP, too
|
||||||
|
- if (protocol == 0 && protocol != IPPROTO_TCP)
|
||||||
|
+ if (protocol != 0 && protocol != IPPROTO_TCP)
|
||||||
|
goto do_socket;
|
||||||
|
|
||||||
|
protocol = IPPROTO_TCP + 256;
|
||||||
|
--
|
||||||
|
2.31.1
|
||||||
|
|
||||||
@ -0,0 +1,75 @@
|
|||||||
|
From 591b3b168d949c45d5b5994332a3007767845434 Mon Sep 17 00:00:00 2001
|
||||||
|
Message-Id: <591b3b168d949c45d5b5994332a3007767845434.1638277575.git.dcaratti@redhat.com>
|
||||||
|
From: Paolo Abeni <paolo.abeni@gmail.com>
|
||||||
|
Date: Thu, 14 Oct 2021 05:05:54 +0200
|
||||||
|
Subject: [PATCH] mptcpize: use explicit file copy instead of rename() (#161)
|
||||||
|
|
||||||
|
The mentioned syscall fails if the involved files belong to
|
||||||
|
different fs, which is pretty much expected in the relevant
|
||||||
|
scenario (tmp file, in tmpfs, and unit file usually under the
|
||||||
|
root partition)
|
||||||
|
|
||||||
|
Instead use sendfile() to explicitly copy all the contents. Note
|
||||||
|
that we need to close and re-open the unit file, as sendfile()
|
||||||
|
expect a O_WRITE fd as the target.
|
||||||
|
---
|
||||||
|
src/mptcpize.c | 24 ++++++++++++++++++++----
|
||||||
|
1 file changed, 20 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/mptcpize.c b/src/mptcpize.c
|
||||||
|
index cb79e09..b502d75 100644
|
||||||
|
--- a/src/mptcpize.c
|
||||||
|
+++ b/src/mptcpize.c
|
||||||
|
@@ -13,6 +13,7 @@
|
||||||
|
|
||||||
|
#include <sys/types.h>
|
||||||
|
#include <sys/stat.h>
|
||||||
|
+#include <sys/sendfile.h>
|
||||||
|
|
||||||
|
#include <argp.h>
|
||||||
|
#include <dlfcn.h>
|
||||||
|
@@ -163,10 +164,12 @@ static int unit_update(int argc, char *argv[], int enable)
|
||||||
|
char *unit, *line = NULL;
|
||||||
|
int append_env = enable;
|
||||||
|
char dst_path[PATH_MAX];
|
||||||
|
+ off_t bytes_copied = 0;
|
||||||
|
+ struct stat fileinfo;
|
||||||
|
+ int dst, unit_fd;
|
||||||
|
size_t len = 0;
|
||||||
|
ssize_t read;
|
||||||
|
FILE *src;
|
||||||
|
- int dst;
|
||||||
|
|
||||||
|
if (argc < 1) {
|
||||||
|
fprintf(stderr, "missing unit argument\n");
|
||||||
|
@@ -210,11 +213,24 @@ static int unit_update(int argc, char *argv[], int enable)
|
||||||
|
error(1, errno, "can't read from %s", unit);
|
||||||
|
free(line);
|
||||||
|
fclose(src);
|
||||||
|
- close(dst);
|
||||||
|
|
||||||
|
- if (rename(dst_path, unit) < 0)
|
||||||
|
- error(1, errno, "can't rename %s to %s", dst_path, unit);
|
||||||
|
+ // copy back the modified file into the original unit
|
||||||
|
+ // note: avoid using rename, as it fails across filesystems
|
||||||
|
+ if (fstat(dst, &fileinfo) < 0)
|
||||||
|
+ error(1, errno, "can't stat %s", dst_path);
|
||||||
|
+
|
||||||
|
+ // re-open the unit file for writing
|
||||||
|
+ // mkstemp already opened the temporary file for R/W so we don't need
|
||||||
|
+ // to touch that file descriptor.
|
||||||
|
+ unit_fd = open(unit, O_TRUNC | O_RDWR);
|
||||||
|
+ if (unit_fd < 0)
|
||||||
|
+ error(1, errno, "can't open %s for writing", unit);
|
||||||
|
|
||||||
|
+ while (bytes_copied < fileinfo.st_size)
|
||||||
|
+ if (sendfile(unit_fd, dst, &bytes_copied, fileinfo.st_size - bytes_copied) < 0)
|
||||||
|
+ error(1, errno, "can't copy from %s to %s", dst_path, unit);
|
||||||
|
+
|
||||||
|
+ close(dst);
|
||||||
|
if (system("systemctl daemon-reload") != 0)
|
||||||
|
error(1, errno, "can't reload unit, manual 'systemctl daemon-reload' is required");
|
||||||
|
|
||||||
|
--
|
||||||
|
2.31.1
|
||||||
|
|
||||||
@ -0,0 +1,24 @@
|
|||||||
|
diff --git a/etc/mptcpd.conf.in b/etc/mptcpd.conf.in
|
||||||
|
index 615c63b..c6b31ac 100644
|
||||||
|
--- a/etc/mptcpd.conf.in
|
||||||
|
+++ b/etc/mptcpd.conf.in
|
||||||
|
@@ -19,7 +19,6 @@ log=@mptcpd_logger@
|
||||||
|
# ----------------
|
||||||
|
# Plugin directory
|
||||||
|
# ----------------
|
||||||
|
-plugin-dir=@pkglibdir@
|
||||||
|
|
||||||
|
# -------------------
|
||||||
|
# Path manager plugin
|
||||||
|
diff --git a/src/mptcp.service.in b/src/mptcp.service.in
|
||||||
|
index 89dfe04..738f59f 100644
|
||||||
|
--- a/src/mptcp.service.in
|
||||||
|
+++ b/src/mptcp.service.in
|
||||||
|
@@ -9,7 +9,6 @@ Documentation=man:mptcpd(8)
|
||||||
|
[Service]
|
||||||
|
Type=simple
|
||||||
|
DynamicUser=yes
|
||||||
|
-Environment=LD_LIBRARY_PATH=@libdir@
|
||||||
|
ExecStart=@libexecdir@/mptcpd --log=journal
|
||||||
|
CapabilityBoundingSet=CAP_NET_ADMIN
|
||||||
|
AmbientCapabilities=CAP_NET_ADMIN
|
||||||
@ -0,0 +1,102 @@
|
|||||||
|
--- a/configure.ac
|
||||||
|
+++ b/configure.ac
|
||||||
|
@@ -207,8 +207,7 @@ AC_CHECK_HEADERS([linux/mptcp.h],
|
||||||
|
AX_CHECK_DEFINE(
|
||||||
|
[linux/mptcp.h],
|
||||||
|
[MPTCP_PM_NAME],
|
||||||
|
- [AC_DEFINE([HAVE_LINUX_MPTCP_H_UPSTREAM], [1])
|
||||||
|
- AX_CHECK_DEFINE([linux/mptcp.h],
|
||||||
|
+ [AX_CHECK_DEFINE([linux/mptcp.h],
|
||||||
|
[MPTCP_PM_EV_GRP_NAME],
|
||||||
|
[AC_DEFINE([HAVE_LINUX_MPTCP_H_UPSTREAM_EVENTS],
|
||||||
|
[1])
|
||||||
|
--- a/include/linux/mptcp_upstream.h
|
||||||
|
+++ b/include/linux/mptcp_upstream.h
|
||||||
|
@@ -2,8 +2,16 @@
|
||||||
|
#ifndef _UAPI_MPTCP_H
|
||||||
|
#define _UAPI_MPTCP_H
|
||||||
|
|
||||||
|
+#ifndef __KERNEL__
|
||||||
|
+#include <netinet/in.h> /* for sockaddr_in and sockaddr_in6 */
|
||||||
|
+#include <sys/socket.h> /* for struct sockaddr */
|
||||||
|
+#endif
|
||||||
|
+
|
||||||
|
#include <linux/const.h>
|
||||||
|
#include <linux/types.h>
|
||||||
|
+#include <linux/in.h> /* for sockaddr_in */
|
||||||
|
+#include <linux/in6.h> /* for sockaddr_in6 */
|
||||||
|
+#include <linux/socket.h> /* for sockaddr_storage and sa_family */
|
||||||
|
|
||||||
|
#define MPTCP_SUBFLOW_FLAG_MCAP_REM _BITUL(0)
|
||||||
|
#define MPTCP_SUBFLOW_FLAG_MCAP_LOC _BITUL(1)
|
||||||
|
@@ -48,6 +56,9 @@ enum {
|
||||||
|
MPTCP_PM_ATTR_ADDR, /* nested address */
|
||||||
|
MPTCP_PM_ATTR_RCV_ADD_ADDRS, /* u32 */
|
||||||
|
MPTCP_PM_ATTR_SUBFLOWS, /* u32 */
|
||||||
|
+ MPTCP_PM_ATTR_TOKEN, /* u32 */
|
||||||
|
+ MPTCP_PM_ATTR_LOC_ID, /* u8 */
|
||||||
|
+ MPTCP_PM_ATTR_ADDR_REMOTE, /* nested address */
|
||||||
|
|
||||||
|
__MPTCP_PM_ATTR_MAX
|
||||||
|
};
|
||||||
|
@@ -73,6 +84,8 @@ enum {
|
||||||
|
#define MPTCP_PM_ADDR_FLAG_SIGNAL (1 << 0)
|
||||||
|
#define MPTCP_PM_ADDR_FLAG_SUBFLOW (1 << 1)
|
||||||
|
#define MPTCP_PM_ADDR_FLAG_BACKUP (1 << 2)
|
||||||
|
+#define MPTCP_PM_ADDR_FLAG_FULLMESH (1 << 3)
|
||||||
|
+#define MPTCP_PM_ADDR_FLAG_IMPLICIT (1 << 4)
|
||||||
|
|
||||||
|
enum {
|
||||||
|
MPTCP_PM_CMD_UNSPEC,
|
||||||
|
@@ -84,6 +97,10 @@ enum {
|
||||||
|
MPTCP_PM_CMD_SET_LIMITS,
|
||||||
|
MPTCP_PM_CMD_GET_LIMITS,
|
||||||
|
MPTCP_PM_CMD_SET_FLAGS,
|
||||||
|
+ MPTCP_PM_CMD_ANNOUNCE,
|
||||||
|
+ MPTCP_PM_CMD_REMOVE,
|
||||||
|
+ MPTCP_PM_CMD_SUBFLOW_CREATE,
|
||||||
|
+ MPTCP_PM_CMD_SUBFLOW_DESTROY,
|
||||||
|
|
||||||
|
__MPTCP_PM_CMD_AFTER_LAST
|
||||||
|
};
|
||||||
|
@@ -177,6 +194,7 @@ enum mptcp_event_attr {
|
||||||
|
MPTCP_ATTR_IF_IDX, /* s32 */
|
||||||
|
MPTCP_ATTR_RESET_REASON,/* u32 */
|
||||||
|
MPTCP_ATTR_RESET_FLAGS, /* u32 */
|
||||||
|
+ MPTCP_ATTR_SERVER_SIDE, /* u8 */
|
||||||
|
|
||||||
|
__MPTCP_ATTR_AFTER_LAST
|
||||||
|
};
|
||||||
|
@@ -192,4 +210,32 @@ enum mptcp_event_attr {
|
||||||
|
#define MPTCP_RST_EBADPERF 5
|
||||||
|
#define MPTCP_RST_EMIDDLEBOX 6
|
||||||
|
|
||||||
|
+struct mptcp_subflow_data {
|
||||||
|
+ __u32 size_subflow_data; /* size of this structure in userspace */
|
||||||
|
+ __u32 num_subflows; /* must be 0, set by kernel */
|
||||||
|
+ __u32 size_kernel; /* must be 0, set by kernel */
|
||||||
|
+ __u32 size_user; /* size of one element in data[] */
|
||||||
|
+} __attribute__((aligned(8)));
|
||||||
|
+
|
||||||
|
+struct mptcp_subflow_addrs {
|
||||||
|
+ union {
|
||||||
|
+ __kernel_sa_family_t sa_family;
|
||||||
|
+ struct sockaddr sa_local;
|
||||||
|
+ struct sockaddr_in sin_local;
|
||||||
|
+ struct sockaddr_in6 sin6_local;
|
||||||
|
+ struct __kernel_sockaddr_storage ss_local;
|
||||||
|
+ };
|
||||||
|
+ union {
|
||||||
|
+ struct sockaddr sa_remote;
|
||||||
|
+ struct sockaddr_in sin_remote;
|
||||||
|
+ struct sockaddr_in6 sin6_remote;
|
||||||
|
+ struct __kernel_sockaddr_storage ss_remote;
|
||||||
|
+ };
|
||||||
|
+};
|
||||||
|
+
|
||||||
|
+/* MPTCP socket options */
|
||||||
|
+#define MPTCP_INFO 1
|
||||||
|
+#define MPTCP_TCPINFO 2
|
||||||
|
+#define MPTCP_SUBFLOW_ADDRS 3
|
||||||
|
+
|
||||||
|
#endif /* _UAPI_MPTCP_H */
|
||||||
Loading…
Reference in new issue