From 1941db93a0ae5c4c54e2342075e8e9cad22a07dc Mon Sep 17 00:00:00 2001 From: Ben Guthro Date: Thu, 3 Nov 2011 11:06:56 -0400 Subject: [PATCH] x86/microcode: check proper return code. After pulling in this change from your tree, I found the following bug, when checking an enum value, which should be considered before inclusion: Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/kernel/microcode_xen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/microcode_xen.c b/arch/x86/kernel/microcode_xen.c index 9d2a06b5bf10..6a73957dd187 100644 --- a/arch/x86/kernel/microcode_xen.c +++ b/arch/x86/kernel/microcode_xen.c @@ -157,7 +157,7 @@ static enum ucode_state xen_request_microcode_user(int cpu, ret = UCODE_OK; out: - if (ret == 0) + if (ret == UCODE_OK) uci->mc = uc; else vfree(uc); -- 2.50.1