KVM: selftests: kvm_vm_elf_load() and elfhdr_get() should close fd
authorReiji Watanabe <reijiw@google.com>
Tue, 20 Dec 2022 17:09:21 +0000 (09:09 -0800)
committerSean Christopherson <seanjc@google.com>
Tue, 24 Jan 2023 18:06:30 +0000 (10:06 -0800)
commitbf10993313e909bbf928bc40fd03141ffe23c7ec
treecdf17713756bba3b241c7d4390873f8e5b6e80d1
parent7cb79f433e75b05d1635aefaa851cfcd1cb7dc4f
KVM: selftests: kvm_vm_elf_load() and elfhdr_get() should close fd

kvm_vm_elf_load() and elfhdr_get() open one file each, but they
never close the opened file descriptor.  If a test repeatedly
creates and destroys a VM with __vm_create(), which
(directly or indirectly) calls those two functions, the test
might end up getting a open failure with EMFILE.
Fix those two functions to close the file descriptor.

Signed-off-by: Reiji Watanabe <reijiw@google.com>
Reviewed-by: Oliver Upton <oliver.upton@linux.dev>
Reviewed-by: Andrew Jones <andrew.jones@linux.dev>
Reviewed-by: Sean Christopherson <seanjc@google.com>
Link: https://lore.kernel.org/r/20221220170921.2499209-2-reijiw@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
tools/testing/selftests/kvm/lib/elf.c