commit 8301ea444abb49f7b7fb939b09c1e23b22977f30
author Philippe Mathieu-Daudé <f4bug@amsat.org> Sun Sep 17 20:28:42 2017 -0300
committer Eduardo Habkost <ehabkost@redhat.com> Mon Oct 09 23:21:52 2017 -0300
tree 22dc7963464b3615aa96e96d6d873f1fb072c74d
parent 530049bc1dcc24c1178a29d99ca08b6dd08413e0

qom/cpu: move cpu_model null check to cpu_class_by_name()

and clean every implementation.

Suggested-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20170917232842.14544-1-f4bug@amsat.org>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Artyom Tarasenko <atar4qemu@gmail.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>

qom/cpu.c

diff --git a/qom/cpu.c b/qom/cpu.c
index 94fa8fe..54c9452 100644
--- a/qom/cpu.c
+++ b/qom/cpu.c
@@ -316,7 +316,12 @@
 
 ObjectClass *cpu_class_by_name(const char *typename, const char *cpu_model)
 {
-    CPUClass *cc = CPU_CLASS(object_class_by_name(typename));
+    CPUClass *cc;
+
+    if (!cpu_model) {
+        return NULL;
+    }
+    cc = CPU_CLASS(object_class_by_name(typename));
 
     return cc->class_by_name(cpu_model);
 }