[v2,0/4] Clean ups in net/net.c

Message

Stefano Garzarella May 9, 2019, 1:03 p.m. UTC

This series is a follow up of "[PATCH] net: avoid to use variable length
array in net_client_init()" [1], so it contains the v2 of that patch,
plus other new related patches.

I discovered an assertion failure while I was testing the patches, so I
added the patch 1 to solve this issue.

Following the Markus' advice, I modified the parsing of IPv6 prefix
(patch 2) and IPv4 host:port (patch 3). Then I removed the get_str_sep()
function (patch 4) because it is no longer used.

@Markus, I modified a little bit what you suggested, introducing
g_strsplit() in order to de-duplicate the qemu_opt_set() and
qemu_opt_set_number(). I hope it's good for you.

v2:
 - Added patches 1,3 and 4
 - Patch 2:
   - clean up parsing of IPv6 prefix [Markus]
   - fixed subject line [Eric]

[1] https://www.mail-archive.com/[e-mail]/msg614561.html

Stefano Garzarella (4):
  net: fix assertion failure when ipv6-prefixlen is not a number
  net: avoid using variable length array in net_client_init()
  net: use g_strsplit() for parsing host address and port
  net: remove unused get_str_sep() function

 net/net.c | 101 +++++++++++++++++++++++++++---------------------------
 1 file changed, 51 insertions(+), 50 deletions(-)