[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 4/4] net/slirp: add ipv6-hostfwd option for
From: |
Samuel Thibault |
Subject: |
Re: [Qemu-devel] [PATCH RFC 4/4] net/slirp: add ipv6-hostfwd option for user netdev type |
Date: |
Sat, 27 Oct 2018 13:38:49 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
Maxim Samoylov, le ven. 26 oct. 2018 03:03:43 +0300, a ecrit:
> +void hmp_ipv6_hostfwd_remove(Monitor *mon, const QDict *qdict)
> +{
Similarly, a lot can be shared, by introducing
const char *hmp_hostfwd_lookup(Monitor *mon, const QDict *qdict, int is_v6)
which will contain all the lookup and tcp/udp parsing part, and
just return the src_str (is_v6 to avoid testing only arg1).
hmp_hostfwd_remove and hmp_ipv6_hostfwd_remove can then just start by
calling it and get src_str that it can then use to parse the IPs.
> + struct in6_addr host_addr = in6addr_any;
> + if (*(p++) != '[') {
> + goto fail_syntax;
> + }
As mentioned, better explicit this in the helper message.
> +static int slirp_ipv6_hostfwd(SlirpState *s, const char *redir_str,
> + Error **errp)
For this one I wouldn't bother factorizing, though.
Samuel
- Re: [Qemu-devel] [PATCH RFC 3/4] slirp: add helpers for ipv6 hostfwd manipulation, (continued)