]> xenbits.xensource.com Git - libvirt.git/commitdiff
Fix indentation of sysinfo data
authorDaniel P. Berrange <berrange@redhat.com>
Thu, 23 Oct 2014 17:35:57 +0000 (18:35 +0100)
committerDaniel P. Berrange <berrange@redhat.com>
Fri, 24 Oct 2014 16:23:52 +0000 (17:23 +0100)
The <sysinfo> data block was indented by 2 spaces too many.
This was missed because we never had any test validating
the XML formatting.

src/conf/domain_conf.c
tests/qemuxml2argvdata/qemuxml2argv-smbios.xml
tests/qemuxml2xmltest.c

index 73c199e80bb2793191e90fc9c634afc5b43b2427..39befb0c2ee59f19c5afb370fc61593ac72f965f 100644 (file)
@@ -17668,17 +17668,6 @@ virDomainNVRAMDefFormat(virBufferPtr buf,
     return 0;
 }
 
-static int
-virDomainSysinfoDefFormat(virBufferPtr buf,
-                          virSysinfoDefPtr def)
-{
-    int ret;
-    virBufferAdjustIndent(buf, 2);
-    ret = virSysinfoFormat(buf, def);
-    virBufferAdjustIndent(buf, -2);
-    return ret;
-}
-
 
 static int
 virDomainWatchdogDefFormat(virBufferPtr buf,
@@ -18890,7 +18879,7 @@ virDomainDefFormatInternal(virDomainDefPtr def,
         virDomainResourceDefFormat(buf, def->resource);
 
     if (def->sysinfo)
-        virDomainSysinfoDefFormat(buf, def->sysinfo);
+        virSysinfoFormat(buf, def->sysinfo);
 
     if (def->os.bootloader) {
         virBufferEscapeString(buf, "<bootloader>%s</bootloader>\n",
index 91b2c10cd4f9d4c96dcad782a060654c2fdffed9..a2caeeaf3f5300d4bdc487d4c09b96549b05a37b 100644 (file)
@@ -36,6 +36,8 @@
       <address type='drive' controller='0' bus='0' target='0' unit='0'/>
     </disk>
     <controller type='ide' index='0'/>
+    <controller type='usb' index='0'/>
+    <controller type='pci' index='0' model='pci-root'/>
     <memballoon model='virtio'/>
   </devices>
 </domain>
index 978e5f1c6aa526840e0b785848b0646503f3f964..0ec4303887b0d5a01975af83a60b6f27d1dc1340 100644 (file)
@@ -405,6 +405,7 @@ mymain(void)
 
     DO_TEST("tap-vhost");
     DO_TEST("shmem");
+    DO_TEST("smbios");
 
     virObjectUnref(driver.caps);
     virObjectUnref(driver.xmlopt);