[Libvirt-cim] [libvirt-cim] [PATCH] remove script for bridge network

[snmishra at linux.vnet.ibm.com]

Quoting Wenchao Xia <xiawenc at linux.vnet.ibm.com>:

> Hi, Danial
>    This is a urgent fix for bug in new release, could u help look at it?
>
>>    libvirt0.9.10 will report error if bridge network was defined
>> with script. This is the fix for it, otherwise VM start would fail.
>>
>> Signed-off-by: Wenchao Xia <xiawenc at linux.vnet.ibm.com>
>> ---
>>   libxkutil/xmlgen.c |    9 +--------
>>   1 files changed, 1 insertions(+), 8 deletions(-)
>>
>> diff --git a/libxkutil/xmlgen.c b/libxkutil/xmlgen.c
>> index 2dcd0d2..31619d8 100644
>> --- a/libxkutil/xmlgen.c
>> +++ b/libxkutil/xmlgen.c
>> @@ -269,18 +269,11 @@ static const char *set_net_source(xmlNodePtr nic,
>>           return NULL;
>>   }
>>
>> -
>> +/* libvirt 0.9.10 report error if script is set with brdige */
>>   static const char *bridge_net_to_xml(xmlNodePtr nic, struct  
>> net_device *dev)
>>   {
>> -        const char *script = "vif-bridge";
>> -        xmlNodePtr tmp;
>>           const char *msg = NULL;
>>
>> -        tmp = xmlNewChild(nic, NULL, BAD_CAST "script", NULL);
>> -        if (tmp == NULL)
>> -                return XML_ERROR;
>> -        xmlNewProp(tmp, BAD_CAST "path", BAD_CAST script);
>> -
>>           msg = set_net_source(nic, dev, "bridge");
>>
>>           return msg;

There is nothing much left in this function after removing those  
lines. So, do you still need this funcion or can the remaining code be  
moved some other place?

Also, please make sure that removing "vif-bridge" does not break any  
other code path.

-Sharad Mishra
>>
>
>
> --
> Best Regards
>
> Wenchao Xia
>
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim at redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim