Re: [libvirt] [PATCHv2 15/27] uml: reject unknown flags

[Matthias Bolte <matthias bolte googlemail com>]

2011/7/8 Eric Blake <eblake redhat com>:
> * src/uml/uml_driver.c (umlOpen, umlDomainGetXMLDesc)
> (umlDomainBlockPeek): Reject unknown flags.
> ---
>  src/uml/uml_driver.c |   14 +++++++++++---
>  1 files changed, 11 insertions(+), 3 deletions(-)
>

> @@ -1559,11 +1562,14 @@ cleanup:
>
>
>  static char *umlDomainGetXMLDesc(virDomainPtr dom,
> -                                 unsigned int flags ATTRIBUTE_UNUSED) {
> +                                 unsigned int flags)
> +{
>     struct uml_driver *driver = dom->conn->privateData;
>     virDomainObjPtr vm;
>     char *ret = NULL;
>
> +    virCheckFlags(0, NULL);
> +
>     umlDriverLock(driver);
>     vm = virDomainFindByUUID(&driver->domains, dom->uuid);
>     umlDriverUnlock(driver);

Don't get fooled by the ATTRIBUTE_UNUSED again. All *DomainGetXMLDesc
use virDomainDefFormat and have to accept all flags that
virDomainDefFormat accepts. I suggest to recheck your series for this
pattern, here it's just the first time that I notice it.

ACK, with that virCheckFlags loosened correctly.

-- 
Matthias Bolte
http://photron.blogspot.com