Skip to content

Commit 7712145

Browse files
GeHao01994sean-jc
authored andcommitted
KVM: selftests: Close opened file descriptor in stable_tsc_check_supported()
Close the "current_clocksource" file descriptor before returning or exiting from stable_tsc_check_supported() in vmx_nested_tsc_scaling_test. Signed-off-by: Hao Ge <gehao@kylinos.cn> Reviewed-by: Vipin Sharma <vipinsh@google.com> Link: https://lore.kernel.org/r/20230405101350.259000-1-gehao@kylinos.cn Signed-off-by: Sean Christopherson <seanjc@google.com>
1 parent d563164 commit 7712145

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

tools/testing/selftests/kvm/x86_64/vmx_nested_tsc_scaling_test.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,12 +126,16 @@ static void stable_tsc_check_supported(void)
126126
goto skip_test;
127127

128128
if (fgets(buf, sizeof(buf), fp) == NULL)
129-
goto skip_test;
129+
goto close_fp;
130130

131131
if (strncmp(buf, "tsc", sizeof(buf)))
132-
goto skip_test;
132+
goto close_fp;
133133

134+
fclose(fp);
134135
return;
136+
137+
close_fp:
138+
fclose(fp);
135139
skip_test:
136140
print_skip("Kernel does not use TSC clocksource - assuming that host TSC is not stable");
137141
exit(KSFT_SKIP);

0 commit comments

Comments
 (0)