public inbox for gwml@vger.gnuweeb.org
 help / color / mirror / Atom feed
From: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
To: Ammar Faiz <ammarfaizi2@gnuweeb.org>
Cc: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>,
	Ahmad Gani <reyuki@gnuweeb.org>,
	GNU/Weeb Mailing List <gwml@vger.gnuweeb.org>
Subject: [PATCH gwproxy v12 4/8] Add DNS resolver code
Date: Fri, 19 Sep 2025 01:47:26 +0700	[thread overview]
Message-ID: <20250918184730.598305-5-alviro.iskandar@gnuweeb.org> (raw)
In-Reply-To: <20250918184730.598305-1-alviro.iskandar@gnuweeb.org>

In preparation to add a new DNS feature that does not rely on
getaddrinfo(), create the initial data structure that contains udp_fd,
server addr, sess_map, and stack to accomodate the new DNS resolver
feature. The stack and sess_map will be needed to map DNS queries to
their corresponding active client session.

Signed-off-by: Alviro Iskandar Setiawan <alviro.iskandar@gnuweeb.org>
---
 src/gwproxy/dns_resolver.c | 252 +++++++++++++++++++++++++++++++++++++
 src/gwproxy/dns_resolver.h |  19 +++
 2 files changed, 271 insertions(+)
 create mode 100644 src/gwproxy/dns_resolver.c
 create mode 100644 src/gwproxy/dns_resolver.h

