diff options
author | Benjamin Poirier <bpoirier@nvidia.com> | 2023-12-11 09:07:16 -0500 |
---|---|---|
committer | Stephen Hemminger <stephen@networkplumber.org> | 2023-12-22 09:54:23 -0800 |
commit | aeb7ee297361fa9af0b3931d799180b2ec595617 (patch) | |
tree | 9b81c589e1a9929b795683fcf51628095bec4a05 | |
parent | 0b8c01b4058eff19e88da4dfa0f598a9c24a8d3b (diff) | |
download | iproute2-aeb7ee297361fa9af0b3931d799180b2ec595617.tar.gz |
bridge: vni: Report duplicate vni argument using duparg()
When there is a duplicate 'vni' option, report the error using duparg()
instead of the generic invarg().
Before:
$ bridge vni add vni 100 vni 101 dev vxlan2
Error: argument "101" is wrong: duplicate vni
After:
$ ./bridge/bridge vni add vni 100 vni 101 dev vxlan2
Error: duplicate "vni": "101" is the second value.
Fixes: 45cd32f9f7d5 ("bridge: vxlan device vnifilter support")
Reviewed-by: Petr Machata <petrm@nvidia.com>
Tested-by: Petr Machata <petrm@nvidia.com>
Signed-off-by: Benjamin Poirier <bpoirier@nvidia.com>
Acked-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
-rw-r--r-- | bridge/vni.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bridge/vni.c b/bridge/vni.c index 56def2f79..ecd4c2b5a 100644 --- a/bridge/vni.c +++ b/bridge/vni.c @@ -104,7 +104,7 @@ static int vni_modify(int cmd, int argc, char **argv) } else if (strcmp(*argv, "vni") == 0) { NEXT_ARG(); if (vni) - invarg("duplicate vni", *argv); + duparg("vni", *argv); vni = *argv; } else if (strcmp(*argv, "group") == 0) { NEXT_ARG(); |