[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v5 2/5] util/qemu-sockets.c: Split host:port parsing out of inet_
From: |
Doug Evans |
Subject: |
[PATCH v5 2/5] util/qemu-sockets.c: Split host:port parsing out of inet_parse |
Date: |
Fri, 19 Feb 2021 16:13:19 -0800 |
The parsing is moved into new function inet_parse_host_and_port.
This is done in preparation for using the function in net/slirp.c.
Signed-off-by: Doug Evans <dje@google.com>
---
Changes from v4:
- move recognition of "[]:port" to separate patch
- allow passing NULL for ip_v6
- fix some formatting issues
Changes from v3:
- this patch is new in v4
- provides new utility: inet_parse_host_and_port, updates inet_parse
to use it
include/qemu/sockets.h | 3 ++
util/qemu-sockets.c | 80 +++++++++++++++++++++++++++++++-----------
2 files changed, 62 insertions(+), 21 deletions(-)
diff --git a/include/qemu/sockets.h b/include/qemu/sockets.h
index 7d1f813576..b1448cfa24 100644
--- a/include/qemu/sockets.h
+++ b/include/qemu/sockets.h
@@ -31,6 +31,9 @@ int socket_set_fast_reuse(int fd);
int inet_ai_family_from_address(InetSocketAddress *addr,
Error **errp);
+const char *inet_parse_host_and_port(const char *str, int terminator,
+ char **hostp, char **portp, bool *is_v6,
+ Error **errp);
int inet_parse(InetSocketAddress *addr, const char *str, Error **errp);
int inet_connect(const char *str, Error **errp);
int inet_connect_saddr(InetSocketAddress *saddr, Error **errp);
diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
index 8af0278f15..3ca6a6fb3d 100644
--- a/util/qemu-sockets.c
+++ b/util/qemu-sockets.c
@@ -615,44 +615,82 @@ static int inet_parse_flag(const char *flagname, const
char *optstr, bool *val,
return 0;
}
-int inet_parse(InetSocketAddress *addr, const char *str, Error **errp)
+/*
+ * Parse an inet host and port as "host:port<terminator>".
+ * Terminator may be '\0'.
+ * The syntax for IPv4 addresses is: address:port. "address" is optional,
+ * and may be empty (i.e., str is ":port").
+ * The syntax for IPv6 addresses is: [address]:port. Upon return the wrapping
+ * [] brackets are removed.
+ * Host names are also supported as hostname:port. It is up to the caller to
+ * distinguish host names from numeric IPv4 addresses.
+ * On success, returns a pointer to the terminator. Space for the address and
+ * port is malloced and stored in *host, *port, the caller must free.
+ * If is_v6 is non-NULL, then it is set to true if the address is an IPv6
+ * address (i.e., [address]), otherwise it is set to false.
+ * On failure NULL is returned with the error stored in *errp.
+ */
+const char *inet_parse_host_and_port(const char *str, int terminator,
+ char **hostp, char **portp, bool *is_v6,
+ Error **errp)
{
- const char *optstr, *h;
+ const char *terminator_ptr = strchr(str, terminator);
+ g_autofree char *buf = NULL;
char host[65];
char port[33];
- int to;
- int pos;
- char *begin;
- memset(addr, 0, sizeof(*addr));
+ if (terminator_ptr == NULL) {
+ /* If the terminator isn't found then use the entire string. */
+ terminator_ptr = str + strlen(str);
+ }
+ buf = g_strndup(str, terminator_ptr - str);
- /* parse address */
- if (str[0] == ':') {
+ if (buf[0] == ':') {
/* no host given */
host[0] = '\0';
- if (sscanf(str, ":%32[^,]%n", port, &pos) != 1) {
- error_setg(errp, "error parsing port in address '%s'", str);
- return -1;
+ if (sscanf(buf, ":%32s", port) != 1) {
+ error_setg(errp, "error parsing port in address '%s'", buf);
+ return NULL;
}
- } else if (str[0] == '[') {
+ } else if (buf[0] == '[') {
/* IPv6 addr */
- if (sscanf(str, "[%64[^]]]:%32[^,]%n", host, port, &pos) != 2) {
- error_setg(errp, "error parsing IPv6 address '%s'", str);
- return -1;
+ if (sscanf(buf, "[%64[^]]]:%32s", host, port) != 2) {
+ error_setg(errp, "error parsing IPv6 address '%s'", buf);
+ return NULL;
}
} else {
/* hostname or IPv4 addr */
- if (sscanf(str, "%64[^:]:%32[^,]%n", host, port, &pos) != 2) {
- error_setg(errp, "error parsing address '%s'", str);
- return -1;
+ if (sscanf(buf, "%64[^:]:%32s", host, port) != 2) {
+ error_setg(errp, "error parsing address '%s'", buf);
+ return NULL;
}
}
- addr->host = g_strdup(host);
- addr->port = g_strdup(port);
+ *hostp = g_strdup(host);
+ *portp = g_strdup(port);
+ if (is_v6 != NULL) {
+ *is_v6 = buf[0] == '[';
+ }
+
+ return terminator_ptr;
+}
+
+int inet_parse(InetSocketAddress *addr, const char *str, Error **errp)
+{
+ const char *optstr, *h;
+ int to;
+ int pos;
+ char *begin;
+
+ memset(addr, 0, sizeof(*addr));
+
+ optstr = inet_parse_host_and_port(str, ',', &addr->host, &addr->port,
+ NULL, errp);
+ if (optstr == NULL) {
+ return -1;
+ }
/* parse options */
- optstr = str + pos;
h = strstr(optstr, ",to=");
if (h) {
h += 4;
--
2.30.0.617.g56c4b15f3c-goog
- [PATCH v5 0/5] Add support for ipv6 host forwarding, Doug Evans, 2021/02/19
- [PATCH v5 2/5] util/qemu-sockets.c: Split host:port parsing out of inet_parse,
Doug Evans <=
- [PATCH v5 3/5] inet_parse_host_and_addr: Recognize []:port (empty ipv6 address), Doug Evans, 2021/02/19
- [PATCH v5 4/5] net/slirp.c: Refactor address parsing, Doug Evans, 2021/02/19
- [PATCH v5 5/5] net: Extend host forwarding to support IPv6, Doug Evans, 2021/02/19
- Re: [PATCH v5 0/5] Add support for ipv6 host forwarding, no-reply, 2021/02/19