1.remove unused variable env 2.remove redundant error handling Signed-off-by: liguang <lig.f...@cn.fujitsu.com> --- target-i386/helper.c | 17 ++++++++--------- 1 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/target-i386/helper.c b/target-i386/helper.c index bf206cf..5686130 100644 --- a/target-i386/helper.c +++ b/target-i386/helper.c @@ -1243,25 +1243,24 @@ int cpu_x86_get_descr_debug(CPUX86State *env, unsigned int selector, X86CPU *cpu_x86_init(const char *cpu_model) { X86CPU *cpu; - CPUX86State *env; Error *error = NULL; cpu = X86_CPU(object_new(TYPE_X86_CPU)); - env = &cpu->env; - env->cpu_model_str = cpu_model; - if (cpu_x86_register(cpu, cpu_model) < 0) { - object_delete(OBJECT(cpu)); - return NULL; - } + if (cpu_x86_register(cpu, cpu_model) < 0) + goto error_out; x86_cpu_realize(OBJECT(cpu), &error); if (error) { error_free(error); - object_delete(OBJECT(cpu)); - return NULL; + goto error_out; } + return cpu; + + error_out: + object_delete(OBJECT(cpu)); + return NULL; } #if !defined(CONFIG_USER_ONLY) -- 1.7.2.5