aboutgitcodebugslistschat
diff options
context:
space:
mode:
authorStefano Brivio <sbrivio@redhat.com>2022-10-07 04:01:56 +0200
committerStefano Brivio <sbrivio@redhat.com>2022-10-15 02:10:36 +0200
commita62ed181db9ba7d85d057365d5331dd35026247f (patch)
tree960239e3c58df5d81b7f95f8489fbe0829380007
parent9de65dd3f43a0ee976cc01250641834d99bbfa74 (diff)
downloadpasst-a62ed181db9ba7d85d057365d5331dd35026247f.tar
passt-a62ed181db9ba7d85d057365d5331dd35026247f.tar.gz
passt-a62ed181db9ba7d85d057365d5331dd35026247f.tar.bz2
passt-a62ed181db9ba7d85d057365d5331dd35026247f.tar.lz
passt-a62ed181db9ba7d85d057365d5331dd35026247f.tar.xz
passt-a62ed181db9ba7d85d057365d5331dd35026247f.tar.zst
passt-a62ed181db9ba7d85d057365d5331dd35026247f.zip
conf, tap: Add option to quit once the client closes the connection
This is practical to avoid explicit lifecycle management in users, e.g. libvirtd, and is trivial to implement. Signed-off-by: Stefano Brivio <sbrivio@redhat.com> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
-rw-r--r--conf.c17
-rw-r--r--passt.15
-rw-r--r--passt.h1
-rw-r--r--tap.c5
4 files changed, 27 insertions, 1 deletions
diff --git a/conf.c b/conf.c
index 7c3e346..779371f 100644
--- a/conf.c
+++ b/conf.c
@@ -712,6 +712,7 @@ static void usage(const char *name)
if (strstr(name, "pasta"))
goto pasta_opts;
+ info( " -1, --one-off Quit after handling one single client");
info( " -t, --tcp-ports SPEC TCP port forwarding to guest");
info( " can be specified multiple times");
info( " SPEC can be:");
@@ -1023,6 +1024,7 @@ void conf(struct ctx *c, int argc, char **argv)
{"no-map-gw", no_argument, &c->no_map_gw, 1 },
{"ipv4-only", no_argument, NULL, '4' },
{"ipv6-only", no_argument, NULL, '6' },
+ {"one-off", no_argument, NULL, '1' },
{"tcp-ports", required_argument, NULL, 't' },
{"udp-ports", required_argument, NULL, 'u' },
{"tcp-ns", required_argument, NULL, 'T' },
@@ -1062,7 +1064,7 @@ void conf(struct ctx *c, int argc, char **argv)
c->no_dhcp_dns = c->no_dhcp_dns_search = 1;
optstring = "dqfel:hI:p:P:m:a:n:M:g:i:D:S:46t:u:T:U:";
} else {
- optstring = "dqfel:hs:p:P:m:a:n:M:g:i:D:S:46t:u:";
+ optstring = "dqfel:hs:p:P:m:a:n:M:g:i:D:S:461t:u:";
}
c->tcp.fwd_in.mode = c->tcp.fwd_out.mode = 0;
@@ -1488,6 +1490,19 @@ void conf(struct ctx *c, int argc, char **argv)
case '6':
v6_only = true;
break;
+ case '1':
+ if (c->mode != MODE_PASST) {
+ err("--one-off is for passt mode only");
+ usage(argv[0]);
+ }
+
+ if (c->one_off) {
+ err("Redundant --one-off option");
+ usage(argv[0]);
+ }
+
+ c->one_off = true;
+ break;
case 't':
case 'u':
case 'T':
diff --git a/passt.1 b/passt.1
index 667c1bc..555a50c 100644
--- a/passt.1
+++ b/passt.1
@@ -303,6 +303,11 @@ Default is to probe a free socket, not accepting connections, starting from
\fI/tmp/passt_1.socket\fR to \fI/tmp/passt_64.socket\fR.
.TP
+.BR \-1 ", " \-\-one-off
+Quit after handling a single client connection, that is, once the client closes
+the socket, or once we get a socket error.
+
+.TP
.BR \-t ", " \-\-tcp-ports " " \fIspec
Configure TCP port forwarding to guest. \fIspec\fR can be one of:
.RS
diff --git a/passt.h b/passt.h
index de79e7b..67281db 100644
--- a/passt.h
+++ b/passt.h
@@ -196,6 +196,7 @@ struct ctx {
char sock_path[UNIX_PATH_MAX];
char pcap[PATH_MAX];
char pid_file[PATH_MAX];
+ int one_off;
int pasta_netns_fd;
diff --git a/tap.c b/tap.c
index bdcc161..78de42c 100644
--- a/tap.c
+++ b/tap.c
@@ -1011,5 +1011,10 @@ void tap_handler(struct ctx *c, int fd, uint32_t events,
return;
reinit:
+ if (c->one_off) {
+ info("Client closed connection, exiting");
+ exit(EXIT_SUCCESS);
+ }
+
tap_sock_init(c);
}