[Qemu-devel] [PATCH v2 0/4] Clean ups in net/net.c

Stefano Garzarella posted 4 patches 4 years, 11 months ago
Test docker-mingw@fedora failed
Test asan failed
Test docker-clang@ubuntu failed
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190509130345.227526-1-sgarzare@redhat.com
Maintainers: Jason Wang <jasowang@redhat.com>
There is a newer version of this series
net/net.c | 101 +++++++++++++++++++++++++++---------------------------
1 file changed, 51 insertions(+), 50 deletions(-)
[Qemu-devel] [PATCH v2 0/4] Clean ups in net/net.c
Posted by Stefano Garzarella 4 years, 11 months ago
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/qemu-devel@nongnu.org/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(-)

-- 
2.20.1