commit 0a4a1512e01228fc59b00d68e86f7099b6439773
author Markus Armbruster <armbru@redhat.com> Tue Oct 31 12:10:57 2023 +0100
committer Markus Armbruster <armbru@redhat.com> Fri Nov 17 10:07:52 2023 +0100
tree a21bc59eef0f08ceef7a6a941c9940bf243a09f6
parent 517b0220efd421acf885eed109571a61e95b192a

net: Fix a misleading error message

The error message

    $ qemu-system-x86_64 -netdev user,id=net0,ipv6-net=fec0::0/
    qemu-system-x86_64: -netdev user,id=net0,ipv6-net=fec0::0/: Parameter 'ipv6-prefixlen' expects a number

points to ipv6-prefixlen instead of ipv6-net.  Fix:

    qemu-system-x86_64: -netdev user,id=net0,ipv6-net=fec0::0/: parameter 'ipv6-net' expects a number after '/'

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-ID: <20231031111059.3407803-6-armbru@redhat.com>

net/net.c

diff --git a/net/net.c b/net/net.c
index c0c0cbe..8e67a20 100644
--- a/net/net.c
+++ b/net/net.c
@@ -1227,7 +1227,7 @@
     int ret = -1;
     Visitor *v = opts_visitor_new(opts);
 
-    /* Parse convenience option format ip6-net=fec0::0[/64] */
+    /* Parse convenience option format ipv6-net=fec0::0[/64] */
     const char *ip6_net = qemu_opt_get(opts, "ipv6-net");
 
     if (ip6_net) {
@@ -1247,8 +1247,8 @@
         if (substrings[1] &&
             qemu_strtoul(substrings[1], NULL, 10, &prefix_len))
         {
-            error_setg(errp, QERR_INVALID_PARAMETER_VALUE,
-                       "ipv6-prefixlen", "a number");
+            error_setg(errp,
+                       "parameter 'ipv6-net' expects a number after '/'");
             goto out;
         }