From 5b403dbfc4133e6d9ecb1fc8f2a88b3d245c37e6 Mon Sep 17 00:00:00 2001 From: Will Deacon Date: Mon, 12 Nov 2012 11:57:21 +0000 Subject: [PATCH] kvm tools: die if init_list__init returns failure If init_list__init returns failure when initialising kvm tool, we can't rely on our structures being fully initialised, so die rather than try to continue and fail gracefully later on. This prevents a SEGV when kvm is not available on the host: Error: '/dev/kvm' not found. Please make sure your kernel has CONFIG_KVM enabled and that the KVM modules are loaded. Warning: Failed init: kvm__init Segmentation fault Signed-off-by: Will Deacon Signed-off-by: Pekka Enberg --- builtin-run.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/builtin-run.c b/builtin-run.c index dec2e74..96e68af 100644 --- a/builtin-run.c +++ b/builtin-run.c @@ -665,7 +665,8 @@ static struct kvm *kvm_cmd_run_init(int argc, const char **argv) printf(" # %s run -k %s -m %Lu -c %d --name %s\n", KVM_BINARY_NAME, kvm->cfg.kernel_filename, kvm->cfg.ram_size / 1024 / 1024, kvm->cfg.nrcpus, kvm->cfg.guest_name); - init_list__init(kvm); + if (init_list__init(kvm) < 0) + die ("Initialisation failed"); return kvm; }