selftests: net: udpgso_bench_rx/tx: Stop when wrong CLI args are provided
authorAndrei Gherzan <andrei.gherzan@canonical.com>
Wed, 1 Feb 2023 00:16:12 +0000 (00:16 +0000)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 2 Feb 2023 12:29:51 +0000 (13:29 +0100)
Leaving unrecognized arguments buried in the output, can easily hide a
CLI/script typo. Avoid this by exiting when wrong arguments are provided to
the udpgso_bench test programs.

Fixes: 3a687bef148d ("selftests: udp gso benchmark")
Signed-off-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Cc: Willem de Bruijn <willemb@google.com>
Reviewed-by: Willem de Bruijn <willemb@google.com>
Link: https://lore.kernel.org/r/20230201001612.515730-2-andrei.gherzan@canonical.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
tools/testing/selftests/net/udpgso_bench_rx.c
tools/testing/selftests/net/udpgso_bench_tx.c

index d0895bd1933f2761b0b100461c384789fec042f0..4058c7451e70da721c50c0f214054abe4a94d782 100644 (file)
@@ -336,6 +336,8 @@ static void parse_opts(int argc, char **argv)
                        cfg_verify = true;
                        cfg_read_all = true;
                        break;
+               default:
+                       exit(1);
                }
        }
 
index f1fdaa270291331e70fb2456003bdd44056b93d5..b47b5c32039f1f4a61535f8dd706ac455bd366e7 100644 (file)
@@ -490,6 +490,8 @@ static void parse_opts(int argc, char **argv)
                case 'z':
                        cfg_zerocopy = true;
                        break;
+               default:
+                       exit(1);
                }
        }