[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 2/4] conf: Parse and format disk <wwn>



Il 29/08/2012 18:36, Osier Yang ha scritto:
> @@ -11557,6 +11564,7 @@ virDomainDiskDefFormat(virBufferPtr buf,
>      if (def->transient)
>          virBufferAddLit(buf, "      <transient/>\n");
>      virBufferEscapeString(buf, "      <serial>%s</serial>\n", def->serial);
> +    virBufferEscapeString(buf, "      <wwn>%s</wwn>\n", def->wwn);

What happens if the wwn was not passed?

If it is NULL, it's a bug (same for serial).

If it is empty, the output will not obey the schema, so the whole
element should be left out.  I think it's best to do the same for
serial, but technically it should be ok.

Paolo


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]