diff --git a/src/gwproxy/dns_resolver.c b/src/gwproxy/dns_resolver.c
new file mode 100644
index 0000000..4d6924f
--- /dev/null
+++ b/src/gwproxy/dns_resolver.c
@@ -0,0 +1,252 @@
+/*
+ * A new experimental DNS resolver that does not rely on getaddrinfo().
+ */
+
+#include <gwproxy/dns_resolver.h>
+#include <gwproxy/syscall.h>
+#include <gwproxy/net.h>
+#include <stdlib.h>
+#include <string.h>
+#include <errno.h>
+
+struct gwp_dns_resolver_map {
+	uint32_t		cap;
+	uint32_t		sp;
+	uint16_t		*stack;
+	struct gwp_conn_pair	**sess_map;
+};
+
+static int alloc_res_map(struct gwp_dns_resolver_map **map, uint32_t cap)
+{
+	struct gwp_dns_resolver_map *m;
+	uint32_t i, local_cap = 2;
+
+	/*
+	 * Keep the capacity a power of 2 for easier doubling to
+	 * max size of 65536.
+	 */
+	while (local_cap < cap)
+		local_cap *= 2;
+
+	if (local_cap > 65536)
+		return -EINVAL;
+
+	m = malloc(sizeof(*m));
+	if (!m)
+		return -ENOMEM;
+
+	m->sess_map = calloc(local_cap, sizeof(*m->sess_map));
+	if (!m->sess_map)
+		goto err_m;
+
+	m->stack = malloc(local_cap * sizeof(*m->stack));
+	if (!m->stack)
+		goto err_sess;
+
+	m->cap = local_cap;
+	m->sp = 0;
+
+	/*
+	 * Fill the stack with all available indexes.
+	 */
+	i = local_cap;
+	while (i--)
+		m->stack[m->sp++] = i;
+
+	*map = m;
+	return 0;
+
+err_sess:
+	free(m->sess_map);
+err_m:
+	free(m);
+	return -ENOMEM;
+}
+
+static void free_res_map(struct gwp_dns_resolver_map *map)
+{
+	if (!map)
+		return;
+
+	free(map->stack);
+	free(map->sess_map);
+	free(map);
+}
+
+static int expand_res_map(struct gwp_dns_resolver_map *map)
+{
+	uint32_t new_cap, i, old_cap = map->cap;
+	struct gwp_conn_pair **new_sess_map;
+	uint16_t *new_stack;
+
+	new_cap = old_cap * 2;
+	if (new_cap > 65536)
+		return -ENOSPC;
+
+	new_sess_map = realloc(map->sess_map, new_cap * sizeof(*map->sess_map));
+	if (!new_sess_map)
+		return -ENOMEM;
+	map->sess_map = new_sess_map;
+	memset(&map->sess_map[old_cap], 0, old_cap * sizeof(*map->sess_map));
+
+	new_stack = realloc(map->stack, new_cap * sizeof(*map->stack));
+	if (!new_stack)
+		return -ENOMEM;
+	map->stack = new_stack;
+
+	i = new_cap;
+	while (i-- > old_cap)
+		map->stack[map->sp++] = i;
+
+	map->cap = new_cap;
+	return 0;
+}
+
+static int insert_map(struct gwp_dns_resolver_map *map,
+		      struct gwp_conn_pair *pair, uint16_t *txid)
+{
+	uint32_t x;
+	int r;
+
+	if (!map->sp) {
+		r = expand_res_map(map);
+		if (r)
+			return r;
+	}
+
+	x = map->stack[--map->sp];
+	map->sess_map[x] = pair;
+	*txid = x;
+	return 0;
+}
+
+static struct gwp_conn_pair *lookup_map(struct gwp_dns_resolver_map *map,
+					uint16_t txid)
+{
+	if (txid >= map->cap)
+		return NULL;
+
+	return map->sess_map[txid];
+}
+
+static void try_reset_map(struct gwp_dns_resolver_map *map)
+{
+	struct gwp_conn_pair **new_sess_map;
+	uint32_t i, new_cap = 16;
+	uint16_t *new_stack;
+
+	if (map->cap <= 16)
+		return;
+
+	/*
+	 * Try to allocate a smaller map first before freeing the
+	 * existing one so we don't lose the existing map if
+	 * one of reallocations fail.
+	 */
+	new_sess_map = calloc(new_cap, sizeof(*new_sess_map));
+	if (!new_sess_map)
+		return;
+
+	new_stack = malloc(new_cap * sizeof(*new_stack));
+	if (!new_stack) {
+		free(new_sess_map);
+		return;
+	}
+
+	/*
+	 * Ok, we are good to go, free the old map and
+	 * replace with the new one.
+	 */
+	free(map->sess_map);
+	free(map->stack);
+	map->sess_map = new_sess_map;
+	map->stack = new_stack;
+	map->cap = new_cap;
+	map->sp = 0;
+
+	i = new_cap;
+	while (i--)
+		map->stack[map->sp++] = i;
+}
+
+static void delete_map(struct gwp_dns_resolver_map *map, uint16_t txid)
+{
+	if (txid >= map->cap)
+		return;
+
+	if (!map->sess_map[txid])
+		return;
+
+	map->sess_map[txid] = NULL;
+	map->stack[map->sp++] = txid;
+
+	/*
+	 * If all slots are free, try to shrink the map.
+	 */
+	if (map->sp == map->cap)
+		try_reset_map(map);
+}
+
+static int init_udp_sock(struct gwp_dns_resolver *gdr, const char *srv_addr)
+{
+	static const int type = SOCK_DGRAM | SOCK_NONBLOCK | SOCK_CLOEXEC;
+	struct gwp_sockaddr addr;
+	int f, r;
+
+	r = convert_str_to_ssaddr(srv_addr, &addr, 53);
+	if (r)
+		return r;
+
+	f = __sys_socket(addr.sa.sa_family, type, 0);
+	if (f < 0)
+		return f;
+
+	r = __sys_connect(f, &addr.sa, sizeof(addr));
+	if (r < 0) {
+		__sys_close(f);
+		return r;
+	}
+
+	gdr->udp_fd = f;
+	return 0;
+}
+
+int gwp_dns_res_init(struct gwp_ctx *ctx, struct gwp_dns_resolver *gdr,
+		     const char *srv_addr)
+{
+	int r;
+
+	r = alloc_res_map(&gdr->sess_map, 16);
+	if (r)
+		return r;
+
+	gdr->srv_addr = strdup(srv_addr);
+	if (!gdr->srv_addr) {
+		r = -ENOMEM;
+		goto err_map;
+	}
+
+	r = init_udp_sock(gdr, gdr->srv_addr);
+	if (r)
+		goto err_srv;
+
+	(void)ctx;
+	return 0;
+
+err_srv:
+	free(gdr->srv_addr);
+err_map:
+	free_res_map(gdr->sess_map);
+	return r;
+}
+
+void gwp_dns_res_free(struct gwp_dns_resolver *res)
+{
+	if (!res)
+		return;
+
+	free_res_map(res->sess_map);
+	free(res->srv_addr);
+	__sys_close(res->udp_fd);
+	memset(res, 0, sizeof(*res));
+}
diff --git a/src/gwproxy/dns_resolver.h b/src/gwproxy/dns_resolver.h
new file mode 100644
index 0000000..d4545f1
--- /dev/null
+++ b/src/gwproxy/dns_resolver.h
@@ -0,0 +1,19 @@
+#ifndef GWPROXY__DNS_RESOLVER_H
+#define GWPROXY__DNS_RESOLVER_H
+
+#include <gwproxy/gwproxy.h>
+
+struct gwp_ctx;
+struct gwp_dns_resolver_map;
+
+struct gwp_dns_resolver {
+	int				udp_fd;
+	char				*srv_addr;
+	struct gwp_dns_resolver_map	*sess_map;
+};
+
+int gwp_dns_res_init(struct gwp_ctx *ctx, struct gwp_dns_resolver *gdr,
+		     const char *srv_addr);
+void gwp_dns_res_free(struct gwp_dns_resolver *gdr);
+
+#endif /* #ifndef GWPROXY__DNS_RESOLVER_H */
-- 
Alviro Iskandar Setiawan


  parent reply	other threads:[~2025-09-18 18:47 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-18 18:47 [PATCH gwproxy v12 0/8] Initial work on integration of DNS parser lib in gwproxy Alviro Iskandar Setiawan
2025-09-18 18:47 ` [PATCH gwproxy v12 1/8] gwproxy: Remove 'struct gwp_dns_query' declaration Alviro Iskandar Setiawan
2025-09-18 18:47 ` [PATCH gwproxy v12 2/8] gwproxy: Introduce __unused macro Alviro Iskandar Setiawan
2025-09-18 18:47 ` [PATCH gwproxy v12 3/8] Add DNS parser code Alviro Iskandar Setiawan
2025-09-18 18:47 ` Alviro Iskandar Setiawan [this message]
2025-09-18 18:47 ` [PATCH gwproxy v12 5/8] dns_resolver: Add DNS resolution interface APIs Alviro Iskandar Setiawan
2025-09-18 23:16   ` Ammar Faizi
2025-09-18 18:47 ` [PATCH gwproxy v12 6/8] gwproxy: Introduce --dns-server and --raw-dns Alviro Iskandar Setiawan
2025-09-18 22:54   ` Ammar Faizi
2025-09-18 23:07     ` Alviro Iskandar Setiawan
2025-09-18 18:47 ` [PATCH gwproxy v12 7/8] epoll: Intregrate the raw DNS feature to epoll Alviro Iskandar Setiawan
2025-09-18 18:47 ` [PATCH gwproxy v12 8/8] Makefile: Introduce --use-new-dns-resolver configure option Alviro Iskandar Setiawan
2025-09-18 18:55 ` [PATCH gwproxy v12 0/8] Initial work on integration of DNS parser lib in gwproxy Alviro Iskandar Setiawan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250918184730.598305-5-alviro.iskandar@gnuweeb.org \
    --to=alviro.iskandar@gnuweeb.org \
    --cc=ammarfaizi2@gnuweeb.org \
    --cc=gwml@vger.gnuweeb.org \
    --cc=reyuki@gnuweeb.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox