]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
KVM: selftests: Add missing "break" in rseq_test's param parsing
authorGavin Shan <gshan@redhat.com>
Mon, 24 Nov 2025 05:04:27 +0000 (15:04 +1000)
committerSean Christopherson <seanjc@google.com>
Tue, 2 Dec 2025 16:49:10 +0000 (08:49 -0800)
In commit 0297cdc12a87 ("KVM: selftests: Add option to rseq test to
override /dev/cpu_dma_latency"), a 'break' is missed before the option
'l' in the argument parsing loop, which leads to an unexpected core
dump in atoi_paranoid(). It tries to get the latency from non-existent
argument.

  host$ ./rseq_test -u
  Random seed: 0x6b8b4567
  Segmentation fault (core dumped)

Add a 'break' before the option 'l' in the argument parsing loop to avoid
the unexpected core dump.

Fixes: 0297cdc12a87 ("KVM: selftests: Add option to rseq test to override /dev/cpu_dma_latency")
Cc: stable@vger.kernel.org # v6.15+
Signed-off-by: Gavin Shan <gshan@redhat.com>
Link: https://patch.msgid.link/20251124050427.1924591-1-gshan@redhat.com
[sean: describe code change in shortlog]
Signed-off-by: Sean Christopherson <seanjc@google.com>
tools/testing/selftests/kvm/rseq_test.c

index 1375fca80bcdbe2533f9a6f7b85e96f069a1fd6b..f80ad6b47d16b0d83e044d661ad8494cc67f2ccc 100644 (file)
@@ -215,6 +215,7 @@ int main(int argc, char *argv[])
                switch (opt) {
                case 'u':
                        skip_sanity_check = true;
+                       break;
                case 'l':
                        latency = atoi_paranoid(optarg);
                        break;