Commit bfad6739 authored by Jim Meyering's avatar Jim Meyering Committed by Anthony Liguori
Browse files

sparc: use g_strdup in place of unchecked strdup 



This avoids a NULL-deref upon strdup failure.
Also update matching free to g_free.

Signed-off-by: default avatarJim Meyering <meyering@redhat.com>
Signed-off-by: default avatarAnthony Liguori <aliguori@us.ibm.com>
parent a5cf8262

target-sparc/cpu.c

+2 −2
Original line number Diff line number Diff line
@@ -643,7 +643,7 @@ static int cpu_sparc_find_by_name(sparc_def_t *cpu_def, const char *cpu_model) 
{

    unsigned int i;

    const sparc_def_t *def = NULL;

    char *s = strdup(cpu_model);

    char *s = g_strdup(cpu_model);

    char *featurestr, *name = strtok(s, ",");

    uint32_t plus_features = 0;

    uint32_t minus_features = 0;
@@ -735,7 +735,7 @@ static int cpu_sparc_find_by_name(sparc_def_t *cpu_def, const char *cpu_model) 
#ifdef DEBUG_FEATURES

    print_features(stderr, fprintf, cpu_def->features, NULL);

#endif

    free(s);

    g_free(s);

    return 0;



 error: