From nobody Thu Nov 13 03:21:49 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1576525323; cv=none; d=zohomail.com; s=zohoarc; b=X1EFNnfX99cq1E2pHEAna6kyVupkidX+jLGBumrsjFXgmG9Tsbsk/TZWx3jyAkpGhjPcPgrLCLTFv+Tw6vV+roN6n7Ekgda/58qiKzyBc2LeYgcMMbw8DbWxXZUEnt6rC0vtXv9er3gKnHqrinBWZ/d2A2DCh6CloCD2Os7DwHc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1576525323; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=mqy5KY9RhCMejzmBW4QX+G2laitf5m3Gxcgdjfc/Kcg=; b=FHBGxlFMa3DgO7Fjiexj+mW4oB4SbEtSvdtAIv69Kh9uMRh6rEWvVPLjU3qlN0M6QXRJzk+eV7K4xzmOWYcWdcXsreMT0pqzcS+l0V2QmdCcpoNfpmgGnuvRPtcQt5pzA/K5komsVEDOfyoDSGyztIape5RsjVXKVqJRab37Brc= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1576525323191384.2166000011433; Mon, 16 Dec 2019 11:42:03 -0800 (PST) Received: from localhost ([::1]:59202 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1igwFl-0007AJ-P5 for importer@patchew.org; Mon, 16 Dec 2019 14:42:01 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:43769) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1igwCm-0003bm-HI for qemu-devel@nongnu.org; Mon, 16 Dec 2019 14:38:57 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1igwCl-00029z-59 for qemu-devel@nongnu.org; Mon, 16 Dec 2019 14:38:56 -0500 Received: from us-smtp-delivery-1.mimecast.com ([207.211.31.120]:34321 helo=us-smtp-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1igwCl-00029W-1d for qemu-devel@nongnu.org; Mon, 16 Dec 2019 14:38:55 -0500 Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-377-DekmTOiDM8GEkQqOPHhvkw-1; Mon, 16 Dec 2019 14:38:51 -0500 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id DE173DB2A; Mon, 16 Dec 2019 19:38:49 +0000 (UTC) Received: from localhost (ovpn-116-90.gru2.redhat.com [10.97.116.90]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6ADAF5D9D6; Mon, 16 Dec 2019 19:38:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1576525134; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=mqy5KY9RhCMejzmBW4QX+G2laitf5m3Gxcgdjfc/Kcg=; b=RGo+E0MP9HYbl2WjWHN0z4JvomCcQL5ie82LGM9G3v5xzM+iqbHUMJ2/RyMYGNz05aKjTv xAu/p4Cnta3CoN0aNAV2jNiu08b2yYIiYVStKSFksSYpH9sir7V4KNsJTqqY7pcZsox7mo 1cdfQIi0fY7ZncrPmZuujz9cinGdcGs= From: Eduardo Habkost To: qemu-devel@nongnu.org, Peter Maydell Subject: [PULL 4/4] i386: Use g_autofree in a few places Date: Mon, 16 Dec 2019 16:38:25 -0300 Message-Id: <20191216193825.1794153-5-ehabkost@redhat.com> In-Reply-To: <20191216193825.1794153-1-ehabkost@redhat.com> References: <20191216193825.1794153-1-ehabkost@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-MC-Unique: DekmTOiDM8GEkQqOPHhvkw-1 X-Mimecast-Spam-Score: 0 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 207.211.31.120 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Paolo Bonzini , Stefano Garzarella , Eduardo Habkost , Richard Henderson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Get rid of 12 explicit g_free() calls. Signed-off-by: Eduardo Habkost Message-Id: <20191025025632.5928-1-ehabkost@redhat.com> Reviewed-by: Stefano Garzarella Signed-off-by: Eduardo Habkost --- target/i386/cpu.c | 41 +++++++++++++---------------------------- 1 file changed, 13 insertions(+), 28 deletions(-) diff --git a/target/i386/cpu.c b/target/i386/cpu.c index de828e29d8..0a9ac65974 100644 --- a/target/i386/cpu.c +++ b/target/i386/cpu.c @@ -1671,11 +1671,8 @@ static char *x86_cpu_type_name(const char *model_nam= e) =20 static ObjectClass *x86_cpu_class_by_name(const char *cpu_model) { - ObjectClass *oc; - char *typename =3D x86_cpu_type_name(cpu_model); - oc =3D object_class_by_name(typename); - g_free(typename); - return oc; + g_autofree char *typename =3D x86_cpu_type_name(cpu_model); + return object_class_by_name(typename); } =20 static char *x86_cpu_class_get_model_name(X86CPUClass *cc) @@ -4226,7 +4223,6 @@ static void mark_unavailable_features(X86CPU *cpu, Fe= atureWord w, uint64_t mask, CPUX86State *env =3D &cpu->env; FeatureWordInfo *f =3D &feature_word_info[w]; int i; - char *feat_word_str; =20 if (!cpu->force_features) { env->features[w] &=3D ~mask; @@ -4239,13 +4235,12 @@ static void mark_unavailable_features(X86CPU *cpu, = FeatureWord w, uint64_t mask, =20 for (i =3D 0; i < 64; ++i) { if ((1ULL << i) & mask) { - feat_word_str =3D feature_word_description(f, i); + g_autofree char *feat_word_str =3D feature_word_description(f,= i); warn_report("%s: %s%s%s [bit %d]", verbose_prefix, feat_word_str, f->feat_names[i] ? "." : "", f->feat_names[i] ? f->feat_names[i] : "", i); - g_free(feat_word_str); } } } @@ -4747,17 +4742,14 @@ static gint x86_cpu_list_compare(gconstpointer a, g= constpointer b) ObjectClass *class_b =3D (ObjectClass *)b; X86CPUClass *cc_a =3D X86_CPU_CLASS(class_a); X86CPUClass *cc_b =3D X86_CPU_CLASS(class_b); - char *name_a, *name_b; int ret; =20 if (cc_a->ordering !=3D cc_b->ordering) { ret =3D cc_a->ordering - cc_b->ordering; } else { - name_a =3D x86_cpu_class_get_model_name(cc_a); - name_b =3D x86_cpu_class_get_model_name(cc_b); + g_autofree char *name_a =3D x86_cpu_class_get_model_name(cc_a); + g_autofree char *name_b =3D x86_cpu_class_get_model_name(cc_b); ret =3D strcmp(name_a, name_b); - g_free(name_a); - g_free(name_b); } return ret; } @@ -4795,9 +4787,9 @@ static void x86_cpu_list_entry(gpointer data, gpointe= r user_data) { ObjectClass *oc =3D data; X86CPUClass *cc =3D X86_CPU_CLASS(oc); - char *name =3D x86_cpu_class_get_model_name(cc); - char *desc =3D g_strdup(cc->model_description); - char *alias_of =3D x86_cpu_class_get_alias_of(cc); + g_autofree char *name =3D x86_cpu_class_get_model_name(cc); + g_autofree char *desc =3D g_strdup(cc->model_description); + g_autofree char *alias_of =3D x86_cpu_class_get_alias_of(cc); =20 if (!desc && alias_of) { if (cc->model && cc->model->version =3D=3D CPU_VERSION_AUTO) { @@ -4811,9 +4803,6 @@ static void x86_cpu_list_entry(gpointer data, gpointe= r user_data) } =20 qemu_printf("x86 %-20s %-48s\n", name, desc); - g_free(name); - g_free(desc); - g_free(alias_of); } =20 /* list available CPU models and flags */ @@ -5252,7 +5241,7 @@ static void x86_cpu_cpudef_class_init(ObjectClass *oc= , void *data) =20 static void x86_register_cpu_model_type(const char *name, X86CPUModel *mod= el) { - char *typename =3D x86_cpu_type_name(name); + g_autofree char *typename =3D x86_cpu_type_name(name); TypeInfo ti =3D { .name =3D typename, .parent =3D TYPE_X86_CPU, @@ -5261,14 +5250,12 @@ static void x86_register_cpu_model_type(const char = *name, X86CPUModel *model) }; =20 type_register(&ti); - g_free(typename); } =20 static void x86_register_cpudef_types(X86CPUDefinition *def) { X86CPUModel *m; const X86CPUVersionDefinition *vdef; - char *name; =20 /* AMD aliases are handled at runtime based on CPUID vendor, so * they shouldn't be set on the CPU model table. @@ -5288,11 +5275,11 @@ static void x86_register_cpudef_types(X86CPUDefinit= ion *def) =20 for (vdef =3D x86_cpu_def_get_versions(def); vdef->version; vdef++) { X86CPUModel *m =3D g_new0(X86CPUModel, 1); + g_autofree char *name =3D + x86_cpu_versioned_model_name(def, vdef->version); m->cpudef =3D def; m->version =3D vdef->version; - name =3D x86_cpu_versioned_model_name(def, vdef->version); x86_register_cpu_model_type(name, m); - g_free(name); =20 if (vdef->alias) { X86CPUModel *am =3D g_new0(X86CPUModel, 1); @@ -6364,9 +6351,8 @@ static void x86_cpu_realizefn(DeviceState *dev, Error= **errp) =20 if (xcc->host_cpuid_required) { if (!accel_uses_host_cpuid()) { - char *name =3D x86_cpu_class_get_model_name(xcc); + g_autofree char *name =3D x86_cpu_class_get_model_name(xcc); error_setg(&local_err, "CPU model '%s' requires KVM", name); - g_free(name); goto out; } =20 @@ -6482,10 +6468,9 @@ static void x86_cpu_realizefn(DeviceState *dev, Erro= r **errp) /* Cache information initialization */ if (!cpu->legacy_cache) { if (!xcc->model || !xcc->model->cpudef->cache_info) { - char *name =3D x86_cpu_class_get_model_name(xcc); + g_autofree char *name =3D x86_cpu_class_get_model_name(xcc); error_setg(errp, "CPU model '%s' doesn't support legacy-cache=3Doff"= , name); - g_free(name); return; } env->cache_info_cpuid2 =3D env->cache_info_cpuid4 =3D env->cache_i= nfo_amd =3D --=20 2.23.0