Commit cf2887c9 authored by Paolo Bonzini's avatar Paolo Bonzini Committed by Eduardo Habkost
Browse files

target-i386: Avoid using locals outside their scope 



x86_cpu_parse_featurestr has a "val = num;" assignment just before num
goes out of scope.  Push num up to fix the issue.

Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
Reviewed-by: default avatarIgor Mammedov <imammedo@redhat.com>
Reviewed-by: default avatarEduardo Habkost <ehabkost@redhat.com>
Signed-off-by: default avatarEduardo Habkost <ehabkost@redhat.com>
parent 7eb24386

target-i386/cpu.c

+1 −1
Original line number Diff line number Diff line
@@ -1975,6 +1975,7 @@ static void x86_cpu_parse_featurestr(CPUState *cs, char *features, 
        const char *name;

        const char *val = NULL;

        char *eq = NULL;

        char num[32];



        /* Compatibility syntax: */

        if (featurestr[0] == '+') {
@@ -2000,7 +2001,6 @@ static void x86_cpu_parse_featurestr(CPUState *cs, char *features, 
        if (!strcmp(name, "tsc-freq")) {

            int64_t tsc_freq;

            char *err;

            char num[32];



            tsc_freq = qemu_strtosz_suffix_unit(val, &err,

                                           QEMU_STRTOSZ_DEFSUFFIX_B, 1000);