On 01/07/15 10:36, Luyao Huang wrote:
> We will free the old parameter value settings in next loop when we
> get scsi controller's driver specific options.
This description doesn't really explain what is the actual problem you
are fixing.
Please be more specific and as this is in the XML parsing code it should
be fairly easy to do a test case for the scenario.
>
> Signed-off-by: Luyao Huang <lhuang@xxxxxxxxxx>
> ---
> src/conf/domain_conf.c | 16 +++++++++++++---
> 1 file changed, 13 insertions(+), 3 deletions(-)
>
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index b9858cd..f7b4a7c 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -6870,11 +6870,21 @@ virDomainControllerDefParseXML(xmlNodePtr node,
>
> cur = node->children;
> while (cur != NULL) {
> + char *queues2 = NULL;
> + char *cmd_per_lun2 = NULL;
> + char *max_sectors2 = NULL;
> +
> if (cur->type == XML_ELEMENT_NODE) {
> if (xmlStrEqual(cur->name, BAD_CAST "driver")) {
> - queues = virXMLPropString(cur, "queues");
> - cmd_per_lun = virXMLPropString(cur, "cmd_per_lun");
> - max_sectors = virXMLPropString(cur, "max_sectors");
> + queues2 = virXMLPropString(cur, "queues");
> + if (queues2)
> + queues = queues2;
> + cmd_per_lun2 = virXMLPropString(cur, "cmd_per_lun");
> + if (cmd_per_lun2)
> + cmd_per_lun = cmd_per_lun2;
> + max_sectors2 = virXMLPropString(cur, "max_sectors");
> + if (max_sectors2)
> + max_sectors = max_sectors2;
> }
> }
> cur = cur->next;
>
Without a proper explanation it's hard to see if the fix is right.
Peter
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list