fix check of vcpuid in virDomainVcpuPinDefParseXML

For emulator, the vcpuid field is always set to -1, instead of parsing
XML for the value of it.

src/conf/domain_conf.c

@@ -7923,7 +7923,7 @@ virDomainVcpuPinDefParseXML(const xmlNodePtr node,
 {
     virDomainVcpuPinDefPtr def;
     xmlNodePtr oldnode = ctxt->node;
-    int vcpuid;
+    int vcpuid = -1;
     char *tmp = NULL;
     int ret;
 
@@ -7934,16 +7934,18 @@ virDomainVcpuPinDefParseXML(const xmlNodePtr node,
 
     ctxt->node = node;
 
+    if (emulator == 0) {
         ret = virXPathInt("string(./@vcpu)", ctxt, &vcpuid);
         if ((ret == -2) || (vcpuid < -1)) {
             virReportError(VIR_ERR_INTERNAL_ERROR,
                            "%s", _("vcpu id must be an unsigned integer or -1"));
             goto error;
-    } else if ((vcpuid == -1) && (emulator == 0)) {
+        } else if (vcpuid == -1) {
             virReportError(VIR_ERR_INTERNAL_ERROR,
                            "%s", _("vcpu id value -1 is not allowed for vcpupin"));
             goto error;
         }
+    }
 
     if (vcpuid >= maxvcpus) {
         virReportError(VIR_ERR_INTERNAL_ERROR,